[FFmpeg-cvslog] ffmpeg: fix aspect ratio setting

[Michael Niedermayer]

On Wed, Apr 06, 2011 at 01:27:02AM +0200, Michael Niedermayer wrote:
> On Tue, Apr 05, 2011 at 04:48:51PM +0200, Stefano Sabatini wrote:
> > ffmpeg | branch: master | Stefano Sabatini <stefano.sabatini-lala at poste.it> | Mon Apr  4 02:15:34 2011 +0200| [d1eb50bb29caad9745631759265f475177df99b9] | committer: Stefano Sabatini
> > 
> > ffmpeg: fix aspect ratio setting
> > 
> > This is done by adding a setsar filter at the beginning of the
> > configured filterchain.
> > 
> > This implementation is more robust, since does not modify the
> > filterchain description (which was creating potential syntax errors),
> > but directly modifies the filterchain structure.
> > 
> > This also changes the ffmpeg -aspect behavior, as following filters in
> > the filterchain can change the DAR/SAR set by the inserted setsar
> > filter.
> > 
> > Signed-off-by: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> > 
> > > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d1eb50bb29caad9745631759265f475177df99b9
> > ---
> > 
> >  ffmpeg.c |   22 ++++++++++------------
> >  1 files changed, 10 insertions(+), 12 deletions(-)
> 
> this breaks fate-dpx
> [setsar @ 0x1a1b990] Invalid string '0:0' for aspect ratio.

reverted for now

please comment, if you agree with baptistes patch or have another
suggestion


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20110406/e1e84b22/attachment.asc>