Closed
Bug 159583
Opened 23 years ago
Closed 23 years ago
simple string bundle optimization
Categories
(Core :: Internationalization, defect, P3)
Core
Internationalization
Tracking
()
RESOLVED
FIXED
mozilla1.2alpha
People
(Reporter: darin.moz, Assigned: darin.moz)
Details
(Keywords: intl)
Attachments
(1 file)
|
1.01 KB,
patch
|
tetsuroy
:
review+
alecf
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
a while back, dbaron pointed out that the string bundle code spends a lot of
time inside the nsCAutoString constructor. so, i took a closer look at the code
and noticed that there are at least two nsCAutoString instances that can be
eliminated. i'm not sure how much of performance win this is, if at all, but
anyways, here's the patch.
|
Assignee | |
Comment 1•23 years ago
|
||
|
|
Assignee | |
Updated•23 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla1.2alpha
code issue, QA -> yokoyama@netscape.com.
Keywords: intl
QA Contact: ruixu → yokoyama
|
||
Comment 3•23 years ago
|
||
Comment on attachment 92936 [details] [diff] [review]
v1 patch
nice. sr=alecf
Attachment #92936 -
Flags: superreview+
|
||
Comment 4•23 years ago
|
||
Comment on attachment 92936 [details] [diff] [review]
v1 patch
/r=yokoyama
thanks
Attachment #92936 -
Flags: review+
|
||
Comment 5•23 years ago
|
||
Comment on attachment 92936 [details] [diff] [review]
v1 patch
a=asa (on behalf of drivers) for checkin to 1.1
Attachment #92936 -
Flags: approval+
|
|
Assignee | |
Comment 6•23 years ago
|
||
fixed-on-trunk
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•