[Bug 708438] codecparsers: HEVC bitstream parser

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Tue Nov 5 12:22:53 PST 2013


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

--- Comment #7 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2013-11-05 20:22:45 UTC ---
Sure, we can get that in.

Note that primary vs. secondary rank does not really matter at all, what
matters is  more none vs. a rank (will it be autoplugged and subjected to
random incoming data or not) ;-)

I noticed there's a bit of a naming inconsistency (also present in the H264
codec parser):

    GstH265NalParser * gst_h265_nal_parser_new (void);

  and then

    gst_h265_parser_identify_nalu(GstH265NalParser *nalparser,...)

rather than

    gst_h265_nal_parser_identify_nalu (GstH265NalParser *nalparser,...)


Any reason not to do

   GstH265NalParser -> GstH265Parser
   gst_h265_nal_parser_*() -> gst_h265_parser_*()

?

(It's more similar to gst_h265_parse_*/GstH265Parse then, but I don't think
that's a problem).

If yes, no need for a patch, I can change it locally, was just wondering.

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