Skip to content

NO MERGE Fix ACTZipper::val_count counting from the root - #110

Draft
imlvts wants to merge 1 commit into
Adam-Vandervorst:masterfrom
imlvts:fix/act-val-count-focus
Draft

imlvts wants to merge 1 commit into
Adam-Vandervorst:masterfrom
imlvts:fix/act-val-count-focus

Conversation

@imlvts

@imlvts imlvts commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

Don't merge because likely gets fixed by cata rework

val_count reset the zipper first, so it counted every value below the zipper root wherever the focus was. Count at and below the focus.

val_count reset the zipper first, so it counted every value below the
zipper root wherever the focus was. Count at and below the focus.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019R2H8fnco29asY2v3TPbtF
@imlvts
imlvts marked this pull request as draft September 19, 2026 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant