On 13:07 Wed 17 Dec , Bram Moolenaar wrote: > > Marcin Szamotulski wrote: > > > On 20:11 Sat 13 Dec , Bram Moolenaar wrote: > > > > > > Marcin Szamotulski wrote: > > > > > > > I attach the promised patch for checking if the range is invalid. > > > > > > Thanks! > > > > Updated version, I figured out that 0 range is needed for :tabe. > > Thanks. I tried including it, but it breaks tests. > Also, it would be useful if you can write tests for the boundary checks.
I see, actually tests fail because one cannot do :888tabnew, which test62 used, and the tests I wrote were testing that going over the range will still work for various other commands. I will update the tests. One more question: there is ZEROR flag for the range which some commands are using. Do you want that the `invalid_range` function tests if the range is >= 0 or >= 1 depending on the ZEROR flag? For example, that would make a difference for the range of :argadd (which can be 0) and :argu which does not accept 0. Best regards, Marcin -- -- 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.
signature.asc
Description: Digital signature
