Re: Strange test in xdisp.c

[Stefan]

> XFASTINT and a test for <= 0 seems a bit odd to me.

>   /* If bottom moved off end of frame, change mode line percentage.  */
>   if (XFASTINT (w->window_end_pos) <= 0
>       && Z != IT_CHARPOS (it))
>     w-> update_mode_line = Qt;

Hmm... indeed odd.  I'd guess that the "== 0" part of "<= 0" is the one used
and the "< 0" part is basically never activated.  I'd hope that if
window_end_pos had ever been something else than an integer, ENABLE_CHECKING
would have cought it (I always run with it).


        Stefan