GNU bug report logs - #51355
Fwd: [PATCH] Fix a memory leak in pgtk_draw_fringe_bitmap

Previous Next

Package: emacs;

Reported by: Sharaf Zaman <shzam <at> sdf.org>

Date: Sat, 23 Oct 2021 16:27:03 UTC

Severity: normal

Tags: patch

Merged with 51354

Fixed in version 29.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


Message #10 received at 51355 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Sharaf Zaman <shzam <at> sdf.org>
Cc: 51355 <at> debbugs.gnu.org, 51354 <at> debbugs.gnu.org,
 Yuuki Harano <masm+github <at> masm11.me>
Subject: Re: bug#51354: [PATCH] Fix a memory leak in pgtk_draw_fringe_bitmap
Date: Sun, 24 Oct 2021 20:09:17 +0200
Sharaf Zaman <shzam <at> sdf.org> writes:

> I discovered that there's a memory leak in the pgtk code, if the windows are
> created/destroyed/moved/shrunk/widened frequently. I've verified it with
> profiler that this indeed is the location (https://i.imgur.com/LnrgGyj.png) and
> fixed it (https://i.imgur.com/zdgnn5K.png).

Makes sense to me, so I've applied it to the pgtk branch, but perhaps
Yuuki has a comment (added to the CCs).

> PS: This is my first time sending a patch over an email, I'm not 100%
> sure if this is how it's done.

Looked perfect.  :-)

This change was small enough to apply without assigning copyright to the
FSF, but for future patches you want to submit, it might make sense to
get the paperwork started now, so that subsequent patches can be applied
speedily. Would you be willing to sign such paperwork?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




This bug report was last modified 3 years and 266 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.