Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
patch 8.0.1772: quickfix: mixup of FALSE and FAIL, returning -1
Problem:    Quickfix: mixup of FALSE and FAIL, returning -1.
Solution:   Use FAIL and INVALID_QFIDX. (Yegappan Lakshmanan)
  • Loading branch information
brammool committed Apr 28, 2018
1 parent 50182fa commit 29ce409
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/quickfix.c
Expand Up @@ -2351,7 +2351,7 @@ qf_jump_edit_buffer(
if (!can_abandon(curbuf, forceit))
{
no_write_message();
retval = FALSE;
retval = FAIL;
}
else
retval = do_ecmd(qf_ptr->qf_fnum, NULL, NULL, NULL, (linenr_T)1,
Expand Down Expand Up @@ -2395,7 +2395,7 @@ qf_jump_edit_buffer(
}

if (*abort)
retval = FALSE;
retval = FAIL;
}

return retval;
Expand Down Expand Up @@ -4172,7 +4172,7 @@ qf_id2nr(qf_info_T *qi, int_u qfid)
for (qf_idx = 0; qf_idx < qi->qf_listcount; qf_idx++)
if (qi->qf_lists[qf_idx].qf_id == qfid)
return qf_idx;
return -1;
return INVALID_QFIDX;
}

/*
Expand Down Expand Up @@ -4889,7 +4889,7 @@ get_errorlist(qf_info_T *qi_arg, win_T *wp, int qf_idx, list_T *list)
}
}

if (qf_idx == -1)
if (qf_idx == INVALID_QFIDX)
qf_idx = qi->qf_curlist;

if (qf_idx >= qi->qf_listcount
Expand Down
2 changes: 2 additions & 0 deletions src/version.c
Expand Up @@ -761,6 +761,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
/**/
1772,
/**/
1771,
/**/
Expand Down

0 comments on commit 29ce409

Please sign in to comment.