Closed
Bug 205981
Opened 22 years ago
Closed 22 years ago
GetSystemFontInfo leaks a PangoFontDescription object
Categories
(Core Graveyard :: GFX: Gtk, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dougt, Assigned: dougt)
References
Details
Attachments
(1 file, 1 obsolete file)
|
1.40 KB,
patch
|
blizzard
:
review+
asa
:
approval1.4+
|
Details | Diff | Splinter Review |
Despite pango's documemtation, you must call pango_font_description_free to free
the descriptor created by pango_font_description_from_string.
|
Assignee | |
Comment 1•22 years ago
|
||
|
|
Assignee | |
Comment 2•22 years ago
|
||
Attachment #123530 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 3•22 years ago
|
||
I still see a pango_x_font_map_for_display clean. This the fontmap really
reference counted?a
|
|
Assignee | |
Updated•22 years ago
|
Attachment #123534 -
Flags: review?(blizzard)
|
||
Updated•22 years ago
|
OS: Windows XP → Linux
|
|
Assignee | |
Updated•22 years ago
|
Attachment #123534 -
Flags: review?(blizzard) → review?(blizzard)
|
||
Comment 4•22 years ago
|
||
Comment on attachment 123534 [details] [diff] [review]
another leak fix included for no extra charge.
r+sr=blizzard
How bogus. Thanks for catching these.
Attachment #123534 -
Flags: review?(blizzard) → review+
|
|
Assignee | |
Comment 5•22 years ago
|
||
Comment on attachment 123534 [details] [diff] [review]
another leak fix included for no extra charge.
lets get this a'ed for 1.4.
Attachment #123534 -
Flags: approval1.4?
|
|
Assignee | |
Comment 6•22 years ago
|
||
Checking in nsDeviceContextGTK.cpp;
/cvsroot/mozilla/gfx/src/gtk/nsDeviceContextGTK.cpp,v <-- nsDeviceContextGTK.cpp
new revision: 1.116; previous revision: 1.115
done
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
|
||
Updated•22 years ago
|
Attachment #123534 -
Flags: approval1.4? → approval1.4+
|
||
Updated•17 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•