[MPlayer-cvslog] r31629 - in trunk: configure etc/codecs.conf

[Reimar Döffinger]

On Sun, Jul 04, 2010 at 11:55:39AM -0400, compn wrote:
> On Sun, 04 Jul 2010 14:28:02 +0200, Diego Biurrun wrote:
> >On Sat, Jul 03, 2010 at 10:54:54PM +0200, reimar wrote:
> >> 
> >> Log:
> >> Support decoding and encoding via libgsm via libavcodec.
> >
> >Should this replace the native wrapper and/or the local library fork?
> 
> thats reimar's plan, see thread on ffmpeg-devel.

(first an apology to the libgsm authors, I can see a lot of good reasons
why it is the way it is, but none of them are relevant to MPlayer).
Alone, not is is not supposed to replace it. In part because it would
not be "built-in" (and thus quite certainly no supported in windows binaries),
but more importantly because libgsm is huge, increadibly messy, nearly
100% based on the spec even when it makes not the slightest sense
(doing the dequant before synthesis, thus having to do 3x as many writes
and additions is a major WTF) and (I think) designed for a pure
16 bit CPU with a special fused multiply, round and shift instruction
and always saturating adds and subtracts seems unacceptable to have as
the only decoder for this format.
So this together with my (probably in corner-cases buggy, but 300 instead
of > 1000 lines and it seems more than 2x as fast) native FFmpeg decoder
is supposed to replace it.
Review pending though...