[FFmpeg-devel] Fwd: [PATCH] Psygnosis YOP demuxer

[Reimar Döffinger]

On Sat, Mar 27, 2010 at 04:51:33PM +0530, Mohamed Naufal wrote:
> On 27 March 2010 16:35, Mohamed Naufal <naufal11 at gmail.com> wrote:
> > On 27 March 2010 15:55, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> >> Mohamed Naufal <naufal11 <at> gmail.com> writes:
> >>
> >>> Patch attached.
> >>
> >> AFAICT, the aspect ratio is wrong.
> >> Test for example
> >> ffmpeg -i CIMOVI03.YOP test.avi
> >> ffmpeg -i CIMOVI03.YOP -s 580x348 test2.avi
> >> The second sample looks more correct (ffmpeg reports "Video: yop, pal8, 580x174,
> >> 12 fps, 12 tbr, 12 tbn, 12 tbc" for the original file), but I don't know what
> >> the exact ratio is. Please look at mobygames to compare.
> >>
> >
> > But http://samples.mplayerhq.hu/game-formats/yop/yop.txt says that the
> > sample at http://samples.mplayerhq.hu/game-formats/yop/CIMOVI01.avi is
> > a reference decoder output and it has the aspect ratio 580x174 too.
> >
> > Naufal
> >
> Ah, I checked out the screenshot at mobygames. Will post the updated
> patch in a bit. Will duplicating each line width wise do?

No, the demuxer should set sample_aspect_ratio correctly.