Re: Patch 9.0.1084

2022-12-20 Fir de Conversatie Christopher Plewright
> The attached patch tries to fix both warnings. Thanks. > It seems that the function peek_input_record_buffer() is not used anywhere. > So the patch removes it. Yeah, it’s not used now. The existing static s_irCache is inner function scoped, so I couldn’t directly inject records to it for t

Patch 9.0.1085

2022-12-20 Fir de Conversatie Bram Moolenaar
Patch 9.0.1085 (after 9.0.1084) Problem:Compiler warns for uninitialized variable. Solution: Initialize the variable. Remove unused function. (John Marriott) Files: src/os_win32.c *** ../vim-9.0.1084/src/os_win32.c 2022-12-20 20:01:09.624090908 + --- src/os_win32.c 202

Re: Patch 9.0.1084

2022-12-20 Fir de Conversatie Bram Moolenaar
John Marriott wrote: > On 21-Dec-2022 07:02, Bram Moolenaar wrote: > > Patch 9.0.1084 > > Problem:Code handling low level MS-Windows events cannot be tested. > > Solution: Add test_mswin_event() and tests using it. (Christopher > > Plewright, > > closes #11622) > > Files:

Re: Patch 9.0.1084

2022-12-20 Fir de Conversatie John Marriott
On 21-Dec-2022 07:02, Bram Moolenaar wrote: Patch 9.0.1084 Problem:Code handling low level MS-Windows events cannot be tested. Solution: Add test_mswin_event() and tests using it. (Christopher Plewright, closes #11622) Files: runtime/doc/builtin.txt, runtime/doc/testing.t

Patch 9.0.1084

2022-12-20 Fir de Conversatie Bram Moolenaar
Patch 9.0.1084 Problem:Code handling low level MS-Windows events cannot be tested. Solution: Add test_mswin_event() and tests using it. (Christopher Plewright, closes #11622) Files: runtime/doc/builtin.txt, runtime/doc/testing.txt, runtime/doc/usr_41.txt, src/ev

Re: [vim/vim] Remove 'r' and 'o' from formatoptions. (PR #11700)

2022-12-20 Fir de Conversatie Gary Johnson
On 2022-12-20, madjxatw wrote: > This behavior is really annoying! Whether to insert a comment leader should be > decided by user but not assumed by the editor. Most of the time when placing a > short trailing comment, I don't want a comment continuation. At least o should > be removed. >From my v

Patch 9.0.1083

2022-12-20 Fir de Conversatie Bram Moolenaar
Patch 9.0.1083 Problem:Empty and comment lines in a class cause an error. Solution: Skip empty and comment lines. (closes #11734) Files: src/vim9class.c, src/testdir/test_vim9_class.vim *** ../vim-9.0.1082/src/vim9class.c 2022-12-19 12:18:06.404218717 + --- src/vim9class.c

Re: Choices for Vim9 class implementation

2022-12-20 Fir de Conversatie puremo...@gmail.com
For what it's worth, I feel that the formal "var memberVariable : Type" syntax for member is (while more verbose) more likely to stand out. The `var` keyword is a signpost and so is likely to be syntax highlighted. Simple `memberVariable : Type` doesn't stand out anywhere and contains mostly id