[MPlayer-dev-eng] [PATCH] Add giflib 5 support
Reimar.Doeffinger at gmx.de
Sat May 4 08:34:25 CEST 2013
On 02.05.2013, at 11:50, Andrew John Hughes <gnu_andrew at member.fsf.org> wrote:
> Trying again... http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2013-April/071595.html
> The attached patch allows mplayer to be built with giflib 5. Tested
> against 1.1 and SVN.
Sorry, I forgot to ask a few things.
I was wondering if there is much "need" for it, since FFmpeg does support gif quite fully I think. Basically I am curious why you care :-)
This is related to whether there is much of a point in keeping support for the old version.
I also think if we want to keep support for the old version it might look nicer if done via #defines (e.g. throwing away extra parameters is one single trivial line of code like that).
If I saw right you also added some extra error checking, it would be kind of nice to have that as a separate patch.
More information about the MPlayer-dev-eng