Closed Bug 3865 Opened 26 years ago Closed 26 years ago

Assertion in nsLocaleFactory

Categories

(MailNews Core :: Internationalization, defect, P3)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: nhottanscp, Assigned: nhottanscp)

Details

Missing ref count initialization in the nsLocale constructor. I will check this in after M3. Index: nsLocale.cpp =================================================================== RCS file: /cvsroot/mozilla/intl/locale/src/nsLocale.cpp,v retrieving revision 1.1 diff -c -r1.1 nsLocale.cpp *** nsLocale.cpp 1999/01/28 23:27:46 1.1 --- nsLocale.cpp 1999/03/17 00:11:43 *************** *** 42,47 **** --- 42,48 ---- PRUint32 i; nsString* key, *value; + NS_INIT_REFCNT(); fHashtable = PL_NewHashTable(LOCALE_HASH_SIZE,&nsLocale::Hash_HashFunction, &nsLocale::Hash_CompareNSString,&nsLocale::Hash_CompareNSString,NULL,NULL);
Target Milestone: M4
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Checked in the fix. Currently nobody is using nsLocale other than self test program. No verification is possible.
No verification will be attempted for M4.
Status: RESOLVED → VERIFIED
This has been in the product long enough and for thread pane headers, sorting seems to be working as intended. I'm going to mark this verified as fixed based on that.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.