[FFmpeg-cvslog] Insert setdar at the begin of a filter chain instead the end.

[Michael Niedermayer]

On Mon, Apr 04, 2011 at 05:45:31PM +0200, Stefano Sabatini wrote:
> On date Monday 2011-04-04 12:31:03 +0200, Michael Niedermayer wrote:
> > On Mon, Apr 04, 2011 at 02:46:47AM +0200, Stefano Sabatini wrote:
> [...]
> > > Check patch (not correct, working on it, some re-design required),
> > > treat sample_aspect the same way w/h is treated.
> > 
> > doesnt work, try:
> > ffmpeg -vframes 3 -y -i ~/videos/matrixbench_mpeg2.mpg test.avi
> > ffprobe test.avi
> > 
> > Now:
> > Stream #0.0: Video: mpeg4, yuv420p, 720x576 [PAR 16:15 DAR 4:3], 25 tbr, 25 tbn, 25 tbc
> > 
> > Your patch:
> > Stream #0.0: Video: mpeg4, yuv420p, 720x576 [PAR 16:15 DAR 4:3], PAR 1:1 DAR 5:4, 25 tbr, 25 tbn, 25 tbc
> 
> $ ffprobe test.avi 
> Input #0, avi, from 'test.avi':
>   Metadata:
>     encoder         : Lavf52.104.0
>   Duration: 00:00:00.12, start: 0.000000, bitrate: 3384 kb/s
>     Stream #0.0: Video: mpeg4, yuv420p, 720x576 [PAR 16:15 DAR 4:3], 25 tbr, 25 tbn, 25 tbc
>     Stream #0.1: Audio: mp2, 48000 Hz, stereo, s16, 64 kb/s
> 
> Works fine here, please elaborate. Patch updated with a fix.

This patch works for me, if you prefer it iam fine with it

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

In fact, the RIAA has been known to suggest that students drop out
of college or go to community college in order to be able to afford
settlements. -- The RIAA
-------------- 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/20110404/b65f9beb/attachment.asc>