Closed Bug 1728104 Opened 2 years ago Closed 2 years ago

Consider moving NumberPart definitions to separate header


(Core :: Internationalization, enhancement, P3)




94 Branch
Tracking Status
firefox94 --- fixed


(Reporter: dminor, Assigned: allstars.chh)



(Whiteboard: [i18n-unification])


(4 files)

At the moment, we're defining NumberPart in NumberFormatField.h, but NumberPart is part of a public API, and NumberFormatField is not. It might make sense to split these up.

Suggested by Anba in the review of

Whiteboard: [i18n-unification]

Yoshi, this is pretty trivial, but maybe you can fix it. It breaks doing local SpiderMonkey non-unified builds. This uses the NumberPart without a definition.

If you set this line to = 1 locally and do a spidermonkey build you will trigger the problem. Also, the ListFormat.cpp files you add need a ScopeICUObject.h include.

Flags: needinfo?(allstars.chh)
Assignee: nobody → allstars.chh
Flags: needinfo?(allstars.chh)
Attachment #9241334 - Attachment description: Bug 1728104. Part 2: Add NumberPart.h. → Bug 1728104 - Part 2: Add NumberPart.h.
Pushed by
Part 1: Fix non-unified build error on ListFormat.cpp. r=tcampbell
Part 2: Add NumberPart.h. r=platform-i18n-reviewers,anba,dminor
Part 3: Remove NumberFormatFields.h from exported headers. r=platform-i18n-reviewers,anba,dminor
Part 4: Merge NumberFormatFieldsUtil.[cpp|h] into NumberFormatFields.[cpp|h]. r=platform-i18n-reviewers,anba,dminor
You need to log in before you can comment on or make changes to this bug.