[FFmpeg-devel] [PATCH 2/2] avformat/rpl: Support files containing Replay IMA ADPCM audio
Zane van Iperen
zane at zanevaniperen.com
Tue May 4 14:57:06 EEST 2021
On 4/5/21 6:10 am, Cameron Cawley wrote:
> + case 2:
> + if(av_stristr(audio_codec, "adpcm") != NULL) {
Nit: use spaces here. Also "if ("
> + ast->codecpar->codec_id = AV_CODEC_ID_ADPCM_IMA_ACORN;
> + break;
> + }
> + break;
I would ask why two breaks, but then I noticed the rest of the code does the same. Never mind.
Otherwise, both this and part 1 lgtm.
Zane
More information about the ffmpeg-devel
mailing list