Lech Lorens wrote (more than a year ago):

> On 28-Mar-2009 Lech Lorens <lech.lor...@gmail.com> wrote:
> > Improved version of patch is attached. I took into consideration most of
> > the suggestions. I will be grateful for any more comments.
> 
> I found an invalid memory access while executing the :tabs command. The
> attached patch fixes the problem.

I finally took some time to look at this patch.  Unfortunately
documentation is missing, I think it must be elsewhere.  Would be a lot
simpler to have it as one patch.

Adding two commands to set the title is a bit too specific.  Why not put
the values in window-local variables?  Then these can be set from any
script that generates a quickfix or location list.  The ":make" and
":grep" commands can also do this.  Advantage is that the values are
available to a custom status line as well.  But perhaps there is a
disadvantage as well?

There is quite a bit of repeated code.  E.g., buf_spname() is often
called to fill a buffer.  Can easily make a separate function for this.

-- 
The question is:  What do you do with your life?
The wrong answer is: Become the richest guy in the graveyard.
                                (billionaire and Oracle founder Larry Ellison)

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.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

Raspunde prin e-mail lui