On Sun, 27 Jul 2014 15:54:39 +0200, Christophe Gisquet 
<[email protected]> wrote:
> 2014-07-27 13:22 GMT+02:00 Christophe Gisquet <[email protected]>:
> > 2014-07-27 13:00 GMT+02:00 Anton Khirnov <[email protected]>:
> >> They are be replaced by passing additional parameters to the transform
> >> functions.
> >
> > Probably ok. I'd thought they might be used in inloop filtering or
> > other tools, but they aren't.
> 
> Actually this may not compatible with range extension:
> https://github.com/OpenHEVC/FFmpeg/blob/rext/libavcodec/hevc.c#L1060
> 

>From a quick look at the spec, I think we can just add two more variables for
the 422 case, there's still no need to store those whole two arrays.

Note that their combined size is something like 160kb. In my quick tests, this
patch makes the decoding 1-2% faster on a random sample, which is rather nice.

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

Reply via email to