[Bug 664274] [API] codecparsers: add gst_mpeg_video_parse_slice() and gst_mpeg_video_parse_sequence_scalable_extension()

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Mon Feb 18 02:58:23 PST 2013


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

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #234906|none                        |reviewed
             status|                            |

--- Comment #20 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2013-02-18 10:58:20 UTC ---
(From update of attachment 234906)
I don't really have an opinion on this patch, it seems like a
micro-optimisation that surely is not going to really make a difference? If in
doubt, I tend to prefer clear code (but again, here it hardly matters). Anyway,
I'll leave it to Gwenole to decide.

If you do push it, please fix up the commit message (weird space placement in
subject line, and coparison->comparison) and the comments in the code (use '*'
in each comment line, and also coparison->comparison).

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