[Bug 687330] New: Some video plugins wrongly use _WIDTH macros for height

[GStreamer (bugzilla.gnome.org)]

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

           Summary: Some video plugins wrongly use _WIDTH macros for
                    height
    Classification: Platform
           Product: GStreamer
           Version: unspecified
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: douglas at halo.gen.nz
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


[from the patch comment]

In videomixer/blend.c, I found a couple of cut-and-pastos like this:

  dest_width = GST_VIDEO_FRAME_WIDTH (destframe); \
  dest_height = GST_VIDEO_FRAME_WIDTH (destframe); \

where the second one should almost certainly be using
GST_VIDEO_FRAME_HEIGHT, not _WIDTH.

Out of interest, I ran this:

$ git grep 'height.\+WIDTH'

and found a similar case in videobox.

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