Hi,

On Mon, Dec 17, 2012 at 1:34 PM, Janne Grunau <[email protected]> wrote:
> -        if (h != h0 || (HAVE_THREADS && h->s.avctx->active_thread_type & 
> FF_THREAD_FRAME)) {
> -            av_log_missing_feature(s->avctx,
> -                                   "Width/height changing with threads", 0);
> +
> +        if (h != h0) {
> +            av_log_missing_feature(s->avctx, "Width/height changing with 
> slice-"
> +                                   "based threading is", 0);
>              return AVERROR_PATCHWELCOME;   // width / height changed during 
> parallelized decoding
>          }

Uhm, no. If size changed from slice-to-slice in the same frame, we're
horrendously broken and should probably error out. This is certainly
not intended or supported. av_log_missing_feature() is wrong here,
just av_log(AV_LOG_ERROR).

Rest is OK.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to