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

[Arpi]

Hi,

> 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.

imho it should be a flag-type option, called -dumpraw or so, and it
should affect both -dumpaudio, -dumpvideo and -dumpsub behaviour.
(it's useful to include packet size in audio too, for some formats,
like vorbis or qdm2)


A'rpi / Astral & ESP-team

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