Re: [Mesa-dev] [PATCH 2/5] glsl: Get rid of hardcoded arrays of shader target names.

2013-12-17 Thread Kenneth Graunke
On 12/17/2013 11:50 AM, Brian Paul wrote: On 12/17/2013 12:28 PM, Paul Berry wrote: [snip] How about this idea instead: rather than use function overloading to distinguish between the two meanings of _mesa_glsl_shader_target_name(), have two functions with separate names, e.g.: const char

Re: [Mesa-dev] [PATCH 2/5] glsl: Get rid of hardcoded arrays of shader target names.

2013-12-17 Thread Paul Berry
On 17 December 2013 10:51, Brian Paul bri...@vmware.com wrote: On 12/17/2013 11:23 AM, Paul Berry wrote: We already have a function for converting a shader type index to a string: _mesa_glsl_shader_target_name(). --- src/glsl/link_atomics.cpp | 8 +++- src/glsl/linker.cpp |

Re: [Mesa-dev] [PATCH 2/5] glsl: Get rid of hardcoded arrays of shader target names.

2013-12-17 Thread Brian Paul
On 12/17/2013 12:28 PM, Paul Berry wrote: On 17 December 2013 10:51, Brian Paul bri...@vmware.com mailto:bri...@vmware.com wrote: On 12/17/2013 11:23 AM, Paul Berry wrote: We already have a function for converting a shader type index to a string:

Re: [Mesa-dev] [PATCH 2/5] glsl: Get rid of hardcoded arrays of shader target names.

2013-12-17 Thread Brian Paul
On 12/17/2013 11:23 AM, Paul Berry wrote: We already have a function for converting a shader type index to a string: _mesa_glsl_shader_target_name(). --- src/glsl/link_atomics.cpp | 8 +++- src/glsl/linker.cpp | 16 ++-- 2 files changed, 9 insertions(+), 15

[Mesa-dev] [PATCH 2/5] glsl: Get rid of hardcoded arrays of shader target names.

2013-12-17 Thread Paul Berry
We already have a function for converting a shader type index to a string: _mesa_glsl_shader_target_name(). --- src/glsl/link_atomics.cpp | 8 +++- src/glsl/linker.cpp | 16 ++-- 2 files changed, 9 insertions(+), 15 deletions(-) diff --git a/src/glsl/link_atomics.cpp