| 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 ==

Reply via email to