[FFmpeg-devel] [PATCH]Add little-endian G726 decoder and use it for Sun AU files

Carl Eugen Hoyos cehoyos at ag.or.at
Fri Aug 2 00:19:51 CEST 2013


Paul B Mahol <onemda <at> gmail.com> writes:

> > If you remember correctly, I had implemented a
> > solution before you implemented get_bits_le.
> > (But I of course prefer the solution with
> > get_bits_le very much.)
> 
> Your solution was just giving noise.

The original solution was bit-exact to the 
current posted one, the patch was just bigger.

> I will repeat once last time:
> 
> Adding another codec is wrong and is hack.

Why?

> Proper solution is implementing code that
> trys both paths and picks one with more sensible output

How would that help in a third-party application 
using lavf that wants to use a different decoder?

Carl Eugen



More information about the ffmpeg-devel mailing list