Closed Bug 8152 Opened 25 years ago Closed 25 years ago

MLK fix in nsStringBundle

Categories

(Core :: Internationalization, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: David.R.Gardiner, Assigned: tao)

Details

(Whiteboard: Has patch and will check in today (6/15))

Attachments

(1 file)

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
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
Summary: MLK fix and enhancement to nsStringBundle → MLK fix in nsStringBundle
Change summary from "MLK fix and enhancement to nsStringBundle" to "MLK in
nsStringBundle"
Summary: MLK fix in nsStringBundle → MLK fix and enhancement to nsStringBundle
Target Milestone: M7
Mark the MLK part M7.
Summary: MLK fix and enhancement to nsStringBundle → MLK fix in nsStringBundle
Whiteboard: Has patch and will check in today
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
QA Contact: teruko → tao
Status: RESOLVED → VERIFIED
QA Contact: tao → david.gardiner
Hi, David:

Would you please verify this bug for us?

Thanks
I ran Purify 6 for NT on StringBundleText and it no longer reports a memory leak
in nsStringBundle.

Is that the verification you require?
Yes, that is what I was asking. Thanks for the help.


-Tao
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: