Yegappan wrote:
> The quickfix code currently checks whether a quickfix/location > stack is empty by using the qf_listcount variable. > The attached patch replaces these checks by using a helper > function. It does change the logic, is this correct? E.g.: - if (qi == NULL || (qi->qf_listcount == 0 - && qf_list_empty(qi, qi->qf_curlist))) + if (qf_stack_empty(qi) || qf_list_empty(qi, qi->qf_curlist)) This changes && into ||. And this: - if (action == ' ' || qi->qf_curlist == qi->qf_listcount) + if (action == ' ' || qf_stack_empty(qi)) Was comparing qf_listcount with qf_curlist, now it compares with zero. -- hundred-and-one symptoms of being an internet addict: 144. You eagerly await the update of the "Cool Site of the Day." /// Bram Moolenaar -- [email protected] -- 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 /// -- -- 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.
