Skip to content

Commit

Permalink
internal: remove MaxDepth
Browse files Browse the repository at this point in the history
Unused since 2020, since the evaluator now does have cycle detection.

Signed-off-by: Daniel Martí <[email protected]>
Change-Id: I80f43267f68848c6ad014d2be1a7d17312e42eee
Dispatch-Trailer: {"type":"trybot","CL":1200972,"patchset":1,"ref":"refs/changes/72/1200972/1","targetBranch":"master"}
  • Loading branch information
mvdan authored and cueckoo committed Sep 10, 2024
1 parent 9ab3787 commit c6479e7
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions internal/internal.go
Original file line number Diff line number Diff line change
Expand Up @@ -445,12 +445,3 @@ type decorated struct {
func (e *decorated) Is(err error) bool {
return errors.Is(e.info, err) || errors.Is(e.cueError, err)
}

// MaxDepth indicates the maximum evaluation depth. This is there to break
// cycles in the absence of cycle detection.
//
// It is registered in a central place to make it easy to find all spots where
// cycles are broken in this brute-force manner.
//
// TODO(eval): have cycle detection.
const MaxDepth = 20

0 comments on commit c6479e7

Please sign in to comment.