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

Paul B Mahol onemda at gmail.com
Thu Aug 1 23:59:10 CEST 2013


On 8/1/13, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Paul B Mahol <onemda <at> gmail.com> writes:
>
>> > whats the status of this patch & thread ?
>>
>> I already did posted patch for g726
>
> Maybe I misremember, but your patch does not
> work or do I miss something?
>
>> and I'm the one who originally found solution.
>
> To some degree, yes.
> (Roman explained it several years ago and I pointed
> out the German Wikipedia article iirc.)
>
>> If I did not posted patch,
>
>> also if I never commited get_bits_le
>
> 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.

I will repeat once last time:

Adding another codec is wrong and is hack.

Proper solution is implementing code that
trys both paths and picks one with more sensible output, which I
already mentioned in another thread.


More information about the ffmpeg-devel mailing list