Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Messed up size accounting panic #1584

Closed
yav opened this issue Oct 26, 2023 · 1 comment
Closed

Messed up size accounting panic #1584

yav opened this issue Oct 26, 2023 · 1 comment
Labels
bug Something not working correctly

Comments

@yav
Copy link
Member

yav commented Oct 26, 2023

The following expression causes a panic:

[ x+y | x <- [(0:Integer)...], y <- [1] ]
 Location:  lookupSeqMap
  Message:   Messed up size accounting
             Nat 1
             1
@yav yav added the bug Something not working correctly label Oct 26, 2023
@yav
Copy link
Member Author

yav commented Oct 26, 2023

After merging PR #1557, this does not panic, but there is still something wrong with the list comprehension. In particular,
indexing in it returns the correct result (e.g., test @ i works fine for any i), but trying to just print the expression, or take more than one element complainsabout invalid sequence index: 1

@yav yav mentioned this issue Oct 27, 2023
@yav yav closed this as completed in d9ea068 Nov 1, 2023
yav added a commit that referenced this issue Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something not working correctly
Projects
None yet
Development

No branches or pull requests

1 participant