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

[Stephen Berman]

On Fri, 05 Nov 2010 19:45:54 +0100 Andreas Schwab <schwab@linux-m68k.org> wrote:

> Stephen Berman <stephen.berman@gmx.net> writes:
>
>> --- 19342,19362 ----
>>       case '-':
>>         {
>>      register int i;
>> !    int fill_len;
>> !    int lots_of_filler = 140 * sizeof (char);
>
> What is the point of sizeof (char)?

Oops, I was just confused. Thanks.

>
>> !    /* Make a string of mode_line_filler characters.  */
>> !    fill_len = 
>> !      (mode_line_target == MODE_LINE_NOPROP
>> !       || mode_line_target == MODE_LINE_STRING) 
>> !      ? 2
>> !      : ((field_width <= 0 || field_width > lots_of_filler)
>> !         ? FRAME_MESSAGE_BUF_SIZE (f) - 1 
>> !         : lots_of_filler);
>> !    for (i = 0; i < fill_len; ++i)
>> !      decode_mode_spec_buf[i] = mode_line_filler;
>
> What happens if mode_line_filler is a non-ASCII character?

The mode line gets filled with a string of raw bytes :-(.  What is the
right thing to do here?

Steve Berman