Hi,
sure thing.
I lost a little bit track of things, but if you have the typos under
control, than we just need to remove the warnings.
Maybe we shouldn't touch the auxiliary file at all, as glimreadGray is ugly
and just slipped into the pull request by accident and we should first
discuss the structure of the file in general.
There should be an auxiliary file for modern OpenGL at some point.
By now, I've a nice collection of types and functions to read in shaders,
textures, render text and the like, which should definitely end up in the
OpenGL package at some point.
Also, there should be just one file for all versions, or at most two files
for OpenGL versions > 3.0 and < 3.0.
I could just make a new pull request with the structure I have in mind and
you can tell me what you think of it.

Best,
Simon


2014-03-02 18:51 GMT+01:00 Robert J. Ennis <notifications@github.com>:

> In src/gl43/gl43aux.jl:
>
> > @@ -1,17 +1,14 @@
> >  module OpenGL
> >
> > -import OpenGLStd.glVertex2i, OpenGLStd.glVertex3i, OpenGLStd.glVertex2d, OpenGLStd.glVertex3d, OpenGLStd.glVertex4d,
> > -       OpenGLStd.glTexCoord2i, OpenGLStd.glTexCoord3i, OpenGLStd.glTexCoord4i, OpenGLStd.glTexCoord2d, OpenGLStd.glTexCoord3d,
> > -       OpenGLStd.glTexCoord4d, OpenGLStd.glNormal3d, OpenGLStd.glColor3f, OpenGLStd.glColor4f, OpenGLStd.glColor3b, OpenGLStd.glColor4b,
> > -       OpenGLStd.glScaled, OpenGLStd.glTranslated, OpenGLStd.glRotated, OpenGLStd.glEnable, OpenGLStd.glDisable, OpenGLStd.glBegin,
> > -       OpenGLStd.glEnd, OpenGLStd.glPushMatrix, OpenGLStd.glLoadIdentity
> > +using Images
> > +import OpenGLStd.glEnable, OpenGLStd.glDisable
> >
>
> Actually, for the 'using Images' line, it's sufficient to have 'import
> Images.imread', since that's the only function we need.
>
> I should also say that I like the idea of supporting grayscale images.
> Wondering if we can have a glimread function that will run the correct
> incantations depending on the image data that is returned by imread(), so
> that users don't have to choose from a list of image reading functions.
>
> Best,
> Rob
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/rennis250/OpenGL.jl/pull/9/files#r10196459>
> .
>