[FFmpeg-cvslog] ffmpeg: fix aspect ratio setting

Michael Niedermayer michaelni at gmx.at
Wed Apr 6 01:27:02 CEST 2011


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.

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- 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/ffd66d2c/attachment.asc>


More information about the ffmpeg-cvslog mailing list