[gstreamer-bugs] [Bug 620441] New: [PATCH] Add support for Y800 and Y16 formats

[GStreamer (bugzilla.gnome.org)]

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

           Summary: [PATCH] Add support for Y800 and Y16 formats
    Classification: Desktop
           Product: GStreamer
           Version: git
        OS/Version: All
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: Normal
         Component: gst-plugins-base
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: martin.bisson at gmail.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
      GNOME target: ---
     GNOME version: ---


Created an attachment (id=162618)
 --> (https://bugzilla.gnome.org/attachment.cgi?id=162618)
A patch implementing the requested feature

The fourcc formats Y800 (aka Y8 or GREY http://www.fourcc.org/yuv.php#Y800) and
Y16 (http://www.fourcc.org/yuv.php#Y16) don't have a GstVideoFormat defined in
gst-libs/gst/video/video.h.

Y800 actually is GST_VIDEO_FORMAT_GRAY8 and Y16 is actually
GST_VIDEO_FORMAT_GRAY16_LE.  However, if the function
gst_video_format_from_fourcc is called with either 'Y800', 'Y8  ' or 'Y16 ', it
will return GST_VIDEO_FORMAT_UNKNOWN.  These formats should be recognized.

I submitted a patch that fixes this by considering these YUV formats the same
as their YUV equivalent.  The only drawback I see from this patch is the fact
that the function gst_video_format_new_caps will return a YUV caps when the
formats GST_VIDEO_FORMAT_GRAY{8,16_LE} are given.  If this is a problem, it
could easily be changed to prioritize video/x-raw-gray caps by changing the
order of the checks in the function.

As soon as I will have submitted this, I will submit a second patch with a
different approach.

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