[Bug 712809] d3dvideosink shows corrupted output with some videos

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=712809
  GStreamer | gst-plugins-bad | git

--- Comment #12 from Sebastian Dröge (slomo) <slomo at coaxion.net> 2014-05-31 13:01:45 UTC ---
(In reply to comment #10)
> (In reply to comment #8)
> > The size calculations for the packed formats in the old code are wrong for one,
> > but that shouldn't be the problem here.
> > 
> > The combined I420/YV12 case is correct, including the plane swapping.
> 
> Where do you see in the documentation that U and V should be swapped ?

YV12 is Y, V, U and I420 is Y, U, V. Windows supports YV12, so we swap here
accordingly to support both. About that part I'm pretty sure, it works ;)

> > 
> > The only potential problem I currently see here is that for NV12 the
> > documentation talks about an *even* number of lines for the Y plane. Otherwise
> > this all looks correct to me.
> 
> VideoInfo adds some height padding for the odd case.

It adds another *line* to the Y component if the height is odd? Really? That
would be a weird default

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.