On Mon, Mar 20, 2017 at 05:42:44PM -0700, Yegappan Lakshmanan wrote:
> On Mon, Mar 20, 2017 at 4:46 PM, James McCoy <[email protected]> wrote:
> > Why keep shoe-horning things into the set{qf,loc}list functions? This
> > just makes it an awkward API to use.
> >
> > I would suggest alternatives of
> >
> > qflistdelete([{list}])
> > loclistdelete({winid}[, {list}])
> > qflistreset()
> > loclistreset({winid})
> >
>
> Some time ago there was a push back on adding more functions for
> the quickfix functionality (as there are already too many quickfix
> commands/functions). So I am trying to add this to the setqflist()
> function.
Yes, I recall those discussions and I think part of the push back was
confusion about the purpose of the functionality that was being added.
The mechanism for setting the properties of the lists is particularly
awkward. I've been slowly working on explicit functions for
manipulating the properties of a list, since that's really orthogonal to
manipulating the list itself.
> Currently setqflist() is used to add or replace an existing
> quickfix list. It makes sense to extend this function to also support
> deleting a quickfix list.
I disagree that it makes sense as part of set{qf,loc}list, but if that's
the way to get the necessary functionality included, then so be it. I'd
expect usable APIs to be more important than simply keeping the function
count low.
At any rate, I know you're working within the constraints that are being
presented to you. Thank you for your work on improving quickfix.
Cheers,
--
James
GPG Key: 4096R/91BF BF4D 6956 BD5D F7B7 2D23 DFE6 91AE 331B A3DB
--
--
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.
