[MPlayer-dev-eng] [PATCH] (hopefully) better dumpvideo

[Arpi]

Hi,

> this is a patch that removes from the dump the four bytes (currently 
> added) indicating the packet size when the original file format is AVI.
> 
> The benefit is that when dumping video (especially mpg in avi) you have 
> the REAL video stream.
> 
> I don't understand why/when adding four extraneous bytes for every 
> packet read from the stream can be useful, so

because most codecs (actually imho all except teh mpeg ones which is rare)
in .avi and other frame-based containers are framed, so you cannot playback
(or use for any purpose) rte resulting dump unles syou know teh size of each
video packet. by dumping the raw data only you lose this info.

the proper solution would be a codec-based decision (skip packet size for
mpeg codecs only) or an commandline option.

> I would extend the patch to the other file formats still treated 
> differently: MOV and ASF.

i vote against it


A'rpi / Astral & ESP-team

--
Developer of MPlayer G2, the Movie Framework for all - http://www.MPlayerHQ.hu