2016-02-25 16:30 GMT+03:00 h_east <[email protected]>: > Hi mattn! > > 2016-2-25(Thu) 21:50:37 UTC+9 mattn: >> On Thursday, February 25, 2016 at 9:07:46 PM UTC+9, h_east wrote: >> > Hi, >> > >> > I think Nikolai's patch is simple and better. Looks good for me. >> > This is perfect when adding a test :-) >> > >> > mattn> >> > Detailed error will be output in get_tv_string_buf_chk(). >> >> No. When the option is passed neither a nor r, it should be error, I think. > > I see. > Is attached patch OK?
As I said earlier if you want to filter out all incorrect strings this should also have `&& act[1] == NUL` condition. Also consider the case when user used `setqflist([], '')`, it will display “E475: Invalid argument: ”. Now guess what argument it meant, this error message looks more like a bug then like a reference to `''`. > > -- > Best regards, > Hirohito Higashi (a.k.a. h_east) > > -- > -- > 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. -- -- 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.
