[FFmpeg-devel] [PATCH] ffprobe: Stash and use width and height before opening the codec

Stefano Sabatini stefasab at gmail.com
Tue Mar 19 23:42:37 CET 2013


On date Tuesday 2013-03-19 20:30:42 +0100, Michael Niedermayer encoded:
> On Tue, Mar 19, 2013 at 07:54:14PM +0100, Stefano Sabatini wrote:
[...]
> >  utils.c |    6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> > 0a4d156eb2d05abcaa48c8e422d1cafda36b029b  0001-lavc-utils-add-VP6F-hack-for-setting-correct-video-s.patch
> > From 6659d9027463960f5a4f2706f2d58556a2e40306 Mon Sep 17 00:00:00 2001
> > From: Stefano Sabatini <stefasab at gmail.com>
> > Date: Mon, 18 Mar 2013 23:17:55 +0100
> > Subject: [PATCH] lavc/utils: add VP6F hack for setting correct video size in
> >  stream
> > 
> > For some obscure reasons avcodec_set_dimensions() resets the width/height
> > so that the resulting value set in the stream is equal to the
> > corresponding coded_width/height, which is not the same as the correct
> > width/height in case of the H.264 and VP6F codecs.
> > 
> > This adds a codec-specific hack for VP6F which disables the call to
> > avcodec_set_dimensions() in avcodec_open2(), like it is done with H264. A
> > proper fix needs to be found yet.
> > 
> > Fix trac ticket #1386.
> > 
> > Based on a patch by Michael Niedermayer.
> > 
> > Trolled-By: Daemon404
> 
> LGTM

Applied, thanks.
-- 
FFmpeg = Fundamental and Frenzy Multimedia Programmable Enhancing Gorilla


More information about the ffmpeg-devel mailing list