To: vim_dev@googlegroups.com

Subject: Patch 7.3.197

Fcc: outbox

From: Bram Moolenaar <Bram@moolenaar.net>

Mime-Version: 1.0

Content-Type: text/plain; charset=UTF-8

Content-Transfer-Encoding: 8bit

------------

Patch 7.3.197

Problem:    When a QuickfixCmdPost event removes all errors, Vim still tries

	    to jump to the first error, resulting in E42.

Solution:   Get the number of error after the autocmd event. (Mike Lundy)

Files:	    src/quickfix.c

*** ../mercurial/vim73/src/quickfix.c	2011-05-10 16:41:13.000000000 +0200

--- src/quickfix.c	2011-05-19 17:34:11.000000000 +0200

***************

*** 2813,2829 ****

  					   (eap->cmdidx != CMD_grepadd

  					    && eap->cmdidx != CMD_lgrepadd),

  					   *eap->cmdlinep);

  #ifdef FEAT_AUTOCMD

      if (au_name != NULL)

  	apply_autocmds(EVENT_QUICKFIXCMDPOST, au_name,

  					       curbuf->b_fname, TRUE, curbuf);

  #endif

      if (res > 0 && !eap->forceit)

-     {

- 	if (wp != NULL)

- 	    qi = GET_LOC_LIST(wp);

  	qf_jump(qi, 0, 0, FALSE);		/* display first error */

-     }

      mch_remove(fname);

      vim_free(fname);

--- 2813,2833 ----

  					   (eap->cmdidx != CMD_grepadd

  					    && eap->cmdidx != CMD_lgrepadd),

  					   *eap->cmdlinep);

+     if (wp != NULL)

+ 	qi = GET_LOC_LIST(wp);

  #ifdef FEAT_AUTOCMD

      if (au_name != NULL)

+     {

  	apply_autocmds(EVENT_QUICKFIXCMDPOST, au_name,

  					       curbuf->b_fname, TRUE, curbuf);

+ 	if (qi->qf_curlist < qi->qf_listcount)

+ 	    res = qi->qf_lists[qi->qf_curlist].qf_count;

+ 	else

+ 	    res = 0;

+     }

  #endif

      if (res > 0 && !eap->forceit)

  	qf_jump(qi, 0, 0, FALSE);		/* display first error */

      mch_remove(fname);

      vim_free(fname);

*** ../vim-7.3.196/src/version.c	2011-05-19 17:25:36.000000000 +0200

--- src/version.c	2011-05-19 17:41:50.000000000 +0200

***************

*** 711,712 ****

--- 711,714 ----

  {   /* Add new patch number below this line */

+ /**/

+     197,

  /**/

-- 

I AM THANKFUL...

...for the clothes that fit a little too snug because it

means I have more than enough to eat.

 /// Bram Moolenaar -- Bram@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    ///