[Bug 63807] No way to filter devices

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=63807

--- Comment #7 from Zeeshan Ali <zeenix at gmail.com> ---
(In reply to comment #6)
> (In reply to comment #2)
> > (In reply to comment #1)
> > > This is fixed by this commit, closing:
> > > http://cgit.freedesktop.org/spice/spice-gtk/commit/
> > > ?id=0ebcc23c0642d2bddcb05ed4152497c99cb3ca1d
> > 
> > Sorry but you are not getting away that easy. :) As I pointed out on IRC,
> > its a very ugly API if you are asking apps to give you a very cryptic string
> > for filter. Surely you can add some enums/flags and allow apps to filter
> > through that. If you fail in that, at the very least add some docs
> > explaining the syntax of the filter string you are expecting from apps.
> 
> I have to disagree, a query string is actually more friendly API here.

You'll have to do better than just saying that. :) Most APIs can be "easy" with
strings but more static APIs are always preferred for a few reasons. Firstly,
you get errors from compiler if you screw up with static API. Secondly, with C
many times developers just reference API from headers rather than docs if the
API is obvious (which we should try our best to do).

Also, its not nice that you have ignored this up til now when someone decided
to fix the issue. :(

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/spice-bugs/attachments/20140716/a79dc359/attachment.html>