RE: [AD] File selector patch |
[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]
> What would you like to read instead ?
>
> if (height == OLD_FILESEL_HEIGHT)
> #ifdef HAVE_DIR_LIST
> height = 161;
> #else
> height = 189;
> #endif
>
> is better ?
Nope. It's just the nasty looking constants scattered through the code,
and their "magic" switching to other nasty looking constants. It startled
me first time I saw it :)
--
Vincent Penquerc'h
Mail converted by MHonArc 2.6.19+ | http://listengine.tuxfamily.org/ |