On Mon, 11 Sep 2006 at 6:20pm, Bram Moolenaar wrote:

>
> Hari Krishna Dara wrote:
>
> > On Sun, 10 Sep 2006 at 1:56pm, Bram Moolenaar wrote:
> >
> > >
> > > Patch 7.0.096
> > > Problem:    taglist() returns the filename relative to the tags file,
while
> > >       the directory of the tags file is unknown. (Hari Krishna Dara)
> > > Solution:   Expand the file name. (Yegappan Lakshmanan)
> > > Files:        src/tag.c
> >
> > I applied the patch and tested that it works in my scenario, thank you.
> > However, I see that the name is expanded only when the tags file is not
> > in the current directory, which means the results can have a mixture of
> > both absolute and relative names (if they are coming from different tag
> > files in different directories). this could create some weird issues
> > with looking up tags in one window and opening the file in another
> > window and if they have differe lcd. This also means, the results can't
> > be cached and used later on, as the current directory could change. Is
> > this intentional?
> >
> > One observation I made is that (on windows) the expanded path doesn't
> > include the drive letter unless the current directory and the directory
> > of the tags files are on different drives. I don't know if this is how
> > Vim normally behaves, but this means, they are not really absolute
> > paths, but relative to the current drive.
> >
> > I could always explicitly do a fnamemodify() on each of the files, so it
> > is not a big deal, but I am wondering how efficient fnamemodify() is, if
> > it involves simple string manipulation or OS calls.
>
> Normally relative file names work just fine.  If you need a full path
> you can use fnamemodify() in a map() to change it.

Yes, that is what I am doing. Is fnamemodify() an expensive operation
involving OS calls?

-- 
Thanks,
Hari

>
> > There was also a request from me some time back to allow a limit
> > argument to the taglist() function such that its response can be
> > improved in scenarios involving large number of matches. Is there a plan
> > to provide a 7.0 patch for this feature?
>
> Not soon.
>
>

__________________________________________________
Do You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com 

Reply via email to