[x265] [PATCH] corrected chroma partion for blockcopy_pp function

[Steve Borho]

On Tue, Nov 5, 2013 at 4:40 AM, <praveen at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Praveen Tiwari
> # Date 1383647996 -19800
> # Node ID 158f6c0760cb595f6c5a4f0b44aa48f78e76c2df
> # Parent  b5bdc10daa02d7d769263c5adb63fe3026ea4ba0
> corrected chroma partion for blockcopy_pp function
>

I liked it better the old way


> diff -r b5bdc10daa02 -r 158f6c0760cb source/test/pixelharness.cpp
> --- a/source/test/pixelharness.cpp      Tue Nov 05 15:28:56 2013 +0530
> +++ b/source/test/pixelharness.cpp      Tue Nov 05 16:09:56 2013 +0530
> @@ -670,15 +670,6 @@
>          }
>      }
>
> -    if (opt.chroma_copy_pp[part])
> -    {
> -        if (!check_block_copy_pp(ref.chroma_copy_pp[part],
> opt.chroma_copy_pp[part]))
> -        {
> -            printf("chroma_copy_pp[%s] failed\n", chromaPartStr[part]);
> -            return false;
> -        }
> -    }
> -
>      if (opt.luma_copy_sp[part])
>      {
>          if (!check_block_copy_sp(ref.luma_copy_sp[part],
> opt.luma_copy_sp[part]))
> @@ -854,7 +845,19 @@
>              return false;
>          }
>      }
> -    return true;
> +
> +    for (int i = 0; i < NUM_CHROMA_PARTITIONS; i++)
> +    {
> +      if (opt.chroma_copy_pp[i])
> +      {
> +          if (!check_block_copy_pp(ref.chroma_copy_pp[i],
> opt.chroma_copy_pp[i]))
> +          {
> +              printf("chroma_copy_pp[%s] failed\n", chromaPartStr[i]);
> +              return false;
> +          }
> +      }
> +    }
> +     return true;
>  }
>
>  void PixelHarness::measurePartition(int part, const EncoderPrimitives&
> ref, const EncoderPrimitives& opt)
> @@ -922,12 +925,6 @@
>          REPORT_SPEEDUP(opt.luma_copy_pp[part], ref.luma_copy_pp[part],
> pbuf1, 64, pbuf2, 128);
>      }
>
> -    if (opt.chroma_copy_pp[part])
> -    {
> -        printf("ccpy_pp[%s]", chromaPartStr[part]);
> -        REPORT_SPEEDUP(opt.chroma_copy_pp[part],
> ref.chroma_copy_pp[part], pbuf1, 64, pbuf2, 128);
> -    }
> -
>      if (opt.luma_copy_sp[part])
>      {
>          printf("lcpy_sp[%s]", lumaPartStr[part]);
> @@ -1055,4 +1052,13 @@
>          printf("downscale");
>          REPORT_SPEEDUP(opt.frame_init_lowres_core,
> ref.frame_init_lowres_core, pbuf2, pbuf1, pbuf2, pbuf3, pbuf4, 64, 64, 64,
> 64);
>      }
> +
> +    for (int i = 0; i < NUM_CHROMA_PARTITIONS; i++)
> +    {
> +        if (opt.chroma_copy_pp[i])
> +        {
> +            printf("ccpy_pp[%s]", chromaPartStr[i]);
> +            REPORT_SPEEDUP(opt.chroma_copy_pp[i], ref.chroma_copy_pp[i],
> pbuf1, 64, pbuf2, 128);
> +        }
> +    }
>  }
> _______________________________________________
> 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/20131105/adf64be4/attachment.html>