Re: [dwm] stacked cpt patch updated!

pancake,

Thanks for the updated cpt patch; I only had to change the default CPTH
value.

There are a few things that could be made clearer / changed.

1. Your example configuration of '{ MODKEY|ShiftMask, XK_q,
clientspertag, "0" },' conflicts with the default binding for quit.

2. I understand that, for me to successfully use nmaster.c, I must
include it before "Layout layouts[]", as stated in your documentation.
However, what was not made apparent was that #RESIZEHINTS must be
specified before you include nmaster.c. Therefore, I recommend you
change the following:

You should modify your config.h to include "nmaster.c" from it after
setting the NMASTER, NCOLS and NROWS macro defintions.

to this (or something similar):

You should modify your config.h to include "nmaster.c" AFTER setting
the NMASTER, NCOLS, NROWS, BORDERPX, and RESIZEHINTS macro definitions.

This would make it a lot easier to compile dwm without trial and error,
especially for a person like me who is not very familiar with C.

Apart from that, thanks for the patch: it works well.

Sincerely,

Antony Jepson.

On Nov 24 23:31, pancake wrote:
> bugs fixed, only needs code cleanup ;)
>
> have fun trying it, and let me know what do you think about this feature.
>
> On Sat, 24 Nov 2007 22:16:04 +0100
> pancake <pancake_AT_youterm.com> wrote:
>
> > NOTE that it's full of bugs i have to go now, but i'll fix it asap ;)
> >
> > On Sat, 24 Nov 2007 21:56:54 +0100
> > pancake <pancake_AT_youterm.com> wrote:
> >
> > > Now the cpt patch of nmaster-4.7.c support stacking, i think this is
> > > better way to handle this. let me know what do you think :)
> > >
> > > http://news.nopcode.org/nmaster-4.7.c
> > >
> > > http://news.nopcode.org/nmaster.gif
> > >
> > > --pancake
> > >
> >
> >
> > --pancake
> >
>
>
> --pancake