Hi Markus, On 31 October 2016 at 13:18, Markus Teich <[email protected]> wrote: > Anselm R Garbe wrote: >> I prefer requiring SchemeLast = N. > > sorry I forgot to update that one last week. Do you prefer SchemeLast as enum > element (The whole enum definition has to move to config.def.h) or as > `#define`? > Either way the declaration of the `scheme` array has to be moved below the > config.h include which breaks the current structure of `dwm.c`. I really > prefer > the ecalloc version which at least does not fail during runtime on OOM, > because > it is only called in `setup()`.
I wrapped my head around this and agree with your original approach. Let's drop SchemeLast in favour for heap allocation. This change is less dramatic than moving the enum to config.h or introducing an ugly define. -Anselm
