2017-02-15 6:51 GMT+09:00 Kazunobu Kuriyama <kazunobu.kuriy...@gmail.com>: <snip>
> I'll see to them all and soon send you a patch which also addresses the >> issue raised by Dominique. >> > Hi Bram Here is the patch. Although this is almost the same as that I sent to Dominique, it has been changed a little to fix a typo and improve style. The differences are: (1) Correction to a typo in the message argument of assert_false() in Test_set_guifontwide(). (2) Replacement of the emptiness checks using binary operators with empty(), except for environment variables. (3) Replacement of the numerical incrementation using an assignment and an arithmetic addition with the single operator +=. (4) Skimming the scripts in testdir, I realized that the scope prefix 'l:' is moderately used there. Judging from the names of the local variables used in test_gui.vim, I now think those 'l:'s are rather redundant and hence have eliminated them from the variables. Regards Kazunobu -- -- 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 vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
fix_Test_set_guifontset-update02.patch
Description: Binary data