[gstreamer-bugs] [Bug 393187] New: ffenc_mpeg4 does not set codec_data on 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=393187

  GStreamer | gst-ffmpeg | Ver: 0.10.10

           Summary: ffenc_mpeg4 does not set codec_data on caps
           Product: GStreamer
           Version: 0.10.10
          Platform: Other
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-ffmpeg
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: thomas at apestaart.org
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


For the .mp4 file format, it is mandatory for the codec_data/extradata/header
bytes to be stored inside the container.  For this to work, ffenc_mpeg4 needs
to put them on the caps as codec_data, so that the muxer can store them.

The attached patch does this for MPEG4 video and fixes a related problem.

Maybe this needs to be exposed as a property because in some containers it is
valid to mux the codec header in stream as well.


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