Skip to content

Fix/overlay zipper root and descend to val - #144

Open
imlvts wants to merge 2 commits into
Adam-Vandervorst:masterfrom
imlvts:fix/overlay-zipper-root-and-descend-to-val
Open

imlvts wants to merge 2 commits into
Adam-Vandervorst:masterfrom
imlvts:fix/overlay-zipper-root-and-descend-to-val

Conversation

@imlvts

@imlvts imlvts commented Sep 29, 2026

Copy link
Copy Markdown
Collaborator

Fix OverlayZipper focus_byte at the root
Its sources may be rooted at different paths, so at the root their focus
bytes differ and the debug assert failed; sibling steps then tried to
move the root. Report None at the root.

Fix OverlayZipper::descend_to_val when the second source stops first
The branch meant to bring the second source down to the first one's
depth descended the first source again, leaving them at different depths.

Its sources may be rooted at different paths, so at the root their focus
bytes differ and the debug assert failed; sibling steps then tried to
move the root. Report None at the root.
The branch meant to bring the second source down to the first one's
depth descended the first source again, leaving them at different depths.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant