[Bug 687350] Add nv12/nv21 converter to i420

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=687350
  GStreamer | gst-plugins-base | 0.10.x

--- Comment #2 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2012-11-02 09:02:05 UTC ---
(In reply to comment #1)
> Review of attachment 227813 [details]:
> 
> Actually this should go through Y444 and not RGB... sounds like another bug
> too.
> 
> ::: gst/ffmpegcolorspace/imgconvert.c
> @@ +1575,3 @@
> +  /* Split UY plane to seperate U and V planes */
> +  for (w = width / 2; w--;) {
> +    for (h = height / 2; h--;) {
> 
> This is wrong for odd width/height
> 
> @@ +1692,3 @@
> +  /* Split UY plane to seperate U and V planes */
> +  for (w = width / 2; w--;) {
> +    for (h = height / 2; h--;) {
> 
> This is wrong for odd width/height too

You got a very good point, let me o back to code then.

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