Re: memleak in FScreenParseGeometryWithScreen

2021-11-25 Thread Thomas Adam
On Fri, Nov 26, 2021 at 12:45:36AM +0100, Dominik Vogt wrote: > On Thu, Nov 25, 2021 at 11:04:25PM +, Thomas Adam wrote: > > On Wed, Nov 24, 2021 at 03:13:34PM +0100, Dominik Vogt wrote: > > > From FScreen.c:FScreenParseGeometryWithScreen(): > > > > > > I can't fix this because I don't

Re: Transforming program names

2021-11-25 Thread Jaimos Skriletz
On Thu, Nov 25, 2021 at 12:50 PM Dominik Vogt wrote: > > Recently there was the question wether the "transform" logic aon > man pages was still necessary. It seems not. > > --program-transform="s/fvwm/xyz/;s/Fvwm/Xyz/" > > Renames all man pages too: > > fvwm*.1 -> xyz*.1 > Fvwm*.1 ->

Re: UPDATE_FVWM_SCREEN macro?

2021-11-25 Thread Thomas Adam
On Wed, Nov 24, 2021 at 03:43:47PM +0100, Dominik Vogt wrote: > Does anybody know why this is a macro and not a function? > (screen.h) Because when I wrote it, it wasn't as complex as it is now. See the ta/update-fvwm-screen branch, I've converted it to a function instead. Kindly, Thomas

Re: memleak in FScreenParseGeometryWithScreen

2021-11-25 Thread Thomas Adam
On Wed, Nov 24, 2021 at 03:13:34PM +0100, Dominik Vogt wrote: > From FScreen.c:FScreenParseGeometryWithScreen(): > > I can't fix this because I don't understand what the code does. I will fix this. Kindly, Thomas

Re: memleak in FScreenParseGeometryWithScreen

2021-11-25 Thread Dominik Vogt
On Thu, Nov 25, 2021 at 11:04:25PM +, Thomas Adam wrote: > On Wed, Nov 24, 2021 at 03:13:34PM +0100, Dominik Vogt wrote: > > From FScreen.c:FScreenParseGeometryWithScreen(): > > > > I can't fix this because I don't understand what the code does. > > I will fix this. Too late, but please do

Re: UPDATE_FVWM_SCREEN macro?

2021-11-25 Thread Dominik Vogt
On Thu, Nov 25, 2021 at 10:56:19PM +, Thomas Adam wrote: > On Wed, Nov 24, 2021 at 03:43:47PM +0100, Dominik Vogt wrote: > > Does anybody know why this is a macro and not a function? > > (screen.h) > > Because when I wrote it, it wasn't as complex as it is now. > > See the

Window title leak

2021-11-25 Thread Dominik Vogt
This is a tough one: ==14237== 11 bytes in 1 blocks are definitely lost in loss record 11 of 102 ==14237==at 0x483877F: malloc (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so) ==14237==by 0x5FF7E4A: strdup (strdup.c:42) ==14237==by 0x14B75A:

Transforming program names

2021-11-25 Thread Dominik Vogt
Recently there was the question wether the "transform" logic aon man pages was still necessary. It seems not. --program-transform="s/fvwm/xyz/;s/Fvwm/Xyz/" Renames all man pages too: fvwm*.1 -> xyz*.1 Fvwm*.1 -> Xyz*.1 (The module names are not transformed, but that's a different