Skip to content

Add Options::emit_anchor_events (default false) to emit Event::Anchor(id, name) - #49

Merged
bourumir-wyngs merged 4 commits into
masterfrom
bw/anchor_event
Oct 29, 2026
Merged

bourumir-wyngs merged 4 commits into
masterfrom
bw/anchor_event

Conversation

@bourumir-wyngs

Copy link
Copy Markdown
Owner

Add Options::emit_anchor_events (default false) to emit Event::Anchor(id, name)
immediately after the anchored scalar or collection-start event. Its span covers the complete
&name token, and its ID matches the node and resolved aliases without relying on allocation
order. Anchor events are presentation metadata (is_node() is false); existing node payloads
and default event streams are unchanged. Together with alias-token spans and Span::tag_span(),
this exposes original property spellings without separate anchor and tag scans.

@codspeed

codspeed Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

1 warning

⚠️ 1 benchmark is dominated by system calls

System calls cannot be simulated consistently and are left out of the measure. The reported value understates the real cost. Switch to the Walltime instrument to measure them.

Learn more about system calls

See affected benchmarks

✅ 13 untouched benchmarks


Comparing bw/anchor_event (55a7e91) with master (83a0e12)

Open in CodSpeed

@codecov

codecov Bot commented Oct 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@bourumir-wyngs
bourumir-wyngs merged commit 3347154 into master Oct 29, 2026
25 checks passed
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