[gstreamer-bugs] [Bug 556336] New: [PATCH] gstffmpegdec.c: prefer demuxer's pixel aspect ratio over the decoder's

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=556336

  GStreamer | gst-ffmpeg | Ver: HEAD CVS
           Summary: [PATCH] gstffmpegdec.c: prefer demuxer's pixel aspect
                    ratio over the decoder's
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-ffmpeg
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: robin at nibor.org
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


When both the demuxer and the decoder provide a non-1:1 PAR, currently the
decoder is preferred. This patch changes it so that the demuxer is preferred,
because that allows a user to control the PAR with the container (changing the
PAR in the video stream is much more difficult). See also bug 555699 for the
same problem with theoradec.

The current code does some "guessing" in that when both specify a PAR, the one
with the non-square PAR is preferred. This means it doesn't respect a square
PAR that is set in the container, but I don't know how important that is. Maybe
that code should be changed to always prefer the demuxer's PAR, that would be
more in line with the behaviour that is now implemented in theoradec and
mpeg2dec (see bug 555699).


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=556336.