[MPlayer-dev-eng] [PATCH] Support newer GIFLIB versions
Alexander Strasser
eclipse7 at gmx.net
Thu Jul 25 01:10:28 CEST 2013
Hi Reimar!
On 2013-07-24 22:39 +0200, Reimar Döffinger wrote:
> On 23.07.2013, at 08:03, Alexander Strasser <eclipse7 at gmx.net> wrote:
> > On 2013-07-23 00:44 +0200, Nicolas Coiffier wrote:
> >> Works for me, and fix previous compilation problems withlibmpdemux/demux_gif.c
> >>
> >> see thread [MPlayer-dev-eng] [PATCH] Fix compilation error in
> >> libmpdemux/demux_gif.c:
> >> http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2013-July/071813.html
> >
> > Great. Thank you for testing. I intent to apply some solution when the
> > discussions are over.
>
> just go ahead, this got lost in time already once, let's not repeat that.
OK, will commit v2 soon if no more problems are found.
> That said, with this kind of code nobody really cares about, that should have working alternatives in FFmpeg etc. I personally don't consider backwards compatibility to be worth even minimal extra complexity.
In princinple I agree with you. But as the reporter had a 4.2 version
and it seems this is still widely spread my initial objective was to make
it work with that version. While investigating I found out about version
5 and tried to make it work with that as well.
If more problems arise now or in future I will propose dropping
backward compatibility.
Alexander
More information about the MPlayer-dev-eng
mailing list