[Bug 692933] [API] codecparsers: mpeg2: add new API that takes GstMpegVideoPacket arguments

Thu Jan 31 04:05:17 PST 2013

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |t.i.m at zen.co.uk
     Ever Confirmed|0                           |1
           Severity|normal                      |enhancement

--- Comment #1 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2013-01-31 12:05:14 UTC ---
I think this makes sense and is nicer API too.

Not entirely sure about the naming and order of arguments though (the
inevitable bikeshedding, apologies): if the API is called
gst_blah_packet_parse_foo() then the first argument should probably be the
packet. That way bindings can map it to packet.parse_foo(). Or you make it
gst_blah_parse_foo_from_packet(), then packet is the last arg.

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