[gstreamer-bugs] [Bug 407282] New: [decodebin2] autoplug-sort signal has GList ** parameter

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=407282

  GStreamer | gst-plugins-base | Ver: HEAD CVS

           Summary: [decodebin2] autoplug-sort signal has GList ** parameter
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-base
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: t.i.m at zen.co.uk
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


This slipped into decodebin2:

  /* signal fired to reorder the proposed list of factories */

  gboolean (*autoplug_sort) (GstElement * element, GstCaps * caps, GList **
list);

As if a GList * parameter wasn't bad enough as a signal parameter, but a GList
** ?!

Can we please quickly change this into a GValueArray before anyone starts using
decodebin2 for anything?

(I'd argue that it was only semi-functional in the last release, so chances
that anyone was using it are pretty slim IMHO, and those that were using
decodebin2 are very unlikely to have been using this signal, I'd think; better
anounce this on the mailing list and fix it than be stuck with it until the end
of the 0.10 cycle ...)


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email