[FFmpeg-devel] Type mismatch in ADPCM

Carl Eugen Hoyos ceffmpeg at gmail.com
Sun Mar 11 12:42:06 EET 2018


2018-03-11 11:27 GMT+01:00 Carlo Bramini <carlo.bramix at libero.it>:
> Hello,
> I see. I expected that adding that could be considered out of the coding guidelines.

You misunderstand:
The issue is not (afaict) the coding guidelines but the fact that the
change would mean we claim a compatibility that we cannot guarantee.

> However, what about the patch attached for fixing the declaration
> of ff_adpcm_afc_coeffs[2][16]?

This would revert 10542491, a relatively recent change: Maybe Paul,
the author, wants to comment. Do you think the code gets more
readable?

Please do not top-post here, Carl Eugen


More information about the ffmpeg-devel mailing list