On Mon, Sep 16, 2013 at 10:06:13PM +0300, Martin Storsjö wrote:
> --- a/libavcodec/svq3.c
> +++ b/libavcodec/svq3.c
> @@ -652,9 +652,9 @@ static int svq3_decode_mb(SVQ3Context *s, unsigned int
> mb_type)
>
> - if ((h->intra16x16_pred_mode = ff_h264_check_intra_pred_mode(h, dir,
> 0)) == -1) {
> - av_log(h->avctx, AV_LOG_ERROR, "check_intra_pred_mode = -1\n");
> - return -1;
> + if ((h->intra16x16_pred_mode = ff_h264_check_intra_pred_mode(h, dir,
> 0)) < 0) {
> + av_log(h->avctx, AV_LOG_ERROR, "check_intra_pred_mode < 0\n");
> + return h->intra16x16_pred_mode;
LGTM, but maybe adjust the name of the function in the av_log() call.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel