On Thu, Oct 17, 2013 at 5:35 AM, <[email protected]> wrote:

> # HG changeset patch
> # User Praveen Tiwari
> # Date 1382006137 -19800
> # Node ID dfae391107c3d2823adabe284f1c08278669d8f0
> # Parent  fc9dbd798ac37ec1acc0596aa179f0deb586c092
> removed unnecessary calculation form rand_srcStride
>
> diff -r fc9dbd798ac3 -r dfae391107c3 source/test/ipfilterharness.cpp
> --- a/source/test/ipfilterharness.cpp   Thu Oct 17 14:14:40 2013 +0530
> +++ b/source/test/ipfilterharness.cpp   Thu Oct 17 16:05:37 2013 +0530
> @@ -279,13 +279,13 @@
>          rand_srcStride = rand() % 100;              // Randomly generated
> srcStride
>          rand_dstStride = rand() % 100;              // Randomly generated
> dstStride
>
> -        opt(pixel_buff + 3 * rand_srcStride,
> +        opt(pixel_buff + rand_srcStride,
>              rand_srcStride,
>              IPF_vec_output_p,
>              rand_dstStride,
>              rand_coeffIdx
>              );
> -        ref(pixel_buff + 3 * rand_srcStride,
> +        ref(pixel_buff + rand_srcStride,
>              rand_srcStride,
>              IPF_C_output_p,
>              rand_dstStride,
>

This test function will be eventually used for both luma and chroma
vertical interpolation functions (they all have the same function
definition), so I think the 3 needs to stay there (perhaps it should be
better documented as the max vertical filter half-distance)

-- 
Steve Borho
_______________________________________________
x265-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/x265-devel

Reply via email to