Fixes bug item:
#2032925 Undo buffer ASSERTs when copying lots of diffs
Currently there is a hard limit of 1024 undo records in the editor code. And as the referred bug demonstrates the limiting code is also buggy.
There is no technical reason nowadays to limit undo records. Those records of course take some memory. With referred bug and debug build it was like 8 megs. And there are over 2000 diffs in that file. We could use ten times more memory without any noticeable effect. And that is a lot of undo buffer.
In addition to the referenced bug, there is usability problem of dropping undo records from memory. It may cause unreversible data loss for user to do only a partial undo after doing copy all for diffs in file.
So this patch removes the undo limit and the only limit will be amount of memory available.
Log in to post a comment.