[gstreamer-bugs] [Bug 363118] New: gst_riff_create_video_caps() should also store variant in the caps

[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=363118

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

           Summary: gst_riff_create_video_caps() should also store variant
                    in the caps
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: Normal
         Component: gst-plugins-base
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: ensonic at sonicpulse.de
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


gst_riff_create_video_caps() as used by the bundles several FourCC tags into
one mimetype. This happens for several *263 variants. The resulting type in the
caps is "video/x-h263" in this case. It is important to preserve the variant as
not all codecs support them all.

For the H263 variants codecs usually decode garbled video if they can reject
variants they don't support.

In http://bugzilla.gnome.org/show_bug.cgi?id=361636 (and #361637) there is a
proposal to add a variant element (G_TYPE_STRING) to the caps.

Riff-lib could easily used the 'codec_name' string. Unfortunately there are no
standard names or types for this.

Alternatively we could also define an enum in
gst-plugins-base/gst-libs/video/video.h as using non-standartized strings for
this isn't nice.


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