[Bug 738033] mpegts : add new descriptors and cleanup

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Tue Oct 28 07:34:08 PDT 2014


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

--- Comment #21 from chamois94 at gmail.com 2014-10-28 14:34:06 UTC ---
(In reply to comment #20)
> A few comments:
> 
> * Please put your email address (alongside your name) in the headers
> * Some of this code was just moved from other files, please include the
> original authors in the headers
> * For the new descriptor parsing you've added, could you also add the
> corresponding support in tests/examples/mpegts/ ? This helps to make sure the
> API is correct.
> * The various API addition need to be added to the docs (docs/libs/)
> 
> The enum addition/movement are fine (once licenses are updated), I'll push them
> once you've updated those.

Done for the first two points.

I'll add unit tests and docs as soon as I have some time.

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