bug#7307: 24.0.50; Mode line had more than just dashes removed

[Stephen Berman]

On Mon, 01 Nov 2010 21:32:12 +0200 Eli Zaretskii <eliz@gnu.org> wrote:

>> From: Stephen Berman <stephen.berman@gmx.net>
>> Date: Mon, 01 Nov 2010 19:11:49 +0100
>> Cc: 7307@debbugs.gnu.org
>> 
>> +  mode_line_filler = !NILP (Fwindow_system (Qnil)) ? lots_of_spaces 
>> +    : lots_of_dashes;
>
> Actually, you want to use `FRAME_WINDOW_P (f)' here.  It does exactly
> what you mean, but with much smaller overhead (for starters, it
> doesn't call Lisp).

Thanks. (I copied Fwindow_system, with the accompanying "This is wrong"
comment, from keyboard.c; I wonder why `FRAME_WINDOW_P (f)' isn't used
there.)

>> -        || field_width > sizeof (lots_of_dashes))
>> +        || field_width > sizeof (mode_line_filler))
>
> mode_line_filler is a pointer, so the result of sizeof here is not
> what you want.

Oops, thanks; I should be more careful with copy 'n' paste.

Thanks for the helpful comments.  I'll wait for Stefan's reply before
posting a corrected (or reworked) patch.

Steve Berman