Skip to content

Commit

Permalink
internal/core/adt: fix structural cycle error in list.Sum
Browse files Browse the repository at this point in the history
The sum in the changed test was summing itself, which
is a structural cycle. The old behavior was to give
a garbage result. We now correctly report the structural
cycle.

Issue #3310

Signed-off-by: Marcel van Lohuizen <mpvl@gmail.com>
Change-Id: Ia37a3a0adf373e9455dc921916a1352ebd580b44
Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1202444
Reviewed-by: Matthew Sackman <matthew@cue.works>
TryBot-Result: CUEcueckoo <cueckoo@cuelang.org>
Unity-Result: CUE porcuepine <cue.porcuepine@gmail.com>
  • Loading branch information
mpvl committed Oct 14, 2024
1 parent cf2f66e commit 48b5a22
Show file tree
Hide file tree
Showing 2 changed files with 67 additions and 49 deletions.
Loading

0 comments on commit 48b5a22

Please sign in to comment.