Weird... our unit test calls this method and don't crash on Windows.

Farsmo, did you compile the bindings yourself or use Roman's packages?

In any case, since this is a crasher issue and a regression (I take?), I'm
prioritizing this P1.

It's not a regression because that example never worked on Windows.

The reason for the crash is that you apparently can't call bindTexture() inside
a glNewList()...glEndList() block.

Qt should definitely detect this if at all possible.


--- a/examples/opengl/textures/textures.py
+++ b/examples/opengl/textures/textures.py
@@ -89,6 +89,7 @@ class GLWidget(QtOpenGL.QGLWidget):

     def initializeGL(self):
         if not GLWidget.sharedObject:
+            self.textures =
[self.bindTexture(QtGui.QPixmap(":/images/side%d.png" % (i + 1))) for i in
range(6)]
             GLWidget.sharedObject = self.makeObject()
         GLWidget.refCount += 1

@@ -137,7 +138,7 @@ class GLWidget(QtOpenGL.QGLWidget):
         glNewList(dlist, GL_COMPILE)

         for i in range(6):
-            self.bindTexture(QtGui.QPixmap(":/images/side%d.png" % (i + 1)))
+            glBindTexture(GL_TEXTURE_2D, self.textures[i])

             glBegin(GL_QUADS)
             for j in range(4):

I reported http://bugreports.qt.nokia.com/browse/QTBUG-17011

I don't think Qt should detect this, this is an OpenGL error, if you have a
OpenGL function call in the wrong place is your fault not Qt fault, besides
OpenGL is another library.

The example works on Linux, maybe due to different OpenGL implementations, btw
if you could provide a patch to the example it would be nice :-)

Oops, I did wait too much to replay after read and forgot that there's a patch,
duh! sorry.

Can you check if after the patch bellow the patch still crashing on Windows?

http://qt.gitorious.org/pyside/pyside-examples/commit/c3752cee9ca8648d49e29e5cd519d158abd1d262

Works fine.

OK, since it wasn't a regression, I dropped it back to P2 (although following
the discussion, this should probably be RESOLVED FIXED, right?)...

released on beta 5