Bug 153984 - The Operator Dictionary lacks entries for U+1EEF0 and U+1EEF1
Summary: The Operator Dictionary lacks entries for U+1EEF0 and U+1EEF1
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: MathML (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Frédéric Wang (:fredw)
URL: https://lists.w3.org/Archives/Public/...
Keywords:
: 122296 (view as bug list)
Depends on: 160462
Blocks: 161306
  Show dependency treegraph
 
Reported: 2016-02-08 07:34 PST by Frédéric Wang (:fredw)
Modified: 2016-08-28 23:57 PDT (History)
7 users (show)

See Also:


Attachments
WIP Patch (16.53 KB, patch)
2016-08-23 06:41 PDT, Frédéric Wang (:fredw)
no flags Details | Formatted Diff | Diff
Patch WIP (33.00 KB, patch)
2016-08-24 08:06 PDT, Frédéric Wang (:fredw)
no flags Details | Formatted Diff | Diff
Patch (39.29 KB, patch)
2016-08-25 05:54 PDT, Frédéric Wang (:fredw)
darin: review+
Details | Formatted Diff | Diff
Patch (42.13 KB, patch)
2016-08-28 23:08 PDT, Frédéric Wang (:fredw)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Frédéric Wang (:fredw) 2016-02-08 07:34:52 PST
These are mentioned on http://www.unicode.org/charts/PDF/U1EE00.pdf referred by the MathML 3 recommendation, but absent from their suggested operator dictionary. I don't think any math font supports them yet.

Stretching operators
The following operators stretch based on the width of the text that is displayed below or above them.
1EEF0 ARABIC MATHEMATICAL OPERATOR MEEM WITH HAH WITH TATWEEL
  • used in Arabic mathematics to denote summation
  • stretched at the tatweel
  → 2211 ∑  n-ary summation
1EEF1 ARABIC MATHEMATICAL OPERATOR HAH WITH DAL
  • used in Persian mathematics to denote limits
  • stretched between the hah and the dal
Comment 1 Frédéric Wang (:fredw) 2016-08-23 06:41:54 PDT
Created attachment 286710 [details]
WIP Patch
Comment 2 Frédéric Wang (:fredw) 2016-08-23 06:44:37 PDT
*** Bug 122296 has been marked as a duplicate of this bug. ***
Comment 3 Frédéric Wang (:fredw) 2016-08-24 08:06:56 PDT
Created attachment 286853 [details]
Patch WIP

Adding a test and fixing a coding error with explicit prefix. I'll add a test for horizontal stretchiness tomorrow.
Comment 4 Frédéric Wang (:fredw) 2016-08-25 05:54:35 PDT
Created attachment 286963 [details]
Patch
Comment 5 Darin Adler 2016-08-27 17:17:16 PDT
Comment on attachment 286963 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=286963&action=review

> Source/WebCore/mathml/MathMLOperatorDictionary.cpp:1109
> +// The following two non-BMP characters are handled separately so that the arrays 'dictionary' and
> +// 'horizontalOperators' can use UChar instead of UChar32 and avoid an increase of memory footprint.

I don’t think this is the right call. The additional memory footprint is about 2kB. We should keep the code simple instead.
Comment 6 Frédéric Wang (:fredw) 2016-08-28 01:08:34 PDT
(In reply to comment #5)
> I don’t think this is the right call. The additional memory footprint is
> about 2kB. We should keep the code simple instead.

OK, I'll rewrite a simpler patch then.
Comment 7 Frédéric Wang (:fredw) 2016-08-28 23:08:30 PDT
Created attachment 287248 [details]
Patch
Comment 8 Frédéric Wang (:fredw) 2016-08-28 23:57:54 PDT
Committed r205111: <http://trac.webkit.org/changeset/205111>