Skip to content

Conversation

dmlloyd
Copy link
Member

@dmlloyd dmlloyd commented May 12, 2025

An item is bound if it is added to the list. Constants are never bound. Derefs are bound once they have been added iff the thing they refer to is bound. Bound items may be used only once. Derefs of constants (or derefs of deref of constants etc.) can be reused many times.

Fixes #372

@Ladicek
Copy link
Contributor

Ladicek commented May 13, 2025

Your code LGTM, but my commit is not directly mergeable :-) Feel free to squash them, or take the commit from https://github.com/Ladicek/gizmo/commits/array-indexing-bug_v2/

An item is bound if it is added to the list. Constants are never bound. Derefs are bound once they have been added iff the thing they refer to is bound. Bound items may be used only once. Derefs of constants (or derefs of deref of constants etc.) can be reused many times.

Fixes quarkusio#372
@dmlloyd dmlloyd force-pushed the array-indexing-bug branch from 5628d35 to 4103458 Compare May 13, 2025 12:00
@Ladicek Ladicek moved this to In Progress in WG - Gizmo 2 May 13, 2025
@Ladicek Ladicek merged commit 642fe41 into quarkusio:main May 13, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in WG - Gizmo 2 May 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

array indexing fails when the index is a bound item
2 participants