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