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

Paul B Mahol onemda at gmail.com
Fri Aug 2 14:28:55 CEST 2013


On 8/1/13, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> 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?

Code for checking be/le should not be in lavf
at first place.


More information about the ffmpeg-devel mailing list