[Mesa-dev] [PATCH] mesa: fix broken glGet*(GL_POLYGON_MODE) query

Marek Olšák maraeo at gmail.com
Wed Jan 31 13:12:47 UTC 2018


Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Wed, Jan 31, 2018 at 3:35 AM, Brian Paul <brianp at vmware.com> wrote:
> This reverts part of the patch which introduced the GLenum16 change.
> Fixes a conform regression found by Roland.
>
> Fixes: f96a69f916aed405 ("mesa: replace GLenum with GLenum16 in
> common structures (v4)")
> ---
>  src/mesa/main/get_hash_params.py | 2 +-
>  src/mesa/main/mtypes.h           | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/main/get_hash_params.py b/src/mesa/main/get_hash_params.py
> index 7cd195c..b61b16b 100644
> --- a/src/mesa/main/get_hash_params.py
> +++ b/src/mesa/main/get_hash_params.py
> @@ -743,7 +743,7 @@ descriptor=[
>    [ "PIXEL_MAP_R_TO_R_SIZE", "CONTEXT_INT(PixelMaps.RtoR.Size), NO_EXTRA" ],
>    [ "PIXEL_MAP_S_TO_S_SIZE", "CONTEXT_INT(PixelMaps.StoS.Size), NO_EXTRA" ],
>    [ "POINT_SIZE_GRANULARITY", "CONTEXT_FLOAT(Const.PointSizeGranularity), NO_EXTRA" ],
> -  [ "POLYGON_MODE", "CONTEXT_ENUM16(Polygon.FrontMode), NO_EXTRA" ],
> +  [ "POLYGON_MODE", "CONTEXT_ENUM2(Polygon.FrontMode), NO_EXTRA" ],
>    [ "POLYGON_OFFSET_BIAS_EXT", "CONTEXT_FLOAT(Polygon.OffsetUnits), NO_EXTRA" ],
>    [ "POLYGON_OFFSET_POINT", "CONTEXT_BOOL(Polygon.OffsetPoint), NO_EXTRA" ],
>    [ "POLYGON_OFFSET_LINE", "CONTEXT_BOOL(Polygon.OffsetLine), NO_EXTRA" ],
> diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
> index 35fafa5..f6fa6f4 100644
> --- a/src/mesa/main/mtypes.h
> +++ b/src/mesa/main/mtypes.h
> @@ -843,8 +843,8 @@ struct gl_point_attrib
>  struct gl_polygon_attrib
>  {
>     GLenum16 FrontFace;         /**< Either GL_CW or GL_CCW */
> -   GLenum16 FrontMode;         /**< Either GL_POINT, GL_LINE or GL_FILL */
> -   GLenum16 BackMode;          /**< Either GL_POINT, GL_LINE or GL_FILL */
> +   GLenum FrontMode;           /**< Either GL_POINT, GL_LINE or GL_FILL */
> +   GLenum BackMode;            /**< Either GL_POINT, GL_LINE or GL_FILL */
>     GLboolean CullFlag;         /**< Culling on/off flag */
>     GLboolean SmoothFlag;       /**< True if GL_POLYGON_SMOOTH is enabled */
>     GLboolean StippleFlag;      /**< True if GL_POLYGON_STIPPLE is enabled */
> --
> 2.7.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list