[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

Sebastian Dröge <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #227813|none                        |needs-work
             status|                            |

--- Comment #1 from Sebastian Dröge <slomo at circular-chaos.org> 2012-11-01 15:49:52 UTC ---
Review of attachment 227813:
 --> (https://bugzilla.gnome.org/review?bug=687350&attachment=227813)

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

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