this repo has no description
0
fork

Configure Feed

Select the types of activity you want to include in your feed.

internal/core/adt: remove a piece of code in isFinal only needed for evalv2

As suggested by the TODO. In a separate commit, for easier bisection
if it ends up causing any issues.

Signed-off-by: Daniel Martí <mvdan@mvdan.cc>
Change-Id: Ia2419403874e2ffb1aca6300574862cc7a22286c
Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1220047
TryBot-Result: CUEcueckoo <cueckoo@cuelang.org>
Reviewed-by: Marcel van Lohuizen <mpvl@gmail.com>
Unity-Result: CUE porcuepine <cue.porcuepine@gmail.com>

-3
-3
internal/core/adt/composite.go
··· 852 852 case *Vertex: 853 853 closed := isClosed || x.ClosedNonRecursive || x.ClosedRecursive 854 854 855 - // TODO(evalv3): this is for V2 compatibility. Remove once V2 is gone. 856 - closed = closed || x.IsClosedList() || x.IsClosedStruct() 857 - 858 855 // This also dereferences the value. 859 856 if v, ok := x.BaseValue.(Value); ok { 860 857 return isFinal(v, closed)