On Thu, Nov 7, 2013 at 5:52 AM, Aarthi Thirumalai <
aar...@multicorewareinc.com> wrote:
> # HG changeset patch
> # User Aarthi Thirumalai
> # Date 1383825146 -19800
> # Thu Nov 07 17:22:26 2013 +0530
> # Node ID e87bef56a1b6ed9a7a20be7d7021c178c8ddc42a
> # Parent 0823d243b4e2f9eee6323a9750eeb811bccbad77
> tcompicyuv: add right boundary padding while applying bottom row padding.
>
> diff -r 0823d243b4e2 -r e87bef56a1b6 source/Lib/TLibCommon/TComPicYuv.cpp
> --- a/source/Lib/TLibCommon/TComPicYuv.cpp Thu Nov 07 16:46:57 2013
> +0530
> +++ b/source/Lib/TLibCommon/TComPicYuv.cpp Thu Nov 07 17:22:26 2013
> +0530
> @@ -395,13 +395,13 @@
>
Good catch, but above here is a line which assigned width back to
m_picWidth, which is the original padded (to 4) width, it needs to be
removed in this case.
>
> for (uint32_t i = 1; i <= pady; i++)
> {
> - memcpy(Y + i * getStride(), Y, width * sizeof(Pel));
> + memcpy(Y + i * getStride(), Y, (width + padx) *
> sizeof(Pel));
> }
>
> for (uint32_t j = 1; j <= pady >> m_vChromaShift; j++)
> {
> - memcpy(U + j * getCStride(), U, (width >> m_hChromaShift)
> * sizeof(Pel));
> - memcpy(V + j * getCStride(), V, (width >> m_hChromaShift)
> * sizeof(Pel));
> + memcpy(U + j * getCStride(), U, ((width + padx) >>
> m_hChromaShift) * sizeof(Pel));
> + memcpy(V + j * getCStride(), V, ((width + padx) >>
> m_hChromaShift) * sizeof(Pel));
> }
> }
> }
> @@ -461,13 +461,13 @@
>
> for (uint32_t i = 1; i <= pady; i++)
> {
> - memcpy(Y + i * getStride(), Y, width * sizeof(Pel));
> + memcpy(Y + i * getStride(), Y, (width + padx) *
> sizeof(Pel));
> }
>
> for (uint32_t j = 1; j <= pady >> m_vChromaShift; j++)
> {
> - memcpy(U + j * getCStride(), U, (width >> m_hChromaShift)
> * sizeof(Pel));
> - memcpy(V + j * getCStride(), V, (width >> m_hChromaShift)
> * sizeof(Pel));
> + memcpy(U + j * getCStride(), U, ((width + padx) >>
> m_hChromaShift) * sizeof(Pel));
> + memcpy(V + j * getCStride(), V, ((width + padx) >>
> m_hChromaShift) * sizeof(Pel));
> }
> }
> }
> @@ -525,13 +525,13 @@
>
> for (uint32_t i = 1; i <= pady; i++)
> {
> - memcpy(Y + i * getStride(), Y, width * sizeof(pixel));
> + memcpy(Y + i * getStride(), Y, (width + padx) *
> sizeof(pixel));
> }
>
> for (uint32_t j = 1; j <= pady >> m_vChromaShift; j++)
> {
> - memcpy(U + j * getCStride(), U, (width >> m_hChromaShift)
> * sizeof(pixel));
> - memcpy(V + j * getCStride(), V, (width >> m_hChromaShift)
> * sizeof(pixel));
> + memcpy(U + j * getCStride(), U, ((width + padx) >>
> m_hChromaShift) * sizeof(pixel));
> + memcpy(V + j * getCStride(), V, ((width + padx) >>
> m_hChromaShift) * sizeof(pixel));
> }
> }
> }
> _______________________________________________
> x265-devel mailing list
> x265-devel@videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>
--
Steve Borho
_______________________________________________
x265-devel mailing list
x265-devel@videolan.org
https://mailman.videolan.org/listinfo/x265-devel
