Hi Bram and list,
2015/2/18(Wed) 17:30:31 UTC+9 h_east:
> Hi Bram,
>
> 2015/2/18(Wed) 1:26:32 UTC+9 Bram Moolenaar:
> > Hirohito Higashi wrote:
> >
> > > 2015/2/9(Mon) 2:21:55 UTC+9 h_east:
> > > > Hi Bram and Nice Vim developers,
> > > >
> > > > 2015/1/6(Tue) 4:58:27 UTC+9 Bram Moolenaar:
> > > > > Yasuhiro Matsumoto wrote:
> > > > >
> > > > > > bug.vim
> > > > > > -------------------
> > > > > > set nocompatible
> > > > > >
> > > > > > inoremap <F5> <C-R>=ListMonths()<CR>
> > > > > >
> > > > > > func! ListMonths()
> > > > > > call complete(col('.'), ['January', 'February', 'March',
> > > > > > \ 'April', 'May', 'June', 'July', 'August', 'September',
> > > > > > \ 'October', 'November', 'December'])
> > > > > > return ''
> > > > > > endfunc
> > > > > > -------------------
> > > > > >
> > > > > > 1. vim -u bug.vim -N
> > > > > >
> > > > > > 2. press F5
> > > > > >
> > > > > > 3. type Ctrl-L while popup menu is visible
> > > > > >
> > > > > > Ctrl-L should not behave like Ctrl-P. ins_compl_key2dir() handles
> > > > > > suffix keys after CTRL-X. And also it handles keys while popup menu
> > > > > > is
> > > > > > visible (ex: CTRL-P/CTRL-N). This should be cleanly separated.
> > > > > >
> > > > > > https://gist.github.com/mattn/2c83d5a17c9e9505e43d
> > > > >
> > > > > Thanks!
> > > >
> > > > I update a patch. (Attached to this email)
> > > > Please check.
> > >
> > > Oops. My previous patch had a problem. so I attached a new patch.
> > > Please check.
> >
> > Thanks for the update.
> >
> > > > And ...
> > > > I found specification inconsistency between 'complete()' and
> > > > 'backspace' option.
> > > >
> > > > How to reproduce:
> > > > 1. cat test.vim
> > > > inoremap <F5> <C-R>=ListMonths()<CR>
> > > >
> > > > func! ListMonths()
> > > > call complete(1, ['One', 'Three'])
> > > > return ''
> > > > endfunc
> > > >
> > > > 2. vim -N -u test.vim
> > > >
> > > > 3. Enter insert-mode and input 'ab' and leave insert-mode.
> > > > iab<Esc>
> > > >
> > > > 4. Reenter insert-mode and use 'complete()'
> > > > a<F5>
> > > >
> > > > 5. Delete two characters.
> > > > <C-H><C-H>
> > > >
> > > > What happen.
> > > > - '4.' Edit start columns 3, but 'complete()' starts 1 (by 1st
> > > > argument).
> > > > - '5.' Can't delete column 1 and 2
> > > > 'backspace' option not contain "start", So vim can not deleted this.
> > > >
> > > > I think this is specification inconsistency.
> > > > How Let's fix?
> > > > (a) Occurred an error when 'complete()' first argument is can not be
> > > > deleted by under the influence of the 'backspace' option.
> > > >
> > > > (b) Adjust the first argument value implicitly in f_complete(). (In
> > > > this case, Ajust 3)
> > > >
> > > > (c) Between 'complete()' is functioning to ignore the 'backspace'
> > > > option.
> > > >
> > > > I would write the patch when you are determined.
> > >
> > > Bram, Please decide a specification.
> >
> > The 'backspace' option should probably be ignored in this situation,
> > since the code says to change those characters. 'backspace' is really
> > to make Vim backwards compatible with Vi, for those users who prefer it
> > that way. I don't think it should apply to Insert mode completion.
>
> Thanks for the reply.
> I was convinced to your opinion. But thinking has changed to one that is by
> analyzing the source code in order to write a patch.
> Because, The completion mode has been finished(*1) before function ins_bs()
> is called When I first press the <C-H> key at reproduce step '5.'.
>
> *1: ins_compl_active() returns 0 and ctrl_x_mode is 0.
>
> Therefore, it is impossible to determination of whether a complementing in
> ins_bs().
>
> So, the specification I'd be in (a) or (b).
> What do you think?
Postscript:
My analysis results and opinion for the Vim that apply my patch
(builtin_func_complete_fix2.patch).
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.
