Hi,

On Thu, Apr 26, 2018 at 10:59 AM, Linwei <[email protected]> wrote:
> I don't really familiar the quickfix.c, but in the patch:
>
> +    /* Shorten all the file names, so that it is easy to read */
> +    shorten_fnames(TRUE);
>
> I noticed that will it shorten_fnames is invoked in void qf_list(exarg_T
> *eap) . does it have to calculate every file names listed in the buffer
> after every :caddexpr ? a complexity of O(N*N) ?
>

Yes. This is the performance penalty I was referring to in my previous e-mail.

>
> Is this a little inefficient ?? I supposed that each single :caddexpr will
> only solve the file name needs to be listed.
>

I will send out a patch later today that does this.

- Yegappan

> But like I said, I don't familiar with quickfix.c, may misunderstand the
> patch.
>

-- 
-- 
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.

Raspunde prin e-mail lui