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

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to