[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#56462: 29.0.50; [PATCH] Memory leak in ns_draw_relief
From: |
Daniel Martín |
Subject: |
bug#56462: 29.0.50; [PATCH] Memory leak in ns_draw_relief |
Date: |
Sat, 09 Jul 2022 16:13:39 +0200 |
I ran the Leaks tool with Emacs 29, and I've found a memory leak in the
NS version of Emacs.
Since commit c7b48b61d08f0b6a08584080badc60fe62ba1db1, in function
ns_draw_relief, static local variables baseCol and lightCol are assigned
to nil separately to their declaration. That has the subtle consequence
that the further down calls to [baseCol release] and [lightCol release]
become a no-op, so each time ns_draw_relief is called it leaks some
instances of NSColor.
The fix is to revert to the previous way those static variables were
declared.
I've attached a patch with the correction. With this fix, the leaks
tool doesn't report the NSColor leaks anymore when I play around with
Emacs 29.
Thanks.
0001-Fix-memory-leak-in-ns_draw_relief.patch
Description: Text Data
- bug#56462: 29.0.50; [PATCH] Memory leak in ns_draw_relief,
Daniel Martín <=