Skip to content

feat: update ast to use arenas, add roxmltree parsing#123

Merged
noahbald merged 11 commits intomainfrom
feat/arenas
Apr 13, 2025
Merged

feat: update ast to use arenas, add roxmltree parsing#123
noahbald merged 11 commits intomainfrom
feat/arenas

Conversation

@noahbald
Copy link
Owner

@noahbald noahbald commented Apr 7, 2025

Updates the following

  • Adds roxmltree parsing
  • Updates ast to use arenas
  • Implementations now specific to parsing, using a common ast representation
  • xml namspaces now handled in parsing instead of serializing, which means a lot of blocked/broken stuff is/can be resolved
  • Uses XmlWriter for more flexible serialization

Adds the following regressions

  • PI expressions aren't serialized accurately
  • 28 more broken w3c tests (default preset)
  • 72 more broken w3c tests (none preset)
  • 209 more broken oxygen icons (default preset)
  • 224 more broken oxygen icons (none preset)

Side effects of changes

  • Some characters are no longer serialized as escape sequences

@noahbald noahbald self-assigned this Apr 7, 2025
@noahbald noahbald marked this pull request as ready for review April 13, 2025 07:33
@noahbald noahbald merged commit 44616a9 into main Apr 13, 2025
20 checks passed
@noahbald noahbald deleted the feat/arenas branch April 21, 2025 10:16
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