[gstreamer-bugs] [Bug 625944] New GstDiscoverer helper library

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=625944
  GStreamer | gst-plugins-base | git

--- Comment #32 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2010-09-16 14:12:17 UTC ---
> >  - does GstDiscovererResult really need to be
> >    flags? This seems awkward to me, and quite
> >    unnecessary too.
> 
>   Without suggestions and rationale for what it should be instead it'll stay a
> flag.

It didn't seem necessary to me to provide either a 'rationale' in favour of
simple error/status enums, nor a suggestion of the obvious alternative, namely
an enum. Both seemed rather obvious. Flags are awkward and unusual (and, imho,
unnecessary here) - you need to justify why they are prefered/needed, not the
other way around.

As for the other things - ok/dunno, maybe Sebastian or others have opinions
that make these things go in one direction or another.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.