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

[Nico]

I'll post the unified patch tonight, thank you for your fixes.

Diego Biurrun wrote:

>Nico writes:
> > attached is the patch to mplayer.1 (I hope it's correct, mc shows it 
> > correctly, but I don't know that strange formatting language).
> > 
> > Watch: the real patch hasn't been committed yet.
> > 
> > Hope to have gained some good karma :)
>
>I would suggest you post a new, complete version of your patch, this
>will be easier to review and commit then.
>
>I suggest you rename the -rawdump option to -dumprawvideo to be more
>consistent with the other -dump options.
>
>Attached is an updated patch with this update included as well as
>better (imho) wording.
>
>Diego
>
>  
>
>------------------------------------------------------------------------
>
>Index: DOCS/en/mplayer.1
>===================================================================
>RCS file: /cvsroot/mplayer/main/DOCS/en/mplayer.1,v
>retrieving revision 1.403
>diff -u -r1.403 mplayer.1
>--- DOCS/en/mplayer.1	7 Jul 2003 23:02:25 -0000	1.403
>+++ DOCS/en/mplayer.1	17 Jul 2003 11:51:51 -0000
>@@ -702,7 +702,14 @@
> Useful when ripping from DVD or network.
> .TP
> .B \-dumpvideo (MPLAYER only)
>-Dump raw compressed video stream to ./\:stream.dump (not very usable).
>+Dumps raw compressed video stream to ./\:stream.dump (not very usable).
>+When the source file is AVI/MOV/ASF MPlayer adds a 4 byte header for every
>+packet to the dump, use \-dumprawvideo to avoid this.
>+.TP
>+.B \-dumprawvideo (MPLAYER only)
>+Dumps raw, headerless compressed video stream to ./\:stream.dump
>+(not very usable).
>+Useful for MPEG1/2 video streams in AVI.
> .TP
> .B \-dvd\-device <path\ to\ device>
> Override default DVD device name /dev/\:dvd.
>  
>
>------------------------------------------------------------------------
>
>_______________________________________________
>MPlayer-dev-eng mailing list
>MPlayer-dev-eng at mplayerhq.hu
>http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>  
>