[gstreamer-bugs] [Bug 620441] New: [PATCH] Add support for Y800 and Y16 formats
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Thu Jun 3 01:41:49 PDT 2010
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.
More information about the Gstreamer-bugs
mailing list