Re: [dwm] New layout

On Wed, Jul 02, 2008 at 11:35:38AM +0200, yy wrote:
> 2008/7/2, Anselm R. Garbe <arg_AT_suckmore.org>:
> > Looking into your pull request I less and less come to the conclusion
> > to add an extra void *aux field to Client for extensions like
> > this (or the previous extensions proposed in the toggle{vert,horiz} pull requestes.
> >
> > With this in 5.1 you could easily keep your pull requestset upwards
> > compatible if you use Client->aux to store all the extensions to
> > Client.
> >
> > What do you think?
> >
> > Then you could separate all your functions into dwmii.c which is
> > included in config.h instead.
> >
> > Kind regards,
> > Anselm
> >
>
> I like the idea, I think it would be well worth to give it a try.

Well, I'm not totally sure, since *aux won't work without adding
at most a void (*cleanupaux)(void *); callback as well. So the
layout algorithm would need to check

        if(!c->aux)
                c->aux = malloc(Foobar);

before doing anything. However I had another look into the
pull request, and it hooks into some other core functions of dwm as
well. Is there any way to get rid of this interference or to
discuss further hook possibilities?

Kind regards,

-- 
 Anselm R. Garbe >< http://www.suckmore.org/ >< GPG key: 0D73F361