On 24/08/15 14:26, Arttu Ylä-Outinen wrote: >>>> + av_assert0(frame->width == ctx->config->width); >>>> + av_assert0(frame->height == ctx->config->height); >>>> + av_assert0(frame->format == avctx->pix_fmt); >>> This may happen normally, do not crash this way, return a clear message >>> stating that it is not currently supported.
> I'll fix that. When would the dimensions of the input video actually > change in the middle? I tried using images of different sizes as input > to avconv but it didn't trigger those asserts. avconv rescales on the fly, a normal user would not. Probably something more structured (e.g. a capability) could be done about it but is unrelated to this patch. > It looks like av_copy_image could be used for copying the pixels. Should > I change this to use that function or is it better to wait until Kvazaar > has the required changes for avoiding the copy entirely? Depends on when you want this integration to land =) If the change happens soon enough I'd use it, kvazaar development speed is quite fast so I'm not really concerned about users using an older version right now. Thanks a lot =) lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
