Hi Arnaud! On Fr, 31 Jul 2015, Arnaud Decara wrote:
> Hello Christian! > > On Friday, 31 July 2015 16:03:21 UTC+2, Christian Brabandt wrote: > > Hi Arnaud! > > > > On Fr, 31 Jul 2015, Arnaud Decara wrote: > > > > > Please find attached a fix. > > > (Both issues with the change operator and 's' should > > > be fixed; undiscovered similar issues that might have > > > occured should be fixed too.) > > > > I believe that breaks test_listlbr.out > > > > Here is a slightly different approach, including a test. The idea is, to > > only recalculate the oap->start_vcol and oap->end_vcol columns, if the > > cursor is on a wrapped line, else that shouldn't be necessary. However, > > I am not completely sure, this is the best solution. > > > Maybe I'm not using correctly the tests but test_librl does not seem > to be broken here. (When I run all the tests via the Makefile, i get > "Test results : ALL DONE", and when I remove test_listlbr.out and > rerun the test separately I don't get any error) > Am I doing something wrong? That should be okay. I am just wondering, because for me that test failed (which was my first idea when I saw that problem). Perhaps this was a problem just for me, so in that case your patch would be preferable for its simplicity. The test should be included nevertheless ;) Best, Christian -- Mathematiker-Glücksrad: "Ich kaufe ein Epsilon." -- -- 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.
