Closed
Bug 8152
Opened 25 years ago
Closed 25 years ago
MLK fix in nsStringBundle
Categories
(Core :: Internationalization, defect, P3)
Tracking
()
VERIFIED
FIXED
M7
People
(Reporter: David.R.Gardiner, Assigned: tao)
Details
(Whiteboard: Has patch and will check in today (6/15))
Attachments
(1 file)
1.23 KB,
patch
|
Details | Diff | Splinter Review |
intl/strres/src/nsStringBundle.cpp The debug text that was printf'd wasn't being free'd properly. Enhancement I also added code that looked for the URL without locale, if the URL+locale could not be found. I'm not sure if this is wanted or not, but otherwise the StringBundleTest doesn't do anything, as it can't find the properties file. I guess another step would be to look for the three files.. eg. file_en_AU.properties file_en.properties file.properties -dave
Reporter | ||
Comment 1•25 years ago
|
||
Comment 2•25 years ago
|
||
I decide to seperate the locale fallback part into a seperate bug report - 8188. Now this bug report is only for the MLK. Please refer the locale fallback part to 8188 so we won't confused our dicussion and easy for IQA to verify. Thanks. Tao, please fix the MLK part. Thanks
Assignee: ftang → tao
Updated•25 years ago
|
Summary: MLK fix and enhancement to nsStringBundle → MLK fix in nsStringBundle
Comment 3•25 years ago
|
||
Change summary from "MLK fix and enhancement to nsStringBundle" to "MLK in nsStringBundle"
Updated•25 years ago
|
Summary: MLK fix in nsStringBundle → MLK fix and enhancement to nsStringBundle
Target Milestone: M7
Comment 4•25 years ago
|
||
Mark the MLK part M7.
Updated•25 years ago
|
Summary: MLK fix and enhancement to nsStringBundle → MLK fix in nsStringBundle
Whiteboard: Has patch and will check in today → Has patch and will check in today (6/15)
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Updated•25 years ago
|
QA Contact: teruko → tao
Reporter | ||
Comment 6•25 years ago
|
||
I ran Purify 6 for NT on StringBundleText and it no longer reports a memory leak in nsStringBundle. Is that the verification you require?
You need to log in
before you can comment on or make changes to this bug.
Description
•