[MPlayer-dev-eng] [PATCH] Fix yuv4mpeg output, remove spurious first frame.

[Linards Ticmanis]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Glen Nakamura wrote:

>>"-vo yuv4mpeg" currently outputs a spurious all-zero (i.e. fully
>>saturated green - *ugly*) first frame. This patch gets rid of this. Now
>>"-ss <some time> -frames 13" for example will give you exactly 13 frames
>>in the stream.yuv file.
>
>
> the blank frames have helped me with a/v sync in the past.  e.g. i
wouldn't
> have to specify an offset when multiplexing the a/v streams.
> but, i didn't like them being green so i use the following patch
> to make them black...

I don't use mplayer for the sound normally if I have a Dolby Digital
source... I use Azid since there I know the Dolby Surround compatible
downmix will really work. I found that the blank frame actually seems to
destroy a/v sync in that case, and this is the reason I wrote this
kludge in the first place. Or am I wrong? A desync of 1 frame is not
easy for me to notice with 100% accuracy.

But, the blank frames are not good when you join more than one source
together... of course I wrote a little C thingy to throw out the
additional yuv4mpeg headers, I guess I could modify it to also throw out
1 frame after each header, but...

I believe mplayer shouldn't output anything that's not there in the
source. Am I right with this general principle?

- --

Linards Ticmanis

 The Master said, "The business of laying on the colors follows the
preparation of the plain ground."
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQE+80i0pXWZ5wbYWboRAvFTAKCyXokjrBMUw1eqnD8CRjjmzGhkLACfdzbA
Cz6ArBeuDi9jnQaLbuqoliw=
=DfuK
-----END PGP SIGNATURE-----