[Bug 737339] gifdec: new element

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue May 2 14:31:41 UTC 2017


https://bugzilla.gnome.org/show_bug.cgi?id=737339

Mathieu Duponchelle <mduponchelle1 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO

--- Comment #19 from Mathieu Duponchelle <mduponchelle1 at gmail.com> ---
I ended up looking at ffmpeg's implementation
(http://ffmpeg.org/doxygen/trunk/gifdec_8c-source.html), and the answer to some
of my review comments is that the proposed patches are an adapted copy paste of
this implementation.

Doing that without retaining the copyrights or even mentioning the original
source isn't good practice.

As such, I'm not sure we want to merge any of these patches, as ffmpeg is
already wrapped by gst-libav.

Setting to NEEDINFO, to let Sebastian or Tim decide

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