Michael Gerz <[EMAIL PROTECTED]> writes:

|                   && oldpar.isLineSeparator(old.pos())
|                   && oldpar.isLineSeparator(old.pos() - 1)
|                   && oldpar.lookupChange(old.pos() - 1).type != 
Change::DELETED) {
| -                     // We need to set the text to Change::INSERTED to
| -                     // get it erased properly
| -                     oldpar.setChangeType(old.pos() -1, Change::INSERTED);
| -                     oldpar.erase(old.pos() - 1);
| +                     oldpar.erase(old.pos() - 1, false); // do not track 
change

I think I hate the comment. :-)

Perhaps we should use a enum to make the code in itself clearer.

oldpar.erase(old.pos() - 1, Tracking::Off);

or something.

-- 
        Lgb

Reply via email to