Open Bug 1719751 Opened 4 months ago Updated 2 months ago

Unify CommonFunctions in SpiderMonkey

Categories

(Core :: Internationalization, task, P3)

task

Tracking

()

People

(Reporter: gregtatum, Unassigned)

References

(Depends on 1 open bug, Blocks 2 open bugs)

Details

(Whiteboard: [i18n-unification], [i18n-unification-help-wanted] )

https://searchfox.org/mozilla-central/source/js/src/builtin/intl/CommonFunctions.cpp
https://searchfox.org/mozilla-central/source/js/src/builtin/intl/CommonFunctions.h

There is a single call to ufmtval_getString in CommonFunctions.cpp.

The header file imports #include "unicode/utypes.h", but that could even be a forward declaration now.

The CallICU function should be removed eventually, once we are done unifying.

Greg to set up other bugs blocking this.

Depends on: 1719747
Flags: needinfo?(gtatum)
Whiteboard: [i18n-unification-help-wanted]
Depends on: 1719693
Flags: needinfo?(gtatum)

This bug can be used track the removal of CallICU.

It seems reasonable to move the following check to ICU4CGlue at the same time:

https://searchfox.org/mozilla-central/source/js/src/builtin/intl/CommonFunctions.h#109-112

Depends on: 1719462, 1719735
Whiteboard: [i18n-unification-help-wanted] → [i18n-unification], [i18n-unification-help-wanted]
You need to log in before you can comment on or make changes to this bug.