Re: [dev][dmenu] loadfont() segmentation fault patch

2011-07-22 Thread Connor Lane Smith
Hey, On 22 July 2011 14:14, lolilolicon wrote: > I for one like the 4.3.1 code better. And it worked. > > Patch attached inline. I do it the humble way. I agree, actually. Pushed to tip. Thanks, cls

[dev][dmenu] loadfont() segmentation fault patch

2011-07-22 Thread lolilolicon
To reproduce the segmentation fault: dmenu -fn "-windows-montecarlo-medium-r-normal--11-110-72-72-c-60-microsoft-cp1252" <<< hello The culprit is loadfont(). This font[1] triggers XLoadQueryFont(), and loadfont() fails in the for loop. At least that's what my quick debug indicated. I did a diff

[dev] dwm 5.9 small patch for non xinerama users

2011-07-22 Thread mauro tonon
Hi. I noted that if i disable Xinerama, i don't use also the following functions: dirtomon, focusmon, tagmon. So, i think it is possible to hold all these functions between "#ifdef XINERAMA ... #endif". The difference in the final binary file size is very little but... A possible patch is attached