Re: [dwm] Fwd: [hackers] separated layout-specific stuff...

On Sat, Aug 11, 2007 at 06:20:02PM +0200, Anselm R. Garbe wrote:
> On Sat, Aug 11, 2007 at 03:11:11PM +0200, markus schnalke wrote:
> > Jeroen Schot <schot-dwm_AT_a-eskwadraat.nl> wrote:
> > > On Sat, Aug 11, 2007 at 07:52:46AM -0500, Kurt H Maier wrote:
> > > > Two #include lines make a file "ugly"? How so?
> > >
> > > It's not about the number of lines, it's about the concept. When I add a
> > > layout (say 'gridmode') it will look:
> > >
> > > #include "tile.h"
> > > #include "float.h"
> > > #include "gridmode.h"
> > > #define LAYOUTS \
> > > static Layout layout[] = { \
> > > /* symbol function */ \
> > > { "[]=", tile }, /* first entry is default */ \
> > > { "><>", floating }, \
> > > { "[#]", gridmode }, \
> > > };
> > >
> > > Looks pretty redundant/useless to me, which I find ugly :)
> >
> > I think that looks clear ... but there is redundance, yes.
>
> Yes there is redundance, but I think it is not worse than
> before. Besides this, I can't think of a sane way how to build
> up the layout list otherwise... So I think let's live with it.
>
> Regards,
> --
> Anselm R. Garbe >< http://www.suckless.org/ >< GPG key: 0D73F361
>
Personally, I find the redundancy a small price to pay for the newfound
ease of adding new layouts.

-- 
Jeremy O'Brien aka neutral_insomniac GPG key: 0xB1140FDB
http://pohl.ececs.uc.edu/~jeremy/jeremy.asc Linux ambelina 2.6.22.1 ppc
7447A, altivec supported PowerBook5,8 GNU/Linux