[FFmpeg-devel] [PATCH] avcodec/alsdec: implement floating point decoding

Thilo Borgmann thilo.borgmann at mail.de
Sat Aug 6 20:05:31 EEST 2016


Am 06.08.16 um 17:26 schrieb Umair Khan:
> On Fri, Aug 5, 2016 at 9:27 PM, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>> Hi!
>>
>> 2016-08-05 16:57 GMT+02:00 Umair Khan <omerjerk at gmail.com>:
>>> Updated patch attached.
>>
>> The Changelog hunk of your patch indicates that the patch should be split.
>> (Or the change to Changelog could be wrong.)
> 
> I don't know. I can do it if the other people also think this.
 
Yes. Split it up into three parts: SoftFloat_IEEE754, MLZ and the ALS decoding of floats.

All these parts alone make sense for a separate commit although they are only used in ALS (by now).
You can (assuming you are planning to maintain your code) also add yourself to MAINTAINERS for SoftFloat & MLZ files you add.

-Thilo


More information about the ffmpeg-devel mailing list