On Wed, Oct 22, 2003 at 09:02:49AM +0200, Andre Poenitz spake thusly:

...

> > +           if (i + 1 < last)   <-- needed to prevent trailing empty row
> >                                     creation after wide inset
> > +                   break;          <-- OK.
> > 
> > This works perfectly on the file José sent me.
> 
> Why this restriction to  i + 1 < last?
> 
> The original code had an unconditional 'break'.

Yes. And as a result, it sometimes happened that a line break was
placed after an inset sticking out of the right margin even if it
(that inset) was the last 'character' of its paragraph... resulting in
an empty trailing row on-screen appearing to contain a character, but
actually not containing any belonging to the paragraph. And thus the
'silly' warning.

You will find this same test also higher up in the code.
 
> As John said: Be _very_ careful in this area.

You too ;-)

> Andre'
 
- Martin 

Attachment: pgp00000.pgp
Description: PGP signature

Reply via email to