On Thu, Jun 13, 2019 at 08:52:22 -0700, Amir Pauker wrote:
>  doc/APIchanges       | 3 +++
>  libavcodec/h264dec.c | 5 +++++
>  libavutil/frame.h    | 1 +
>  libavutil/version.h  | 2 +-

You should split the introduction of the API (libavutil and doc) and
the use of the API (libavcodec) into two separate patches.

> +    if( (ret < 0 || h->slice_ctx->er.error_occurred) && h->cur_pic_ptr){

Please fix the bracket/whitespace style:
https://ffmpeg.org/developer.html#Code-formatting-conventions

->
    if ((ret < 0 || h->slice_ctx->er.error_occurred) && h->cur_pic_ptr) {

Moritz
_______________________________________________
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