[Bug 767908] jpeg2000parse: use enums for colorspace and sampling, rather than strings

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue Jun 21 19:07:15 UTC 2016


https://bugzilla.gnome.org/show_bug.cgi?id=767908

--- Comment #11 from boxerab at gmail.com <boxerab at gmail.com> ---

> 
> ::: ext/openjpeg/gstopenjpegenc.c
> @@ +118,3 @@
>          "num-components = (int) [1, 4], "
>          GST_RTP_J2K_SAMPLING_LIST ","
> +        GST_JPEG2000_SAMPLING_COLORSPACE_LIST "; "
> 
> This might be worth moving to the library too... but the naming should all
> be more consistent. GST_JPEG2000_* maybe, and call it in the style of
> GST_VIDEO_FORMATS_ALL

Thanks. You mentioned a plan to eventually replace colorspace with the sampling
field. If that's the case, should we not be using something more general than
JPEG2000 to namespace the sampling strings?

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