Bram, I still like this change. I especially like that it makes it obvious when a region of text was already sorted (this can be useful sometimes).
I noticed, though, that I made an omission here: The u_save() function is still called at the top. This means that, even though &mod may not be set, the undo record is still created in all cases. Example; 1. echo -e 'a\nb\nc' >foo 2. vim --clean foo 3. :sort 4. :undolist See that there is a change to undo when no change happened and &mod is not set. This is, obviously, low priority. I just wanted to not it here in case the fix was simple enough. I tried a few things to remove the most recent undo record when no sorting happens, but with no success. Not a critical problem, but I always want my patches to be perfect. Thanks, Jason -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
