If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Refactor the MathML Operator Dictionary

NEW
Unassigned

Status

()

Core
MathML
8 months ago
8 months ago

People

(Reporter: fredw, Unassigned, Mentored)

Tracking

(Blocks: 2 bugs, {helpwanted})

Trunk
helpwanted
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox54 affected)

Details

(Whiteboard: [good second bug])

(Reporter)

Description

8 months ago
layout/mathml/nsMathMLOperators.cpp could be rewritten a bit:

* Use a hardcoded static table instead of using XPCOM to parse mathfont.properties into memory.
* Use a sorted table instead of a hash table. See mfbt/BinarySearch.h for a helper search function.
* Minimize the size of an entry (OperatorData). For example, use bitfields for the rspace/space, flags or form.

See https://trac.webkit.org/browser/trunk/Source/WebCore/mathml/MathMLOperatorDictionary.cpp#L35 for the implementation in WebKit.
You need to log in before you can comment on or make changes to this bug.