| Other than the following comments, I think the spec looks great.
|
| 1) It doesn't make sense for glXGetProcAddress to return GLU entrypoints.
| glX doesn't even know or care that GLU exists, so it certainly shouldn't
| be returning pointers into GLU. If there's an actual need to get
| pointers to GLU extensions, we should define a gluGetProcAddress. I
| propose that we don't.
I agree.
|
| 2) I don't see any reason why 1.0 core functions shouldn't be queryable
| (next to the last issue).
I agree.
|
| 3) A NULL return value should not indicate anything except that something
| is wrong (and glError should be checked in that case to find out what
| went wrong). NULL certainly shouldn't give any indication as to whether
| the queried extension exists or is valid.
if its a glX function it shouldn't raise a gl error. It could raise
a glX error, if we went and added a new token or hijacked an existing
one (BadImplementation?).
|
| 4) I don't think it's necessary to specify that "glXGetProcAddress(foo) ==
| &foo". In fact, I think it's an implementation detail that may not be
| true in all implementations, so it's much better not to constrain the
| implementation by mentioning it.
I could go either way on this. I think the spec should say something about
it, like either it does == or that applications shouldn't rely on it being ==