[PATCH] wined3d: Use GL_ARB_texture_mirror_clamp_to_edge when available.

Henri Verbeet hverbeet at gmail.com
Mon Oct 28 05:21:40 CDT 2013


On 26 October 2013 22:23, Rico Schüller <kgbricola at web.de> wrote:
> Well, I did a small update for the script (it may need some
> improvements...). Please see the attached patch for make_opengl. I have
> several questions to this:

I didn't review the code very closely, but at first sight it looks
like you have a couple of unrelated and whitespace only changes that
should probably be in separate patches. I think there are also e.g.
some comments that should be updated / removed. I can try to answer
some of the questions, but ultimately it's up to Alexandre.

> 1. Is it the correct way to just change the script or do we prefer
> another way?
Seems fine to me. Although it's a bit annoying that Khronos doesn't
seem to want to make any guarantees about the stability of the XML
format.

> 2. Is it fine to use XML::Simple as a dependency?
I think so, but I also think it's common to just put things like "use
XML::Simple;" at the top of the file.

> 3. Do we need to support gles functions and enums?
Not initially, but we may want to in the future.

> 4. Why was $pseudo_to_opengl used in the script?
For mapping type. E.g. in gl.spec, ClearColor() takes 4 parameters of
"ColorF", which is mapped to GLfloat in gl.tm. In the XML files that
information is available as the "group" attribute, but we don't need
it for our purposes. (See also
https://cvs.khronos.org/svn/repos/ogl/trunk/doc/registry/public/api/readme.pdf)



More information about the wine-devel mailing list