nsUnicodeMappingUtil singleton leaked on shutdown

VERIFIED FIXED in mozilla0.9.7

Status

()

Core
Internationalization
P3
normal
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: dbaron, Assigned: dbaron)

Tracking

({mlk})

Trunk
mozilla0.9.7
PowerPC
Mac System 9.x
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

5.88 KB, patch
Mike Pinkerton (not reading bugmail)
: review+
Simon Fraser
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

16 years ago
The nsUnicodeMappingUtils singleton is leaked on shutdown, leaking a good bunch
of other stuff (since it leaks the nsPref singleton) and thus cluttering up the
leak stats.
(Assignee)

Comment 1

16 years ago
Created attachment 58833 [details] [diff] [review]
patch
Comment on attachment 58833 [details] [diff] [review]
patch

looks good (except for the header garbage, did you use IE to post the patch?).
have you tested this on regular CFM builds, or just mach-o? r=pink.
Attachment #58833 - Flags: review+
(Assignee)

Comment 3

16 years ago
I actually tested it on a classic build.  I can't get the CFM build to build (I
think the latest problem was some sort of link error).

And yes, I used IE to post the patch because the build of Mozilla I had didn't
have working file upload controls...
(Assignee)

Updated

16 years ago
Status: NEW → ASSIGNED
Keywords: mlk
Priority: -- → P3
Target Milestone: --- → mozilla0.9.7

Comment 4

16 years ago
Comment on attachment 58833 [details] [diff] [review]
patch

sr=sfrsaer
Attachment #58833 - Flags: superreview+
(Assignee)

Comment 5

16 years ago
Fix checked in 2001-11-27 17:15 PDT.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 6

16 years ago
Verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.