Menu

#258 multiple const GLchar *const* errors

1.12.0
wont-fix
None
5
2015-01-24
2014-10-05
le-squeak
No

https://sourceforge.net/p/glew/bugs/257/

~~~
From 8bd09d59e059224c4b73af640c57b1db606c2899 Mon Sep 17 00:00:00 2001
From: lesqueak lesqueak@sf.net
Date: Sat, 4 Oct 2014 14:39:39 -0400
Subject: [PATCH] Fix const GLchar const string


auto/core/gl/GL_VERSION_1_4 | 2 +-
auto/core/gl/GL_VERSION_2_0 | 2 +-
auto/core/gl/GL_VERSION_3_0 | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/auto/core/gl/GL_VERSION_1_4 b/auto/core/gl/GL_VERSION_1_4
index 2c9ba46..080e6f6 100644
--- a/auto/core/gl/GL_VERSION_1_4
+++ b/auto/core/gl/GL_VERSION_1_4
@@ -48,7 +48,7 @@ http://www.opengl.org/documentation/specs/version1.4/glspec14.pdf
void glFogCoorddv (const GLdouble coord)
void glFogCoordPointer (GLenum type, GLsizei stride, const void
pointer)
void glMultiDrawArrays (GLenum mode, const GLint first, const GLsizei count, GLsizei drawcount)
- void glMultiDrawElements (GLenum mode, const GLsizei count, GLenum type, const void const indices, GLsizei drawcount)
+ void glMultiDrawElements (GLenum mode, const GLsizei
count, GLenum type, const void indices, GLsizei drawcount)
void glPointParameteri (GLenum pname, GLint param)
void glPointParameteriv (GLenum pname, const GLint params)
void glPointParameterf (GLenum pname, GLfloat param)
diff --git a/auto/core/gl/GL_VERSION_2_0 b/auto/core/gl/GL_VERSION_2_0
index f6479d0..2f52256 100644
--- a/auto/core/gl/GL_VERSION_2_0
+++ b/auto/core/gl/GL_VERSION_2_0
@@ -108,7 +108,7 @@ http://www.opengl.org/documentation/specs/version2.0/glspec20.pdf
void glGetProgramInfoLog (GLuint program, GLsizei bufSize, GLsizei
length, GLchar infoLog)
void glGetShaderiv (GLuint shader, GLenum pname, GLint
param)
void glGetShaderInfoLog (GLuint shader, GLsizei bufSize, GLsizei length, GLchar infoLog)
- void glShaderSource (GLuint shader, GLsizei count, const GLchar const string, const GLint* length)
+ void glShaderSource (GLuint shader, GLsizei count, const GLchar
string, const GLint length)
GLint glGetUniformLocation (GLuint program, const GLchar
name)
void glGetUniformfv (GLuint program, GLint location, GLfloat params)
void glGetUniformiv (GLuint program, GLint location, GLint
params)
diff --git a/auto/core/gl/GL_VERSION_3_0 b/auto/core/gl/GL_VERSION_3_0
index c87cdab..9e6bca0 100644
--- a/auto/core/gl/GL_VERSION_3_0
+++ b/auto/core/gl/GL_VERSION_3_0
@@ -113,7 +113,7 @@ https://www.opengl.org/registry/doc/glspec30.20080923.pdf
GLboolean glIsEnabledi (GLenum cap, GLuint index)
void glBeginTransformFeedback (GLenum primitiveMode)
void glEndTransformFeedback (void)
- void glTransformFeedbackVaryings (GLuint program, GLsizei count, const GLchar const varyings, GLenum bufferMode)
+ void glTransformFeedbackVaryings (GLuint program, GLsizei count, const GLchar ** varyings, GLenum bufferMode)
void glGetTransformFeedbackVarying (GLuint program, GLuint index, GLsizei bufSize, GLsizei * length, GLsizei * size, GLenum * type, GLchar * name)
void glClampColor (GLenum target, GLenum clamp)
void glBeginConditionalRender (GLuint id, GLenum mode)
--

~~~~~~

1 Attachments

Discussion

  • le-squeak

    le-squeak - 2014-10-05

    Sorry about the formatting in that ticket.

    The patch shows what is wrong:

     const GLchar *const* errors 
    

    should be:

     const GLchar ** errors 
    
     
  • Nigel Stewart

    Nigel Stewart - 2014-10-06
    • status: open --> wont-fix
    • assigned_to: Nigel Stewart
    • Priority: 1 --> 5
     
  • Nigel Stewart

    Nigel Stewart - 2014-10-06

    These did indeed change in GLEW 1.11.0 to reflect the upstream changes from opengl.org. GLEW aims to match glext.h (and others)

    See: https://www.opengl.org/registry/api/GL/glext.h

    $ grep glMultiDrawElements\  glext.h 
    GLAPI void APIENTRY glMultiDrawElements (GLenum mode, const GLsizei *count, GLenum type, const void *const*indices, GLsizei drawcount)
    
    $ grep glShaderSource\   glext.h 
    GLAPI void APIENTRY glShaderSource (GLuint shader, GLsizei count, const GLchar *const*string, const GLint *length);
    

    No my initial inclination is to close this issue as "not a bug", or perhaps "will not fix".