combine i18n libraries

RESOLVED FIXED in mozilla1.2beta



17 years ago
17 years ago


(Reporter: alecf, Assigned: alecf)



Windows 2000

Firefox Tracking Flags

(Not tracked)


(Whiteboard: fix in hand)


(1 attachment)



17 years ago
In a current build, we have a bunch of small i18n libraries:

20      ucharuti.dll
46      chardet.dll
39      nslocale.dll
15      strres.dll
12      lwbrk.dll
132     total

These could be combined into one dll, as they are related in function and are
are critical to general operation of gecko... combining them will save 8-12k per
DLL as well as reduce runtime DLL overhead of about 30k per DLL. (saving 32-48k
on disk and 120k at runtime)

Comment 1

17 years ago
ok, for example on my release build, I combined chardet.dll and lwbrk.dll which
were 56k and 28k respectively. the combined dll is 60k, saving 24k. (now
obviously the nightlies are a bit smaller here, and will probably save slightly

Comment 2

17 years ago
continuing status: adding unicharutil (28k in my build) increased the DLL by
12k, another 16k saved.

Comment 3

17 years ago
ok, a final update: I combined all the DLLs together and in my release build,
the combined size went down from 188 to 108k:

28      ucharuti.dll
48      nslocale.dll
28      lwbrk.dll
28      strres.dll
56      chardet.dll
188     total

108     i18n.dll

so that's a savings of ~80k on my build, probably slightly less on a nightly.
this is not to mention 4 fewer dlls. patch forthcoming


17 years ago
Blocks: 163737
Priority: -- → P2
Target Milestone: --- → mozilla1.2beta

Comment 4

17 years ago
code issue, QA to for now.
Keywords: intl
QA Contact: ruixu → yokoyama

Comment 6

17 years ago
now I just need some reviews... anyone listening? :)
Whiteboard: fix in hand

Comment 7

17 years ago
cc some i18n people

roy, naoki, or brian - can I get a review?

i18n.xml is a new mac project which just has all the C++ files that are in each
of the current libraries.
Comment on attachment 99744 [details] [diff] [review]
combine 5 dlls into one

Attachment #99744 - Flags: superreview+

Comment 9

17 years ago
Frank, I believe you know the most about the converters so would you care to
review this?

Comment 10

17 years ago
*** Bug 49527 has been marked as a duplicate of this bug. ***

Comment 11

17 years ago
this isn't the converter library (uconv) - these are all the other i18n libraries. 

Comment 12

17 years ago
+  deleteThisFile("Components", "gfx2.dll");
+  deleteThisFile("Components", "gfx2.xpt");
is here? beside that , I think all other are fine. r=ftang

Comment 13

17 years ago
Comment on attachment 99744 [details] [diff] [review]
combine 5 dlls into one

Attachment #99744 - Flags: review+

Comment 14

17 years ago
Comment on attachment 99744 [details] [diff] [review]
combine 5 dlls into one

wait, how can this work ?
Do you need to change the module file ?  I see you take out the old module
file, but don't you need to supply new module file ?
Attachment #99744 - Flags: review+

Comment 15

17 years ago
the files are already in the tree, as I indicated above. 
already checked in though, before I saw the above comments.
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 16

17 years ago
Comment on attachment 99744 [details] [diff] [review]
combine 5 dlls into one

r=ftang , please take care the gfx2 stuff I mentioned eariler
Attachment #99744 - Flags: review+
You need to log in before you can comment on or make changes to this bug.