8152 - MLK fix in nsStringBundle

Status: VERIFIED FIXED

(Core :: Internationalization, defect, P3)

Description

[David Gardiner]

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

Comment 1

[David Gardiner]

Attachment: Fix debug leak + non-locale URL search

Comment 2

[Frank Tang]

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

Comment 3

[Frank Tang]

Change summary from "MLK fix and enhancement to nsStringBundle" to "MLK in
nsStringBundle"

Comment 4

[Frank Tang]

Mark the MLK part M7.

Comment 5

[tao]

Hi, David:

Would you please verify this bug for us?

Thanks

Comment 6

[David Gardiner]

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

Is that the verification you require?

Comment 7

[tao]

Yes, that is what I was asking. Thanks for the help.


-Tao