Skip to content

Remove unused XPath functions min, max, ReplaceAll, FontSizeGuess - #906

Open
moritz-gross wants to merge 1 commit into
daisy:mainfrom
moritz-gross:remove-unused-xpath-functions
Open

moritz-gross wants to merge 1 commit into
daisy:mainfrom
moritz-gross:remove-unused-xpath-functions

Conversation

@moritz-gross

Copy link
Copy Markdown
Collaborator

None of these were called from any rule file or from Rust code via XPath. They were registered under a "Not used: remove??" comment.

  • Delete Min, Max and ReplaceAll and their registrations
  • Drop the XPath registration for FontSizeGuess; its only remaining use was em_from_value() from canonicalize.rs, which is now a free pub(crate) function

None of these were called from any rule file or from Rust code via XPath.
They were registered under a "Not used: remove??" comment.
  - Delete Min, Max and ReplaceAll and their registrations
  - Drop the XPath registration for FontSizeGuess; its only remaining use was em_from_value() from canonicalize.rs, which is now a free pub(crate) function
@github-actions

Copy link
Copy Markdown
Linux library size: 0.59 MiB (-0.08%)
Revision Release liblibmathcat.so
Base (3aee7a4) 0.59 MiB (614,768 bytes)
PR (283081e) 0.59 MiB (614,296 bytes)
Change -472 bytes (-0.08%)

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

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

1 participant