[EMAIL PROTECTED] (Lars Gullik Bjønnes) writes:

| Allan Rae <[EMAIL PROTECTED]> writes:
>
| | On Wed, 5 Jun 2002, Allan Rae wrote:
>>
>>> It's not quite as simple as errors after a newline not being removed.
>>> Some are, but those in the overly long testcase aren't.  I will
>>> investigate this weekend.
>>
| | I've figured it out.  This change:
>>
| | 2002-03-13  Lars Gullik Bjønnes  <[EMAIL PROTECTED]>
>>
| |     * paragraph.C: change several algorithm to be more appripriate for
| |     the problem domain. This is lookip in FontList and in the InsetList.
>
| Hmm, ... bad boy...
>
| | modified Paragraph::InsetIterator(pos) so that it no longer returned
| | an inset_iterator with (it.pos >= pos) but now only returns one if
| | (it.pos == pos).  The net result being that removeAutoInsets() can't
| | iterate through the inset list deleting auto-insets as it goes by
| | simply calling InsetIterator(pos) -- unless the insets are
| | consecutive like they very often are.

Are you sure that the autodelinsets cannot just use the inset_iterator
directly (perhaps even the one in buffer.h).

...also ... are you sure that a change in InsetIterator to just use >=
will not fix it?

-- 
        Lgb


Reply via email to