Oops, I forgot to send a copy of the email below to vim_dev. Since the patch I'm proposing now contains some modifications to src/testdir/Make_dos.mak and src/testdir/Make_ming.mak, I'd be more than happy if someone would give a glance at it and tell me if it works or not for Windows.
Best regards, Kazunobu Kuriyama ---------- Forwarded message ---------- From: Kazunobu Kuriyama <[email protected]> Date: 2017-02-19 20:41 GMT+09:00 Subject: Re: [patch] Tests on 'guiheadroom' and an GUI test enhancement To: Bram Moolenaar <[email protected]> 2017-02-19 7:21 GMT+09:00 Bram Moolenaar <[email protected]>: > > Kazunobu Kuriyama wrote: > > > In addition to the tests mentioned in the title, this patch includes some > > proposal enhancement on the GUI tests. > > > > As written in gui-init in gui.text, some options and variables, such as > > guiheadroom and menu, have to be handled before/at GUI startup in order > for > > them to take effect at runtime. > > > > Currently, the test scripts are read together with '-U NONE'. > > > > So, if a gvimrc is allowed to use for such options and variables, the GUI > > tests are expected to be improved. > > > > The attached patch proposes a way to accomplish that. Basically, the GUI > > tests are divided into two parts, one for the existing tests and another > > for new tests (currently, only a single test there, actually) that need > > gvimrc setup. Each part needs to run its own vim instance, thus two test > > scripts. > > > > Any comments or suggestions are welcome :) > > Thanks. I'll await comments before including this. > Updated. Please find a patch attached to this email. > Coes the init test only work for Unix? > Oh my... To address this issue, the ':set' command in gui_init.vim is now guarded with an IF statement so that it won't be executed for GUIs which don't support 'guiheadroom', and new two different target-rule's are added to Make_dos.mak and Make_ming.mak so that the new GUI tests won't be executed through a wrong rule. In addition, the make variable NO_PLUGIN is renamed FORTIFIED, and a new variable WEAKLY_FORTIFIED is introduced (Please see their definitions at the place where NO_PLUGIN had been defined). Owing to them, some redundant variables of lengthy names which were introduced in the previous patch have been removed. In the rule of the '.vim.res' target of Makefile, '-U NONE' appears twice; one from FORTIFIED, or NO_PLUGIN, and another from the rule itself. The latter is removed to reduce redundancy. If you find anything wrong, please let me know. Best regards, Kazunobu Kuriyama -- -- 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.
gui_tests-update01.patch
Description: Binary data
