[MPlayer-dev-eng] [PATCH 2/2] vo_xv: Add NV12/NV21 support

Ville Syrjälä syrjala at sci.fi
Tue Mar 7 23:46:21 EET 2017

On Tue, Mar 07, 2017 at 09:48:29PM +0100, Reimar Döffinger wrote:
> On Sun, Feb 26, 2017 at 11:57:28PM +0200, Ville Syrjälä wrote:
> > Add support for NV12/NV21. Useful for cases when your hardware doesn't
> > do YV12/I420.
> It would be much easier and nicer for us to review if you could
> simply not re-indent the code you didn't modify.

I suppose I could have just put an early return for NV12/21 and
leave the rest as is. But it's a fairly tiny diff so figured it's
still legible even after diff mangles things in its usual silly way.
But I can redo it in whatever way people prefer.

I wonder if anyone's ever implement a diff algorithm that actually
makes good patches? At some point I was thinking that something
which tries to handle C blocks rather than lines might be a
decent approach.

> (or send a diff -uw variant or...)
> Quite bit worrying that the code doesn't actually check
> if we really support the format before it claims to support it
> (in query_format).

Yeah. It explodes in some interesting ways if the Xv adaptor supports
the specified format but vo_xv doesn't. I thought about fixing it, but
then I decided that if it's been like this all these years no one
likely cares ;)

Ville Syrjälä
syrjala at sci.fi

More information about the MPlayer-dev-eng mailing list