Re: [vim/vim] (Try to) gracefully handle out-of-memory in bindtextdomain(). (PR #15116)

2024-07-03 Fir de Conversatie Christ van Willegen
e we don't check the return value. Getting an out of memory error here looks very unlikely... Christ van Willegen > -- -- 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.v

Re: [vim/vim] .wq! Deletes All data except the line where the cursor is located. (Issue #14892)

2024-06-01 Fir de Conversatie Christ van Willegen
Op za 1 jun. 2024 11:58: > >1. Position the cursor on any line and type the following command: > >:.wq! > > > I guess that the . indicates the range of lines to write, the current one... Christ van Willegen > -- -- You received this message from the "

Re: [vim/vim] cbuffer/cad doesn't take visual selection as range '<,'>cad (Issue #14638)

2024-04-26 Fir de Conversatie Christ van Willegen
a newer version you can try? Christ van Willegen > -- -- 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

Re: Commit: NSIS: Fix a few issues with gvim.nsi

2024-04-10 Fir de Conversatie Christ van Willegen
t; + File ${VIMTOOLS}\libsodium.dll > +!endif > > SetOutPath $0 > > Looking at the diff, I can only see that the path to libsodium has changed, but I don't see any references to libgcc_s_sjlj-1.dll nor anything that looks like a PATCHLEVEL. Was there more to include in this p

Re: [vim/vim] Add gettext() package and encoding argument (PR #12447)

2024-03-10 Fir de Conversatie Christ van Willegen
Hi, On Sat, Mar 9, 2024 at 6:07 PM Christian Brabandt wrote: > is this ready? > Yes, but I'm totally unhappy with the way you need to invoke the Makefile to get this to work, but I couldn't think of a better way :-( Christ van Willegen -- -- You received this message from the &q

Re: [vim/vim] Check for printf argument overflow (PR #13799)

2024-03-10 Fir de Conversatie Christ van Willegen
On Sat, Mar 9, 2024 at 6:06 PM Christian Brabandt wrote: > is this ready now? > Yes. Or at least, ready for review :-) Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For

Re: Commit: patch 9.1.0155: can only get getregion() from current buffer

2024-03-07 Fir de Conversatie Christ van Willegen
Typo comment: + - {pos1} and {pos2} must belong to a |bufloaded()| buffer. + - It is evaluated in current window context, this makes a + different if a buffer is displayed in a different window and + 'virtualedit' or 'list' is set *this makes a difference Christ van Willegen -- -- You

Re: Commit: patch 9.0.1948: Vim9: object variable "this." should only be used in constructor

2023-09-29 Fir de Conversatie Christ van Willegen
*argend = NUL; > + > semsg(_(e_cannot_use_an_object_variable_except_with_the_new_method_str), > arg); > + *argend = c; > + break; > + } > + > Will this check fail if the class method using "this." is called newCalculation()? Or are an

Re: [vim/vim] patch 9.0.1704: Cannot use positional arguments for printf() (0c6181f)

2023-08-31 Fir de Conversatie Christ van Willegen
%1$0*2$.*3$d %4$d *E1404* (the [flags] are specified _after_ the {n$} positional argument specifier...) Sorry about that error in the docs! Christ van Willegen -- -- You received this message from the "vim_dev" ma

Re: Commit: patch 9.0.1791: No tests for the termdebug plugin

2023-08-27 Fir de Conversatie Christ van Willegen
path('gcc') > +if g:GCC->empty() > + throw 'Skpped: gcc is not found in $PATH' > +endif > + > "Skipped" (2 x) Christ van Willegen > -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to

Re: Updates on the Vim project

2023-08-27 Fir de Conversatie Christ van Willegen
ble in other languages. This same code has been added to NeoVim (as far as I can tell), so you can reuse translations back and forth. Christ van Willegen > -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying t

Re: patch 9.0.1704: Cannot use positional arguments for printf()

2023-08-15 Fir de Conversatie Christ van Willegen
Op di 15 aug. 2023 21:27 schreef John Marriott : > On 14-Aug-2023 02:15, Christian Brabandt wrote: > > patch 9.0.1704: Cannot use positional arguments for printf() > > > > Commit: > https://github.com/vim/vim/commit/0c6181fec4c362eb9682d5af583341eb20cb1af5 > &g

Re: Farewell to Bram and dedicate upcoming Vim 9.1 to him (#12749)

2023-08-13 Fir de Conversatie Christ van Willegen
..@vim.org. A similar message is in the syntax/README, IIRC... Christ van Willegen -- -- 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

Re: patch 9.0.1691: wrong viewport restored for incsearch and smoothscroll

2023-08-12 Fir de Conversatie Christ van Willegen
I tried contacting the author, but got their email address wrong, so I'll just copy my message here. Would you accept this patch, or do you want a PR? I looked at your PR, and since things are still 'in flux', I thought I'd share this off-list, beacuse I would like to add to your PR, and that

Re: The future of the Vim project

2023-08-09 Fir de Conversatie Christ van Willegen
e community without Bram, but at them moment I think we can continue for quite some time 'in his spirit'. Christ van Willegen -- -- 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://

Re: [vim/vim] Vim's future (Discussion #12736)

2023-08-06 Fir de Conversatie Christ van Willegen
un-contraditionary changes (mostly bug-fixes) > I could also read through them to see what they are all about, since you're still on your holiday. Shall I add a LGTM comment for the PRs I've seen? Christ van Willegen -- -- You received this message from the "vim_dev" maillist. D

Re: [vim/vim] I'm sad to hear the news. (Issue #12730)

2023-08-05 Fir de Conversatie Christ van Willegen
van Willegen -- -- 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

Re: Patch 9.0.1673

2023-07-08 Fir de Conversatie Christ van Willegen
get it. https://datatracker.ietf.org/doc/rfc2325/history/ 1998-04-01 (System) RFC published So, it's a joke alright... Christ van Willegen -- -- 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

Re: test_codestyle failure

2023-06-04 Fir de Conversatie Christ van Willegen
ected 0 but got 642 > command line..script > /home/runner/work/vim/vim/src/testdir/runtest.vim[569]..function > RunTheTest[52]..Test_source_files line 40: ../list.c: missing white space > after "if"/"while"/"for": Expected 0 but got 2340 > Check l

Re: [vim/vim] Fix that setvbuf causes assertion error (PR #12467)

2023-05-30 Fir de Conversatie Christ van Willegen
Hi, Op di 30 mei 2023 08:00 schreef K.Takata : > When Vim is compiled with debug mode on MSVC, the following command causes > an assertion error: > This leaks memory, though, because of the malloc() call... Christ van Willegen > -- -- You received this message from the "

Re: [vim/vim] Untranslated strings in the append_arg_number() function of the buffer.c file (Issue #12429)

2023-05-23 Fir de Conversatie Christ van Willegen
ations, could you advise me on how to proceed with PR #12140 ? CodeCov tells me that it drops coverage a lot, but I disagree with the computer :-) Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are reply

Re: Patch 9.0.1497

2023-04-29 Fir de Conversatie Christ van Willegen
Op za 29 apr. 2023 16:50 schreef Christ van Willegen : > Hi, > > Op za 29 apr. 2023 13:10 schreef Bram Moolenaar : > >> >> Patch 9.0.1497 >> Problem:The ruler percentage can't be localized. >> Solution: Use a string that can be translated. (Emir Sari, c

Re: Patch 9.0.1497

2023-04-29 Fir de Conversatie Christ van Willegen
, perc); And specify %3d%% or %%%-3d in the format string? Ah, I think not, because you need the space _before_ the %... I did create #12140 to be able to specify the %$ format, but that doesn't handle this, either. Christ van Willegen > -- -- You received this message from the "vim_dev"

Re: [vim/vim] Format positional arguments (PR #12140)

2023-03-25 Fir de Conversatie Christ van Willegen
I think this one is ready for review. Christ van Willegen -- -- 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 b

Re: [vim/vim] *printf vs vim_*printf for translation (Issue #10577)

2023-03-02 Fir de Conversatie Christ van Willegen
Op do 2 mrt. 2023 13:08 schreef 依云 : > If I send you the output of "git diff", is that enough to test? > > Yes! > I'll be home in about 4.5 hours. That would, probably, be in the middle of the night for you... I'll send it via direct email... Christ van Willegen &

Re: [vim/vim] *printf vs vim_*printf for translation (Issue #10577)

2023-03-02 Fir de Conversatie Christ van Willegen
Hi, Op do 2 mrt. 2023 11:20 schreef 依云 : > Hi Christ van Willegen, where is your patch that I can try out? > On my computer so far :-) If I send you the output of "git diff", is that enough to test? Christ van Willegen > -- -- You received this message from the &qu

Re: [vim/vim] *printf vs vim_*printf for translation (Issue #10577)

2023-02-26 Fir de Conversatie Christ van Willegen
ight > > argument from through va_arg(). > > > > I'll try to see if I can come up with something... > > Christ van Willegen I didn't realize this was on my to do list so long ago already! Anyway, I'm (finally!) working on this, and got quite a bit working alread

Basic vim key bindings

2022-12-17 Fir de Conversatie Christ van Willegen
Hi, Totally off topic for this list, but I ran into it and _had_ to share it... I'll get back in my cave now... Christ van Willegen https://fosstodon.org/@ids1024/109515255660791632 -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply belo

Re: Health update

2022-10-27 Fir de Conversatie Christ van Willegen
Op do 27 okt. 2022 20:55 schreef Bram Moolenaar : > > I am very sorry to report that I have a medical problem. > Beterschap!! Christ van Willegen > -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are

Re: Patch 9.0.0632

2022-10-01 Fir de Conversatie Christ van Willegen
Op za 1 okt. 2022 16:34 schreef Bram Moolenaar : > > > + Calling a function in an expr option ~ > + > *expr-option-function* > + A few options, such as 'foldexpr', are an expresison that is evaluated > to get > + a value. > * expression Christ van Willegen > --

Re: Patch 8.2.5146

2022-06-21 Fir de Conversatie Christ van Willegen
Typo: Op di 21 jun. 2022 23:16 schreef Bram Moolenaar : > + // Although unlikely, it is possible that the expression > invokes a > + // substitute command (it might fail, but still). Therefore > keep > + // an array if eval results. > Array of -- -- You

Re: [vim/vim] *printf vs vim_*printf for translation (Issue #10577)

2022-06-15 Fir de Conversatie Christ van Willegen
Hi, Op wo 15 jun. 2022 12:55 schreef Bram Moolenaar : > A generic solution would be to make the "%1$s" form work with > vim_snprintf(). I don't know how complicated this is, getting the right > argument from through va_arg(). > I'll try to see if I can come up with so

Re: Patch 8.2.5089

2022-06-14 Fir de Conversatie Christ van Willegen
_alloc(rettv) == FAIL) > + return; > ... + if (rettv_list_alloc(rettv) == FAIL) > + return; > + > ... + if (rettv_list_alloc(rettv) != OK) > + return; > + > Any specific reason why the first compares use == FAIL and the third != OK ? Christ van Willeg

Re: Patch 8.2.4776

2022-04-17 Fir de Conversatie Christ van Willegen
m_dev+unsubscr...@googlegroups.com. > To view this discussion on the web visit > https://groups.google.com/d/msgid/vim_dev/20220417201719.8D9731C03BA%40moolenaar.net > . > Here's another one: //is cleared, we my bounce, but no worse than before. May Christ van Willegen &

Re: Patch 8.2.4698

2022-04-06 Fir de Conversatie Christ van Willegen
initialzed list from a function works > lines =<< trim END > vim9script > var list: list > * uninitialized Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the

Re: Patch 8.2.4317

2022-02-07 Fir de Conversatie Christ van Willegen
is no need to import sys, it's done by default. > > + > *python-environment* > + Environment variables set in Vim are not always available in Python. > This > + depends on how Vim and Python were build. Also see > built Christ van Willegen > -- -- You received this mess

Re: Patch 8.2.4047

2022-01-09 Fir de Conversatie Christ van Willegen
> { > char *name = (char *)self->buf->buf->b_fname; > @@ -5378,7 +5378,7 @@ BufferRange(BufferObject *self, PyObject *args) > BufferRepr(BufferObject *self) > { > if (self->buf == INVALID_BUFFER_VALUE) > - return PyString_FromFormat("", > s

Re: Patch 8.2.3893

2021-12-25 Fir de Conversatie Christ van Willegen
e end of the function. Sorry for the sketchy test, I don't have much experience in writing tests for Vim9... Christ van Willegen > -- -- 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 inf

Re: [vim/vim] Tweak vb.vim; change maintainership (PR #9233)

2021-11-29 Fir de Conversatie Christ van Willegen
Op ma 29 nov. 2021 18:13 schreef Tim Chase : > *@Gumnos* commented on this pull request. > -- > > In runtime/syntax/vb.vim > <https://github.com/vim/vim/pull/9233#discussion_r758570913>: > > > > > " String and Character cont

Re: Patch 8.2.3662

2021-11-24 Fir de Conversatie Christ van Willegen
rn (wp->w_p_fdm[0] != NUL && wp->w_p_fdm[2] == 'r'); > } > > // foldmethodIsSyntax() {{{2 > *** ../vim-8.2.3661/src/version.c 2021-11-24 14:59:12.573593141 + > --- src/version.c 2021-11-24 15:13:09.260714187 + > *** > *** 75

Re: Patch 8.2.3557

2021-10-22 Fir de Conversatie Christ van Willegen
Hi, Op vr 22 okt. 2021 23:18 schreef Bram Moolenaar : > > > + def DoTest() > + const Goo = G() > + assert_equal(42, Foo) > + enddef > + DoTest() > + END > Goo? Christ van Willegen > -- -- You received this message from the

Re: Patch 8.2.3520

2021-10-16 Fir de Conversatie Christ van Willegen
that the abbreviation for the option is tsrfu and the variable is called thsfu, but that's just nitpicking I guess... Christ van Willegen > -- -- 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

Re: [vim/vim] A possible divide by zero bug in misc2.c (#8767)

2021-08-17 Fir de Conversatie Christ van Willegen
;w_width is a non zero value, _and_ the call to win_col_off(curwin) returns the same value, then width can become 0. I do not have the code here to check if this is possible... Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post! T

Re: Patch 8.2.3173

2021-07-18 Fir de Conversatie Christ van Willegen
+ INIT(= N_("E1211: List required for argument %d")); > + EXTERN char e_bool_required_for_argument_nr[] > + INIT(= N_("E1211: Bool required for argument %d")); > It's not a problem that these error numbers are the same? Christ van Willegen > -- -- Y

Re: Patch 8.2.2865

2021-05-18 Fir de Conversatie Christ van Willegen
(eap, , line_arg, _to_free) == FAIL ) would make this case faster, and really skip getting the function body. Or do you _need_ to get the function body here in order to go to the correct spot in the script?? Christ van Willegen -- -- You received this message from the "vim_dev"

Re: Patch 8.2.2454

2021-02-03 Fir de Conversatie Christ van Willegen
lled with spaces > + leadcol += (colnr_T) (ptr - line) + 1; > + } > } > I guess this won't work for multi byte characters, although these won't match whitespace. So, the code probably works as is... ignore this comment ;-) Christ van Willegen > -- -- You received this m

Re: Patch 8.2.2345

2021-01-14 Fir de Conversatie Christ van Willegen
setcursor(); > + out_flush(); > + } > + } > + #endif > If the (Un)focus escape sequence happens to be sent twice, the key_name[1] will not be set to KE_IGNORE. Is that a problem? Christ van Willegen > -- -- You received this message from t

Re: Patch 8.2.2254

2020-12-31 Fir de Conversatie Christ van Willegen
* Returns: > ! * Number option: gov_number, *numval gets value. > ! * Tottle option: gov_bool, *numval gets value. > Toggle option meant here?? Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text yo

Re: Patch 8.2.2069

2020-11-29 Fir de Conversatie Christ van Willegen
Op zo 29 nov. 2020 14:21 schreef Bram Moolenaar : > > Patch 8.2.2069 > > + " Test for updating the quickfix buffer whenever the assocaited quickfix > list > + " is changed. > Associated Christ van Willegen > -- -- You received this message from the "v

Re: [vim/vim] Popup window with title is flickering on gVim.exe (#7334)

2020-11-20 Fir de Conversatie Christ van Willegen
Op vr 20 nov. 2020 11:40 schreef Naruhiko Nishino : > This PR will be fix the flickering. > + if (0 < title_len) + { Personally, I find this inverted condition hard to read. Could you change it to tl>0? Thanks! Christ van Willegen -- -- You received this

Re: Patch 8.2.1715

2020-09-21 Fir de Conversatie Christ van Willegen
mNpixmapData, menu->xpm); XtSetArg(args[n++], XmNlabelLocation, XmBOTTOM); } would have been nicer in the first place, I think... No {} needed! I assume that those are function calls! Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post!

Re: Patch 8.2.0970

2020-06-15 Fir de Conversatie Christ van Willegen
Hi, two typos fixed below... Christ van Willegen On Sat, Jun 13, 2020 at 3:48 PM Bram Moolenaar wrote: > Patch 8.2.0970 > Problem:Terminal properties are not available in Vim script. > Solution: Add the terminalprops() function. > Files: src/term.c, src/proto/t

Re: Patch 8.2.0790

2020-05-18 Fir de Conversatie Christ van Willegen
. Fix if you see fit to do so. Christ van Willegen > -- -- 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 becaus

Re: Patch 8.2.0619

2020-04-23 Fir de Conversatie Christ van Willegen
({'a': 99, 'b': 100}, d) > endfunc > > + " Cannot use += with a locked dick This is probably true, but I guess you meant 'dict' here.. Christ van Willegen -- 09 F9 11 02 9D 74 E3 5B D8 41 56 C5 63 56 88 C0 -- -- You received this message from the "vim_dev" maillist. Do

Re: Patch 8.2.0541

2020-04-11 Fir de Conversatie Christ van Willegen
> --- 99,110 > # define rb_ary_detransient rb_ary_detransient_stub > #endif > > + // On macOS pre-installed Ruby defines "SIZEOF_TIME_T" as "SIZEOF_LONG" so > it > + // conflicts with the definition in config.h then causes macro-redifned > warn

Re: Patch 8.2.0296

2020-02-22 Fir de Conversatie Christ van Willegen
Op za 22 feb. 2020 14:27 schreef Bram Moolenaar : > > > > + *v:numbersize* > *numbersize-variable* > + v:numbersize Number of bits in a Number. This is normally 64, but on > some > + systems it my be 32. > +

Re: Patch 8.2.0199

2020-02-02 Fir de Conversatie Christ van Willegen
) > " works with and without :call > assert_equal(4, g:counter) > ! assert_equal(4, g:counter) > unlet g:counter > enddef > If it's supposed to work with and without call(), why did you delete one of them? Christ van Willegen -- -- You received this messag

Re: Patch 8.1.2269

2019-11-07 Fir de Conversatie Christ van Willegen
Op do 7 nov. 2019 23:09 schreef Bram Moolenaar : > > Patch 8.1.2269 > > *** > *** 3367,3372 > --- 3360,3367 > break; > #endif > *pbuf_end++ = *str++; > + if (pbuf_end - pbuf + 1 >= LSIZE) > + bre

Re: Patch 8.1.2115

2019-10-05 Fir de Conversatie Christ van Willegen
m_strchr(p, ' ') != NULL) > + { > + len = STRLEN(p) + 3; // two quotes and a trailing NUL > + cmd = alloc(len); > Shouldn't you check for a NULL pointer here? Christ van Willegen > -- -- You received this message from the "vim_dev" mailli

Re: [vim/vim] Add support for conditional assignment using ?= (#4695)

2019-07-20 Fir de Conversatie Christ van Willegen
Hi Yegappan, On Fri, Jul 19, 2019 at 8:39 AM Yegappan Lakshmanan wrote: > On Thu, Jul 18, 2019 at 11:31 PM Christ van Willegen > wrote: > > > > Hi, > > > > Op vr 19 jul. 2019 07:56 schreef Yegappan Lakshmanan > > : > >> > >> I have por

Re: [vim/vim] Add support for conditional assignment using ?= (#4695)

2019-07-19 Fir de Conversatie Christ van Willegen
et var ?= value", conditional assignment. Patch by Dave Eggum, >2006 Dec 11. > > There are one or two places in the patch that include code or comments that implements the .= operator. Is that intentional? Christ van Willegen > -- -- You received this message from the "

Re: Patch 8.1.1714

2019-07-18 Fir de Conversatie Christ van Willegen
gt; buf_T *buf = NULL; > ! dict_T*d = NULL; > int nr; > int i; > > ! if (argvars != NULL) > { > ! // Check arguments look OK. "Check that arguments look OK" looks better IMHO Christ van Wil

Re: Patch 8.1.1713

2019-07-18 Fir de Conversatie Christ van Willegen
e > ! down. A click in the lower half wil scroll the text one line up. > However, > this is limited so that the popup does not get smaller. > I would suggest the text to read "A click in the upper half of the scrollbar will scroll the text down one line. A click in the lower half wil scroll

Re: [vim/vim] [RFC] Enhance OptionSet event to provide also the old value (#4118)

2019-06-15 Fir de Conversatie Christ van Willegen
) && (options[opt_idx].flags & P_SECURE)) > return e_secure; > > +/* Safe the global value before changeing anything. This is needed as for > > s/Safe/Save/ ? > Also, it should be "changing" Christ van Willegen > -- -- You received this

Re: Patch 8.1.1416

2019-05-29 Fir de Conversatie Christ van Willegen
"line", wp->w_winrow + 1); > + dict_add_number(dict, "col", wp->w_wincol + 1); > + dict_add_number(dict, "width", wp->w_width); > + dict_add_number(dict, "height", wp->w_height); > + } > + } > What happens if

Various small textual changes

2019-05-06 Fir de Conversatie Christ van Willegen
of the languages I 'touched', so I can't take over maintenance. Please consider including these with Vim. Christ van Willegen -- -- 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://w

Re: Patch 8.1.1276

2019-05-06 Fir de Conversatie Christ van Willegen
txt @@ -132,7 +132,7 @@ When using replace mode, the text properties stay on the same character positions, even though the characters themselves change. -When text property columns are not updated ~ +When are text property columns not updated ~ - When setting the line with |setline()| or through

Patch for VisVim

2019-04-17 Fir de Conversatie Christ van Willegen
Hi, I see that Heiko Erhardt has stopped development of this plugin, so I'm sending this patch to the list instead. It just fixes some comments, nothing more. Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below th

Re: [vim/vim] segfault in nfa_postprocess (#4042)

2019-02-27 Fir de Conversatie Christ van Willegen
Op wo 27 feb. 2019 15:03 schreef Bram Moolenaar : > Please include the Vim patch number. A hex number is not very useful > That was a specific commit to Github, I guess... Christ > -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text

Re: [vim/vim] stackoverflow in regexp_nfa.c addstate() (#3941)

2019-02-12 Fir de Conversatie Christ van Willegen
Op di 12 feb. 2019 22:15 schreef Bram Moolenaar Closed #3941 <https://github.com/vim/vim/issues/3941> via 57ee2b6 > <https://github.com/vim/vim/commit/57ee2b6e0b5b730d12ee9db00a8e2a577df9e374> > . > That commit looks totally unrelated to regular expressions...

Re: [vim/vim] Add escaping for arguments on Cygwin system(scp and rsync) (#2220)

2019-02-08 Fir de Conversatie Christ van Willegen
Op vr 8 feb. 2019 20:55 schreef bam80 It seems the email address NdrOchip@ScampbellPfamily.AbizM-NOSPAM is > invalid. > > That's because you didn't remove the capital letters (NOSPAM) from the > address. > > I did. Still no reply :( > Including the ones inside the address?

Re: Patch 8.1.0879

2019-02-08 Fir de Conversatie Christ van Willegen
*p = '/'; > + > + #if defined(FEAT_MBYTE) && defined(WIN3264) > FEAT_MBYTE was recently promoted, right? Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For mo

Re: [vim/vim] Avoided undefined behaviour / unnecessary conditional. (#3913)

2019-02-07 Fir de Conversatie Christ van Willegen
The declaration of char *cp should be left at the top of the function. Setting the value can be where you put it. Christ van Willegen Op do 7 feb. 2019 22:30 schreef Friedrich2 Found with cppcheck 1.86. Please c. f. the commit descriptions. > -- > You ca

Re: Patch 8.1.0798

2019-01-24 Fir de Conversatie Christ van Willegen
Op do 24 jan. 2019 01:39 schreef Bram Moolenaar > Christ van Willegen wrote: > > > Op wo 23 jan. 2019 21:57 schreef Bram Moolenaar > > > > > > > Patch 8.1.0798 > > > > > > + if (len > 0) > > > + to->vva

Re: Patch 8.1.0798

2019-01-23 Fir de Conversatie Christ van Willegen
v_ga.ga_len = len; > Will this never fail? Christ van Willegen > -- -- 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 th

Re: [doc][patch] Update intro.txt

2019-01-07 Fir de Conversatie Christ van Willegen
Op ma 7 jan. 2019 21:17 schreef Bram Moolenaar > Ken Takata wrote: > > > * The latest version of Elvis is 2.2. Not being developed now. > > Yeah, it appears to have faded away. > Left the building? I'll see myself out now... Christ van Willegen > -- -- You re

[patch] Several typos in Vim documentation and comments

2018-10-23 Fir de Conversatie Christ van Willegen
ple. If they keep failing, can I send the patches directly to the list? Christ van Willegen -- -- 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.

Re: [vim/vim] Pasting multiple line commands behaves differently than in the past (< vim-8.0)? (#3545)

2018-10-16 Fir de Conversatie Christ van Willegen
Op di 16 okt. 2018 14:51 schreef zdohnal : > Would anyone mind giving me a hint what can cause behavior what user > experiences? Was it possible to do this style of copy/pasting in the past > and it changed? Or is it the bug? > Sounds like bracketed paste mode interfering somehow?

Re: When started as "vim-huge --clean -? |less" the Gnome version tries (and fails) to start a GUI, then seems to hang

2018-10-10 Fir de Conversatie Christ van Willegen
initstr += 2;" as wel...) Bram, should the pattern at the top of this function be changed to note that these names for the executable are not recognized? Not sure if 'viex' or 'viexim' would make sense... Christ van Willegen -- -- You received this message from the "vim_dev" maillist

Re: Patch 8.1.0406

2018-09-18 Fir de Conversatie Christ van Willegen
t; > *** ../vim-8.1.0405/src/testdir/test_startup.vim2018-09-16 > 16:28:08.358268312 +0200 > --- src/testdir/test_startup.vim > + " Test the -V[N] argument to set the 'version' option to [N] > Verbose? Christ van Willegen -- -- You received this message from the &q

Re: Patch 8.1.0394

2018-09-17 Fir de Conversatie Christ van Willegen
bution... Christ van Willegen -- -- 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 t

Re: Problem with Vim 8.1.332 and 8.1.347 viewing .tgz files

2018-09-07 Fir de Conversatie Christ van Willegen
Hi, Perhaps you were bitten by https://github.com/vim/vim/commit/fc65cabb15d0236bce001ad78e12a40511caf941#commitcomment-30321957 I don't know if and when Bram has updated this. Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type

Re: Patch 8.1.0245

2018-08-07 Fir de Conversatie Christ van Willegen
uot; condition looks weird. It looks as if the variable u_sync_once is set to 2 unconditionally, and then tested if it is == 1. I'm probably wrong, though... Christ van Willegen > -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply be

Re: i am such an ingrate, forgive me

2018-06-23 Fir de Conversatie Christ van Willegen
Do you have expandtabs set? That would explain this, I think... Christ van Willegen -- -- 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 --- Yo

Re: [vim/vim] Fix typo in autocmd.txt (#2989)

2018-06-07 Fir de Conversatie Christ van Willegen
greeing). "don't" is > the correct form ("buffers that don't have a name", where the subject of > "don't" is "that" = "buffers"). > The patch proposes don't, so it is correct. Christ van Willegen -- -- You received this message from the

Re: Patch 8.1.0020

2018-05-23 Fir de Conversatie Christ van Willegen
Hi Hirohito Higashi, On Wed, May 23, 2018 at 12:05 PM, h_east <h.east@gmail.com> wrote: > Hi Christ, > > Oh, It's my mistake. > Attached patch fixes this. > > Thanks for reporting! You're welcome! Christ van Willegen -- -- You received this message from the "

Re: Patch 8.1.0020

2018-05-23 Fir de Conversatie Christ van Willegen
String turn time value into a String Shouldn't both these functions have "String" as their return type? Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For m

Re: [vim/vim] Cabal syntax (#2922)

2018-05-17 Fir de Conversatie Christ van Willegen
Hi, Someone will surely mention this, but you should move the old maintainer to a line with "Previous maintainer" Christ van Willegen -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For mor

Re: Patch 8.0.1735

2018-04-19 Fir de Conversatie Christ van Willegen
*/ struct buffblock { buffblock_T *b_next;/* pointer to next buffblock */ char_u b_str[FLEX_ARRAY]; /* contents */ }; This way, you only need to write the code once, and the compiler gets tested in both cases. If the compiler(s) all get flexible array compatibility, the

Re: doc nit redux

2018-03-23 Fir de Conversatie Christ van Willegen
g and/or end of strings, not out of the middle > > I need to learn to have more faith in myself > If you tested it, I'm sure you've seen the result. I only looked at the docs, wondered about the missing "r" and thought "that's because it's part of the mask and the rest of the resul

Re: doc nit

2018-03-23 Fir de Conversatie Christ van Willegen
characters in the 'mask' parameter is an 'r'... Christ van Willegen -- -- 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 mes

Re: [vim/vim] Ensure a minimum width for the completion popup menu (#2314)

2017-11-13 Fir de Conversatie Christ van Willegen
ng width and height. Christ van Willegen -- -- 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 su

Re: [vim/vim] Need to unlet ctrl to quit term (#2283)

2017-11-04 Fir de Conversatie Christ van Willegen
Op 4 nov. 2017 11:49 schreef "Christian Brabandt" <vim-dev-git...@256bit.org >: I am sorry, I can hardly understand. Please give precise steps to reproduce the issue. He means not letting go of the Ctrl key, but holding it all the time. Christ van Willegen -- -- You recei

Re: Patch 8.0.1016

2017-08-30 Fir de Conversatie Christ van Willegen
l GA Weird sig... Christ van Willegen -- -- 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 subsc

Re: [vim/vim] Add ! for ":csope find ..." command (#1769)

2017-06-14 Fir de Conversatie Christ van Willegen
Hi, This part looks weird: - qf_jump(qi, 0, 0, forceit); + + if (!forceit) + { + qf_jump(qi, 0, 0, forceit); + } Might as well write qf_jump(qi, 0, 0, false) there... Christ van Willegen -- -- You received

Re: CodeAi Fixes a Null Pointer Dereference

2017-05-11 Fir de Conversatie Christ van Willegen
a 'should never happen' case. I looked at the invocations of that function, and saw that wp is used everywhere. So it either has been checked, or was NULL way before ever reaching this point. YMMV, maybe I didn't investigate long enough... Christ van Willegen -- -- You received this message fr

Re: [vim/vim] CodeAi auto-fix of nullpointer (#1551)

2017-03-10 Fir de Conversatie Christ van Willegen
e, wp === curwin (by the if above it), therefore it can't be == NULL... Regards, Christ van Willegen -- -- 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/maill

Re: Patch 8.0.0382

2017-02-27 Fir de Conversatie Christ van Willegen
*/ ! c = *ptr Now you miss that assignment into c. Christ van Willegen -- -- 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 receive

Re: [patch] :make doesn't support encoding conversion

2017-01-03 Fir de Conversatie Christ van Willegen
On Tue, Jan 3, 2017 at 9:50 AM, Ken Takata <ktakata65...@gmail.com> wrote: > Hi, > > 2017/1/3 Tue 17:38:15 UTC+9 Christ van Willegen wrote: >> Here as well (is it my viewer?? Ah! Mixed tabs and spaces! It isn't my >> viewer after all! Are mixed spaces/tabs allowed in

Re: [patch] :make doesn't support encoding conversion

2017-01-03 Fir de Conversatie Christ van Willegen
> +#ifdef FEAT_MBYTE > +if ((eap->cmdidx != CMD_make) && (eap->cmdidx != CMD_lmake)) > +{ > +/* Use 'genc' for grep commands if it is set. */ > +if (*curbuf->b_p_genc != NUL) > +enc = curbuf->b_p_genc; > +else if (*p_genc != NU

  1   2   >