[x265] [PATCH] removed unnecessary calculation form rand_srcStride

Steve Borho steve at borho.org
Thu Oct 17 20:37:04 CEST 2013


On Thu, Oct 17, 2013 at 5:35 AM, <praveen at multicorewareinc.com> 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20131017/a0be6b73/attachment.html>


More information about the x265-devel mailing list