2016-09-14 18:56 GMT+02:00 Yogender Kumar Gupta <yogender.gu...@gmail.com>:
> Please let me know if anything else needs to be taken care of.

if (condition) {
    do();
} else {
  do2();
}

like everywhere else in FFmpeg (and on top of your patch).

+    if (ctx->aq && ctx->temporalAQ)
+    {
+        av_log(avctx, AV_LOG_ERROR, "AQ and Temporal AQ are not
supported together\n");
+    }

Printing the error is sufficient, neither return nor unsetting one of them?

Please do not top-post here, Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to