[Bug 702826] gdkpixbufoverlay: image overlay doesn't work for -ve offset

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Jun 21 09:45:21 PDT 2013


https://bugzilla.gnome.org/show_bug.cgi?id=702826
  GStreamer | gst-plugins-good | 1.0.7

Nicolas Dufresne <nicolas.dufresne> changed:

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

--- Comment #1 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2013-06-21 16:45:16 UTC ---
Review of attachment 247464:
 --> (https://bugzilla.gnome.org/review?bug=702826&attachment=247464)

This seems more correct, I'd need to look at more context, meawhile there is
some small fixes required.

::: ext/gdk_pixbuf/gstgdkpixbufoverlay.c
@@ +141,3 @@
+          "For +ve value, horizontal offset of overlay image in pixels from"
+          " left of video image. For -ve value, horizontal offset of overlay"
+          " image in pixels from right of video image", G_MININT, G_MAXINT, 0,

Might be me, but I see no difference in those two, ve is not very common
acronym I think.

@@ +459,3 @@
+    video_height+overlay->offset_y - overlay_meta->height +
+    (overlay->relative_y * overlay_meta->height) :
+    overlay->offset_y + (overlay->relative_y * overlay_meta->height);

Coding style require that operators (-+) have spaces before and after. It's
quite clear though that the original code would not work with negative values.

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


More information about the gstreamer-bugs mailing list