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

[Nico]

Here it is:

-rawdump as flag to disable the 4 bytes; it's disabled by default

BTW,
Albeau, do you plan to apply my dvb patch? Or is there anything wrong?

                    Nico

Alban Bedel wrote:

>Hi Arpi,
>
>on Mon, 14 Jul 2003 22:10:32 +0200 you wrote:
>
>  
>
>>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
>>    
>>
>I vote for a command line option.
>	Albeu
>
>  
>

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: mplayer-20030714.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20030714/3bca7fc1/attachment.asc>