Ignore further requests to undoMergeAllEdits if we are already doing so - otherwise, the new undo safe point we add overrides the original one, meaning it takes an extra call to undo to get back there - which defeats the object of undoMergeAllEdits.

Review Request #109257 - Created March 3, 2013 and submitted

Information
Simon St James
kate
Reviewers
kate
Ignore further requests to undoMergeAllEdits if we are already doing so - otherwise, the new undo safe point we add overrides the original one, meaning it takes an extra call to undo to get back there - which defeats the object of undoMergeAllEdits.
No automated tests break; will be indirectly tested via the tests added in 37925997dcad0e7e4dfef4a2063f7ca569f12522. I don't think anything actually uses setUndoMergeAllEdits except for Vim, but I'm posting it for review as it changes KateDocument.
Simon St James
Christoph Cullmann
Commit Hook
Simon St James
Review request changed

Status: Closed (submitted)

Loading...