[FFmpeg-devel] ffmpeg -s breaks aspect ration (regression)

[Michael Niedermayer]

Hi

On Wed, Jul 20, 2011 at 04:44:56PM +0200, Jean Delvare wrote:
> Last time I updated ffmpeg, I hit a problem where option -s breaks the
> aspect ratio. I bisected the repository and found that this regression
> was introduced by:
> 
> commit 7a11c82fb760619d0e17ca234381b50d6c675cc0
> Author: Michael Niedermayer <michaelni at gmx.at>
> Date:   Mon Jan 31 20:48:35 2011 +0100
> 
>     vsrc_buffer: add sample_aspect_ratio fields to arguments.
>     
>     This fixes aspect handling in ffmpeg.
>     
>     This is based on a patch by Baptiste.
>     
>     Signed-off-by: Anton Khirnov <anton at khirnov.net>
> 
> My input video looks like:
> 
> Stream #0.0[0x1e0]: Video: mpeg2video (Main), yuv420p, 720x576 [PAR 64:45 DAR 16:9], 9500 kb/s, 25 fps, 25 tbr, 90k tbn, 50 tbc
> 
> I am passing -s 640x360 to ffmpeg to get square pixels, before the bug
> I would get:
> 
> Stream #0.0(und): Video: mpeg4, yuv420p, 640x360 [PAR 1:1 DAR 16:9], 640 kb/s, 25 fps, 25 tbr, 25 tbn, 25 tbc
> 
> and the video would look OK. Since the commit above, I get:
> 
> Stream #0.0(und): Video: mpeg4, yuv420p, 640x360 [PAR 1:1 DAR 16:9], 743 kb/s, PAR 91:64 DAR 91:36, 25 fps, 25 tbr, 25 tbn, 25 tbc

This bug seems to have been fixed as i cannot reproduce this anymore

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- 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-devel/attachments/20110801/b42b795a/attachment.asc>