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

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Aug 6 09:35:43 PDT 2010


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

--- Comment #9 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2010-08-06 16:35:37 UTC ---
> First of all I don't think this belongs into core... There's stuff about audio
> and video in here.

Like what? It's an enum or a bunch of flags. It's not more "audio" or "video"
than GST_TAG_AUDIO_CODEC.

It seems silly to scatter clearly related API all over the place just to avoid
the word AUDIO in an enum.

Code like this in gstplaybin2.c

  if (!gst_factory_list_is_type (factory, GST_FACTORY_LIST_SINK))
    return GST_AUTOPLUG_SELECT_TRY;

just screams "please, make me part of the GstElementFactory API" IMHO. I'm just
talking about the factory list stuff here.


> GST_FACTORY_LIST_AV_SINKS

I think AV should be expanded, and SINKS should maybe be singular to match
_ENCODER etc?

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



More information about the Gstreamer-bugs mailing list