Closed
Bug 906154
Opened 11 years ago
Closed 11 years ago
move gCategoryImp to its point of use to avoid a static constructor
Categories
(Core :: Internationalization, defect)
Core
Internationalization
Tracking
()
RESOLVED
FIXED
mozilla27
People
(Reporter: froydnj, Unassigned)
Details
Attachments
(1 file)
1.19 KB,
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Comment 1•11 years ago
|
||
No reason that we need to initialize this until someone asks for it.
Reporter | ||
Updated•11 years ago
|
Attachment #791438 -
Flags: review?(smontagu)
(I'd have considered using the module constructor and destructor rather than function scope statics (which I think aren't great either, though not sure what the compiler generated code for them looks like these days). But maybe this is an improvement anyway.)
Comment 3•11 years ago
|
||
Is this interface even used any more? I think it's now only some vestigial remnants after having its functionality replaced by mozilla::unicode
Flags: needinfo?(jfkthame)
Reporter | ||
Comment 4•11 years ago
|
||
Ping to smontagu and jfkthame....
Comment 5•11 years ago
|
||
Huh, sorry - I thought I'd responded to this, but apparently that was only in my head. :( AFAIK, this is no longer used anywhere in Gecko or Firefox, as we use the COM-free mozilla::unicode functions directly. However, it's possible that there is external code (e.g. addons, or maybe front-end code in other apps such as Thunderbird) that relies on it, so I'd be hesitant to remove it completely without further investigation.
Flags: needinfo?(jfkthame)
Comment 6•11 years ago
|
||
Comment on attachment 791438 [details] [diff] [review] move gCategoryImp to its point of use to avoid a static constructor Review of attachment 791438 [details] [diff] [review]: ----------------------------------------------------------------- So if we aren't going to remove the interface this is presumably a worthwhile improvement
Attachment #791438 -
Flags: review?(smontagu) → review+
Reporter | ||
Comment 7•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d6357352d724
Flags: in-testsuite-
Comment 8•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d6357352d724
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in
before you can comment on or make changes to this bug.
Description
•