[FFmpeg-devel] Type mismatch in ADPCM

Carlo Bramini carlo.bramix at libero.it
Sat Mar 24 12:10:41 EET 2018


Hello,

> > > > 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?
> > 
> > Excuse me... I was just wondering what to do
> 
> You could answer above question.

I'm sorry, I was thinking that you did the question to Mr. Paul, since you were requesting a comment from him.
For me, I would say that this change makes the code more correct rather than more readable, which should be a more important reason, but that's just my opinion.

Sincerely.


More information about the ffmpeg-devel mailing list