setqftitle: documentation/reference; maybe include the patch in a contrib dir?

[blueyed]

There is the following code in plugin/ag.vim:

if exists('*setqftitle')
  call setqftitle('ag' . args)
endif

I wondered where it might come from, and then found a mailing list thread about it:
https://groups.google.com/d/msg/eclim-dev/_FGmSoZ24ak/Bl1Jc1eFI1QJ

It still applies (except for the syntax hunk).

Is this still the latest patch?
Have you proposed it for inclusion in Vim?

btw, there's a warning when building:

eval.c: In function ‘f_getqftitle’:
eval.c:11784:26: warning: assignment makes pointer from integer without a cast [enabled by default]
     rettv->vval.v_string = get_errortitle(wp);
                          ^

[ervandew]

I never got around to fixing the warning, but there was another patch posted to vim-dev that, after some changes, looks to be a better alternative to mine:

https://groups.google.com/d/msg/vim_dev/X7VVPd4Do5s/svKaV6Eah6YJ

It is in vim's ever growing todo list:

Patch to allow setting w:quickfix_title via setqflist() and setloclist()
functions. (Christian Brabandt, 2013 May 8, update May 21)

I need to update ag and eclim to both use the new patch's method for when it is hopefully merged into vim.

[blueyed]

Ah.. I have came across that thread a while ago.

But I noticed that given your patch, ag causes an error:

Error detected while processing function <SNR>29_Ag:                                                                                                                      
line   58:                                                                                                                                                                
E730: using List as a String                                                                                                                                              
E116: Invalid arguments for function setqftitle

[blueyed]

It has landed in Neovim finally: neovim/neovim#4337 (not yet in Vim)