Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AppleGLGetProcAddress: 820 warnings #22

Closed
miguelmartin75 opened this issue Oct 26, 2013 · 1 comment
Closed

AppleGLGetProcAddress: 820 warnings #22

miguelmartin75 opened this issue Oct 26, 2013 · 1 comment

Comments

@miguelmartin75
Copy link
Contributor

There are 820 warnings for Mac builds due to the AppleGLGetProcAddress's signature. Here is a sample of the warnings:

/Users/miguel/Documents/Programming/Projects/Repos/magnum/external/OpenGL/GL/gl_magnum.c:2711:86: warning:
  passing 'char [12]' to parameter of type 'const GLubyte *' (aka
  'const unsigned char *') converts between pointers to integer types with
  different sign [-Wpointer-sign]
...GLubyte * (CODEGEN_FUNCPTR *)(GLenum))IntGetProcAddress("glGetString");
                                                         ^~~~~~~~~~~~~
/Users/miguel/Documents/Programming/Projects/Repos/magnum/external/OpenGL/GL/gl_magnum.c:86:57: note:
  expanded from macro 'IntGetProcAddress'
            #define IntGetProcAddress(name) AppleGLGetProcAddress(name)
                                                                  ^
/Users/miguel/Documents/Programming/Projects/Repos/magnum/external/OpenGL/GL/gl_magnum.c:9:52: note:
  passing argument to parameter 'name' here
static void* AppleGLGetProcAddress (const GLubyte *name)
@mosra
Copy link
Owner

mosra commented Oct 26, 2013

Thanks for the detailed report and pull request. This issue is actually related to glLoadGen, which was used to generate gl_magnum.c file. I found a possible solution at https://bitbucket.org/alfonse/glloadgen/issue/29/compilation-errors-on-mac-os-x and added the change in ebe691e. It fixes this issue and should also fix #21, hopefully it won't cause any regressions. Please test and report any problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants