Skip to content

Fix ProductZipper::val_count outside the last factor - #130

Open
imlvts wants to merge 1 commit into
Adam-Vandervorst:masterfrom
imlvts:fix/product-zipper-val-count
Open

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

Conversation

@imlvts

@imlvts imlvts commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

val_count delegated to the core zipper, which only sees the current factor, and panicked elsewhere. Count by walking a copy of the product zipper below the focus.

Maybe this belongs to new cata?

val_count delegated to the core zipper, which only sees the current
factor, and panicked elsewhere. Count by walking a copy of the product
zipper below the focus.
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