Re: [Mesa-dev] [PATCH 01/10] util: fix new gcc6 warnings

2016-02-18 Thread Ian Romanick
This patch is Reviewed-by: Ian Romanick On 02/17/2016 09:34 AM, Rob Clark wrote: > src/util/hash_table.h:111:23: warning: ‘_mesa_fnv32_1a_offset_bias’ defined > but not used [-Wunused-const-variable] > static const uint32_t _mesa_fnv32_1a_offset_bias = 2166136261u; >

[Mesa-dev] [PATCH 01/10] util: fix new gcc6 warnings

2016-02-17 Thread Rob Clark
src/util/hash_table.h:111:23: warning: ‘_mesa_fnv32_1a_offset_bias’ defined but not used [-Wunused-const-variable] static const uint32_t _mesa_fnv32_1a_offset_bias = 2166136261u; ^~ I went w/ enum, rather than #define, mostly because a lowercase

Re: [Mesa-dev] [PATCH 01/10] util: fix new gcc6 warnings

2016-02-16 Thread Matt Turner
On Tue, Feb 16, 2016 at 11:37 AM, Ian Romanick wrote: > On 02/16/2016 10:57 AM, Rob Clark wrote: >> src/util/hash_table.h:111:23: warning: ‘_mesa_fnv32_1a_offset_bias’ defined >> but not used [-Wunused-const-variable] >> static const uint32_t _mesa_fnv32_1a_offset_bias =

Re: [Mesa-dev] [PATCH 01/10] util: fix new gcc6 warnings

2016-02-16 Thread Rob Clark
On Tue, Feb 16, 2016 at 2:37 PM, Ian Romanick wrote: > On 02/16/2016 10:57 AM, Rob Clark wrote: >> src/util/hash_table.h:111:23: warning: ‘_mesa_fnv32_1a_offset_bias’ defined >> but not used [-Wunused-const-variable] >> static const uint32_t _mesa_fnv32_1a_offset_bias =

Re: [Mesa-dev] [PATCH 01/10] util: fix new gcc6 warnings

2016-02-16 Thread Ian Romanick
On 02/16/2016 10:57 AM, Rob Clark wrote: > src/util/hash_table.h:111:23: warning: ‘_mesa_fnv32_1a_offset_bias’ defined > but not used [-Wunused-const-variable] > static const uint32_t _mesa_fnv32_1a_offset_bias = 2166136261u; >^~ > >

[Mesa-dev] [PATCH 01/10] util: fix new gcc6 warnings

2016-02-16 Thread Rob Clark
src/util/hash_table.h:111:23: warning: ‘_mesa_fnv32_1a_offset_bias’ defined but not used [-Wunused-const-variable] static const uint32_t _mesa_fnv32_1a_offset_bias = 2166136261u; ^~ Signed-off-by: Rob Clark ---