[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.
>> 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