[x265] [PATCH] fix : square chroma transform expected error message

[Deepthi Nandakumar]

Can we have more detail on this, Ashok? Both trWidthC, trHeightC and widthC
and heightC should have the same values. Why exactly was this assert being
triggered in the regression tests?


On Mon, May 19, 2014 at 7:19 PM, <ashok at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Ashok Kumar Mishra<ashok at multicorewareinc.com>
> # Date 1400507347 -19800
> #      Mon May 19 19:19:07 2014 +0530
> # Node ID 8647c7861144eee4a0f96687794607b3e98d7b9f
> # Parent  ba2a9f61ea06f0ac799d8c0247eec770065465bb
> fix :  square chroma transform expected error message
>
> diff -r ba2a9f61ea06 -r 8647c7861144 source/Lib/TLibEncoder/TEncSearch.cpp
> --- a/source/Lib/TLibEncoder/TEncSearch.cpp     Fri May 16 19:20:46 2014
> +0900
> +++ b/source/Lib/TLibEncoder/TEncSearch.cpp     Mon May 19 19:19:07 2014
> +0530
> @@ -2975,7 +2975,7 @@
>                  else
>                  {
>                      int16_t *ptr = resiYuv->getCbAddr(absTUPartIdxC);
> -                    X265_CHECK(trWidthC == trHeightC, "square chroma
> transform expected\n");
> +                    X265_CHECK(widthC == heightC, "square chroma
> transform expected\n");
>
>  primitives.blockfill_s[(int)g_convertToBit[trWidthC]](ptr,
> resiYuv->m_cwidth, 0);
>                  }
>                  if (absSumV)
> @@ -2991,7 +2991,7 @@
>                  else
>                  {
>                      int16_t *ptr =  resiYuv->getCrAddr(absTUPartIdxC);
> -                    X265_CHECK(trWidthC == trHeightC, "square chroma
> transform expected\n");
> +                    X265_CHECK(widthC == heightC, "square chroma
> transform expected\n");
>
>  primitives.blockfill_s[(int)g_convertToBit[trWidthC]](ptr,
> resiYuv->m_cwidth, 0);
>                  }
>                  cu->setCbfPartRange(absSumU ? setCbf : 0, TEXT_CHROMA_U,
> absTUPartIdxC, tuIterator.m_absPartIdxStep);
> @@ -3348,7 +3348,7 @@
>                  {
>                      int16_t *ptr =
> m_qtTempShortYuv[qtlayer].getCbAddr(tuIterator.m_absPartIdxTURelCU);
>                      const uint32_t stride =
> m_qtTempShortYuv[qtlayer].m_cwidth;
> -                    X265_CHECK(trWidthC == trHeightC, "square chroma
> transform expected\n");
> +                    X265_CHECK(widthC == heightC, "square chroma
> transform expected\n");
>
>  primitives.blockfill_s[(int)g_convertToBit[widthC]](ptr, stride, 0);
>                  }
>
> @@ -3416,7 +3416,7 @@
>                  {
>                      int16_t *ptr =
>  m_qtTempShortYuv[qtlayer].getCrAddr(tuIterator.m_absPartIdxTURelCU);
>                      const uint32_t stride =
> m_qtTempShortYuv[qtlayer].m_cwidth;
> -                    X265_CHECK(trWidthC == trHeightC, "square chroma
> transform expected\n");
> +                    X265_CHECK(widthC == heightC, "square chroma
> transform expected\n");
>
>  primitives.blockfill_s[(int)g_convertToBit[widthC]](ptr, stride, 0);
>                  }
>
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20140519/72f98aa0/attachment-0001.html>