> > Why not
> > 
> >     for (iter = 0; iter < nitems(name_to_kbfunc); iter++) {
> >         func = &name_to_kbfunc[iter]; /* if you want to use func */
> > 
> > then you don't need the last NULL entry in the array or add code
> > to not break existing code.
> 
> See: http://c-faq.com/decl/extarraysize.html
> 
> The diff moves the declaration of name_to_kbfunc[] to calmwm.h and
> externs it for use in kbfunc.c, but it's still defined in conf.c.
>

Yeah, I see that now, though the use of nitems is used throughout
the code already and deviating from that and making that particular
array a "special case" seems kind of messy and somewhat confusing.

Reply via email to