Re: [dev] [dwm] drw_scm_free() and drw_clr_free() in drw.h

From: Hiltjo Posthuma <hiltjo_AT_codemadness.org>
Date: Mon, 29 Sep 2025 18:50:19 +0200

On Sun, Sep 28, 2025 at 06:52:41PM +0000, Listeria monocytogenes wrote:
> On Sat Sep 27, 2025 at 10:16 AM UTC, Hiltjo Posthuma wrote:
> > Hi,
> >
> > Thanks for the feedback.
> >
> > I've pushed patches to dwm and dmenu.
>
> Hi,
>
> Thanks for the quick reply.
>
> Given that drw_scm_create() is the one making the allocation, I would
> consider having drw_scm_free() call free(3), as I don't see a case for
> keeping the allocated buffer.
>
> > Please test them.
>
> Will do.
>

Yeah that makes sense. I've pushed patches to dmenu, dwm and libsl again to
change this.

Thanks,

-- 
Kind regards,
Hiltjo
Received on Mon Sep 29 2025 - 18:50:19 CEST

This archive was generated by hypermail 2.3.0 : Mon Sep 29 2025 - 19:00:10 CEST