Skip to content

Simplify BaseNode::base_node - #907

Merged
NSoiffer merged 1 commit into
daisy:mainfrom
moritz-gross:base-node-use-is-modified
Oct 24, 2026
Merged

NSoiffer merged 1 commit into
daisy:mainfrom
moritz-gross:base-node-use-is-modified

Conversation

@moritz-gross

Copy link
Copy Markdown
Collaborator
  • Use IsNode::is_modified instead of a hard-coded copy of the MATHML_MODIFIED_NODES list
  • Replace tail recursion with a loop
  • Fix the doc comment, which was copied from an unrelated function

  - Use IsNode::is_modified instead of a hard-coded copy of the MATHML_MODIFIED_NODES list
  - Replace tail recursion with a loop
  - Fix the doc comment, which was copied from an unrelated function
@github-actions

Copy link
Copy Markdown
Linux library size: 0.59 MiB (-0.01%)
Revision Release liblibmathcat.so
Base (3aee7a4) 0.59 MiB (614,768 bytes)
PR (dd4216d) 0.59 MiB (614,728 bytes)
Change -40 bytes (-0.01%)

Built with default features, Rust 1.96.0, and Ubuntu 24.04. Workflow run.

@NSoiffer
NSoiffer merged commit 5966a07 into daisy:main Oct 24, 2026
14 checks passed
@github-project-automation github-project-automation Bot moved this from Triage to Done in MathCAT Project Board Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants