Skip to content

Conversation

@lentinj
Copy link
Collaborator

@lentinj lentinj commented Aug 8, 2025

Merge the ~unrelated tidy-ups and/or debugging helpers from #981 / #972

They won't help solve the problem, but some QoL improvements whilst trying to debug what's going on.

lentinj added 6 commits August 7, 2025 14:16
All the documentation says 0x1 is label, but was actually 0x2. Swap
them around.
Add toString() as a repr-like method to use in debugging. Show how to
use it in console.log & use it as the default debug label for nodes.
Early exit for the unlikely !node.dvar case, remove length assignments,
document code blocks.
The main reanchor() doesn't need a deanchor() first, so neither should
reanchor_at_node().

This also means that the deanchor export is no longer required.
Combine them together to reduce mental load. And it doesn't check that
the node is *in* (i.e. contained by), but *on* (i.e. a bit is visible).
@lentinj lentinj merged commit c940e0a into main Aug 8, 2025
1 check passed
@lentinj lentinj deleted the issue-972-debugging-tidyup branch August 8, 2025 15:11
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