The type hints for pointers (e.g., POINTER(GLenum)) might not be correct.

Thanks for this PR! This is helpful for users.

However, one thing I would do is instead of making this a double function call (function in a function), which has a runtime overhead cost, there are a couple solutions that won't affect runtime:

1. glClearColor: Callable[[None], GLfloat | float, GLfloat | float, GLfloat | float, GLfloat | float] = _link_function('glClearColor', None, [GLfloat, GLfloat, GLfloat, GLfloat], requires='OpenGL 1.0')
   While this works, it doesn't allow any argument names.

2. Stub file, (in this case) pyglet/gl/gl.pyi
   You would define them like this:

def glClearColor(red: GLfloat | float, green: GLfloat | float, blue: GLfloat | float, alpha: GLfloat | float) -> None:
    ...

I think the stub file is more clean in my opinion, and IDE's will lookup the stub file signature instead .

I think adding the type annotations in a stub file is a good idea.

This looks good, thanks @Ross-TheBoss!

Question though: is it necessary to also include all of the GL constants in the stub files?

This looks good, thanks @Ross-TheBoss!

Question though: is it necessary to also include all of the GL constants in the stub files?

Stub files override all inspections on my IDE, if you don't have them, it will act as if the variables don't exist.

This looks good, thanks @Ross-TheBoss!

Question though: is it necessary to also include all of the GL constants in the stub files?

Including the GL constants in the stub files is necessary but I currently also define the constants' values which is not necessary. Originally, I liked that I could see the value assigned to a constant in Pycharm's tooltip, but the convention seems to be not to include this information in the stub file.

OK that explains it. I assumed there was a reason to include them.

One last thing I just noticed: shouldn't we alias GLbyte to c_byte instead of c_char?

GLbyte = c_char
...
GLchar = c_char

ben@hammer ~ $ cat /usr/include/GL/gl.h | grep GLbyte
typedef signed char	GLbyte;		/* 1-byte signed */

GLbyte

I didn't notice that as I wasn't making changes to that part of the code. We should probably alias GLbyte to c_byte instead as that's more specific and less implementation dependent. Using c_char works but only coincidentally as whether a char is signed or unsigned depends on the platform. c_byte however is a signed char in C.

GLbyte is supposed to be specifically a signed 8-bit integer: OpenGL types

OK. I have made that change now.

I guess GLbyte is not commonly used in OpenGL to begin with, which is probably why nobody noticed 😅