[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