[x265] [PATCH] Modify TComSlice structure to support multiple color space formats
Steve Borho
steve at borho.org
Wed Jan 8 00:59:52 CET 2014
On Fri, Jan 3, 2014 at 7:09 AM, <ashok at multicorewareinc.com> wrote:
> # HG changeset patch
> # User ashok at multicorewareinc.com
> # Date 1388754532 -19800
> # Fri Jan 03 18:38:52 2014 +0530
> # Node ID d7f6034a4ead619cde6977033f908d879789b1d1
> # Parent d1f5cad1ac2f4394616633751f1b885178848b49
> Modify TComSlice structure to support multiple color space formats
>
> diff -r d1f5cad1ac2f -r d7f6034a4ead source/Lib/TLibCommon/TComSlice.cpp
> --- a/source/Lib/TLibCommon/TComSlice.cpp Fri Jan 03 18:37:35 2014
> +0530
> +++ b/source/Lib/TLibCommon/TComSlice.cpp Fri Jan 03 18:38:52 2014
> +0530
> @@ -1077,8 +1077,6 @@
> m_scalingListCoef[sizeId][listId] = new
> int[X265_MIN(MAX_MATRIX_COEF_NUM, (int)g_scalingListSize[sizeId])];
> }
> }
> -
> - m_scalingListCoef[SCALING_LIST_32x32][3] =
> m_scalingListCoef[SCALING_LIST_32x32][1]; // copy address for 32x32
> }
>
> /** destroy quantization matrix array
> diff -r d1f5cad1ac2f -r d7f6034a4ead source/Lib/TLibCommon/TComSlice.h
> --- a/source/Lib/TLibCommon/TComSlice.h Fri Jan 03 18:37:35 2014 +0530
> +++ b/source/Lib/TLibCommon/TComSlice.h Fri Jan 03 18:38:52 2014 +0530
> @@ -778,6 +778,7 @@
> int m_SPSId;
> int m_VPSId;
> int m_chromaFormatIdc;
> + bool m_colorPlaneFlag;
>
> uint32_t m_maxTLayers; // maximum number of temporal layers
>
> @@ -863,6 +864,10 @@
>
> void setChromaFormatIdc(int i) { m_chromaFormatIdc = i; }
>
> + void setSeparateColorPlaneFlag(bool c) { m_colorPlaneFlag = c;}
> +
> + bool getSeparateColorPlaneFlag() { return m_colorPlaneFlag;}
>
ugh, please don't add any more set/get methods, just make the variable
public and access it directly.
> +
> static int getWinUnitX(int chromaFormatIdc) { assert(chromaFormatIdc
> > 0 && chromaFormatIdc <= MAX_CHROMA_FORMAT_IDC); return
> g_winUnitX[chromaFormatIdc]; }
>
> static int getWinUnitY(int chromaFormatIdc) { assert(chromaFormatIdc
> > 0 && chromaFormatIdc <= MAX_CHROMA_FORMAT_IDC); return
> g_winUnitY[chromaFormatIdc]; }
> _______________________________________________
> 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/20140107/69161a0e/attachment.html>
More information about the x265-devel
mailing list