On 05/07/16 11:58, Martin Storsjö wrote:
> On Fri, 1 Jul 2016, Martin Storsjö wrote:
> 
>> Index 0 is 16x16, 1 is 8x8, 2 is 4x4.
>> ---
>> libavcodec/vp8dsp.h | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavcodec/vp8dsp.h b/libavcodec/vp8dsp.h
>> index 4864cf7..b8a30f9 100644
>> --- a/libavcodec/vp8dsp.h
>> +++ b/libavcodec/vp8dsp.h
>> @@ -70,12 +70,12 @@ typedef struct VP8DSPContext {
>>     void (*vp8_h_loop_filter_simple)(uint8_t *dst, ptrdiff_t stride,
>> int flim);
>>
>>     /**
>> -     * first dimension: width>>3, height is assumed equal to width
>> +     * first dimension: 2-(width>>3), height is assumed equal to width
> 
> FWIW; 2-(width>>3) might give the right values, but isn't technically
> quite right either. The right expression would be 4-log2(width) (or
> 4-av_ctz(width)) - should we go with that instead?
> 

Sure.

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

Reply via email to