[gstreamer-bugs] [Bug 626181] [pbutils] Add enhanced gstfactorylists

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #21 from Edward Hervey <bilboed at gmail.com> 2010-08-19 13:53:07 UTC ---
(In reply to comment #20)
> Looks good but you say that the first flags are exclusive. As such the ANY
> define doesn't make sense. If anything there should be an ANY_MEDIA define,
> which has bit 48 and above set only.

Gah, good point. let me change that so that ANY is only for the lower 47 bits.

ANY_MEDIA can't be done with all higher bits set, since you might have some
factories that don't have any media-specific class... and that would fall into
the search with no media specified.

> 
> Now there's only the question if we really want to use a GValueArray here or if
> some other data structure would be more appropiate. IMHO GValueArray is a bit
> uncomfortable to use from C :)

  I'm fine with switching it to a GList of refcounted GstPluginFeature and
share the same code as the other plugin feature list code from
gstpluginfeature/gstregistry.

> 
> What are you going to do with the gst_caps_list_container_formats() and other
> similar functions? New patch against -base later?

  New patch later on, that part is still work in progress and not 100% critical
imho.

  I'm cooking up a new patch.

-- 
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.