[x265] [PATCH] check_IPFilterChroma_primitive, stride made equal to min width 2, fix for 2XN block

[Steve Borho]

On Wed, Oct 16, 2013 at 10:40 AM, <praveen at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Praveen Tiwari
> # Date 1381938013 -19800
> # Node ID 5914800260d95a9bac7ce3eec2291e5c781e8422
> # Parent  86f1dcc53a1a101b7acd169d608b1d089efeb888
> check_IPFilterChroma_primitive, stride made equal to min width 2, fix for
> 2XN block
>

Queued, but you'll need to explain to me why this is a necessary or an
improvement.


>
> diff -r 86f1dcc53a1a -r 5914800260d9 source/test/ipfilterharness.cpp
> --- a/source/test/ipfilterharness.cpp   Wed Oct 16 20:30:59 2013 +0530
> +++ b/source/test/ipfilterharness.cpp   Wed Oct 16 21:10:13 2013 +0530
> @@ -270,31 +270,26 @@
>
>  bool IPFilterHarness::check_IPFilterChroma_primitive(filter_pp_t ref,
> filter_pp_t opt)
>  {
> -    int rand_srcStride, rand_dstStride, rand_coeffIdx;
> +    int srcStride, dstStride, rand_coeffIdx;
>
>      for (int i = 0; i <= 100; i++)
>      {
>
>          rand_coeffIdx = rand() % 8;                // Random coeffIdex in
> the filter
> -        rand_srcStride = rand() % 100;              // Randomly generated
> srcStride
> -        rand_dstStride = rand() % 100;              // Randomly generated
> dstStride
>
> -        if (rand_srcStride < 32)
> -            rand_srcStride = 32;
> +         srcStride = 2;
> +         dstStride = 2;
>
> -        if (rand_dstStride < 32)
> -            rand_dstStride = 32;
> -
> -        opt(pixel_buff + 3 * rand_srcStride,
> -            rand_srcStride,
> +        opt(pixel_buff + 3 * srcStride,
> +            srcStride,
>              IPF_vec_output_p,
> -            rand_dstStride,
> +            dstStride,
>              rand_coeffIdx
>              );
> -        ref(pixel_buff + 3 * rand_srcStride,
> -            rand_srcStride,
> +        ref(pixel_buff + 3 * srcStride,
> +            srcStride,
>              IPF_C_output_p,
> -            rand_dstStride,
> +            dstStride,
>              rand_coeffIdx
>              );
>
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>



-- 
Steve Borho
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20131016/186be6c0/attachment.html>