Hirohito Higashi wrote:

> Hi Bram, Ken and list!
> 
> 2016-5-7(Sat) 6:23:37 UTC+9 Bram Moolenaar:
> > Ken Takata wrote:
> > 
> > > 2016/5/6 Fri 2:14:57 UTC+9 Bram Moolenaar wrote:
> > > > Ken Takata wrote:
> > > > 
> > > > > 2016/5/6 Fri 0:19:06 UTC+9 Bram Moolenaar wrote:
> > > > > > Patch 7.4.1818
> > > > > > Problem:    Help completion adds @en to all matches except the 
> > > > > > first one.
> > > > > > Solution:   Remove "break", go over all items.
> > > > > > Files:      src/ex_getln.c
> > > > > 
> > > > > This patch has a problem when help files with multiple languages are 
> > > > > installed.
> > > > > E.g. When @ja and @en are installed and 'helplang' is set to ja,
> > > > > 
> > > > >   :h screenrow()<C-D>
> > > > >   screenrow() screenrow()
> > > > > 
> > > > > But this should be:
> > > > > 
> > > > >   :h screenrow()<C-D>
> > > > >   screenrow() screenrow()@en
> > > > > 
> > > > > Or, when 'helplang' is not ja, this should be:
> > > > > 
> > > > >   :h screenrow()<C-D>
> > > > >   screenrow()@ja screenrow()@en
> > > > > 
> > > > > @en should not be removed, when there is a same help item for another
> > > > > language.
> > > > 
> > > > I guess the problem is that it depends on the order, the check for a
> > > > duplicate help tag in another language can happen after a language was
> > > > already removed.
> > > 
> > > Do you have any ideas to fix this?
> > > I don't have it, but the following behavior is totally useless:
> > > 
> > >   :h screenrow()<C-D>
> > >   screenrow() screenrow()
> > 
> > From the top of my head: First do the part that loops over all matches,
> > do the other removing later.
> 
> I wrote a patch with a test.
> Please check an attached patch.

Thanks!

-- 
The chat program is in public domain.  This is not the GNU public license.
If it breaks then you get to keep both pieces.
                -- Copyright notice for the chat program

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

-- 
-- 
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 vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Raspunde prin e-mail lui