[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