old problem with QGLContext and render pixmap

[mickey]

sorry, I don't understand good but how copy it on the stack; thanks

[jacek]

I don't understand good but how copy it on the stack

If you have something like:

void GlWidget::initializeGL()
{
   ...
   _textureId = generateTexture();
   ...
}

void GlWidget::initializeGL()
{
   ...
   _textureId = generateTexture();
   ...
}

To copy to clipboard, switch view to plain text mode 

then each time you invoke initializeGL() you will loose the previous value of _textureId member variable.

You can avoid that like this:

void GlWidget::renderPixmap(...)
{
   int oldTextureId = _textureId;   
   QPixmap result( QGLWidget::renderPixmap( ... ) );
   _textureId = oldTextureId;
   return result;
}

void GlWidget::renderPixmap(...)
{
   int oldTextureId = _textureId;   
   QPixmap result( QGLWidget::renderPixmap( ... ) );
   _textureId = oldTextureId;
   return result;
}

To copy to clipboard, switch view to plain text mode 

[mickey]

sorry but I can't understand what's the textureID; but after I call genPixmap (now I put It inside myWidget class and don't change everythings), I call updateGL(),and everythings appear as before; then I think my way to put texture is ok.; i think is only a context problem....could you explain better what do you want to do with saving this texture ids?? thanks

[jacek]

could you explain better what do you want to do with saving this texture ids??

Could you post your initializeGL()? I might not remember your program well.

[mickey]

take it. thanks

[jacek]

What does plain.initGL() do?

[mickey]

it initialize every textures....gen tex, bind....plain it's an object and onto this I put the texture...

[jacek]

it initialize every textures....gen tex, bind....

That's the place where the problem is. Can I see it?

[mickey]

it's very complicated. Can I mail it you? thanks

[jacek]

Can I mail it you?

Yes, you can.

[jacek]

Here's the problematic part:

void Plain::initGL() {
        ...
        glGenTextures(dimX*dimZ, tex);
        glGenTextures(1, &lastTex);
        ...
        glBindTexture(GL_TEXTURE_2D, lastTex );
        ...
        glBindTexture(GL_TEXTURE_2D, tex[contite]);
        ...
}

void Plain::initGL() {
        ...
        glGenTextures(dimX*dimZ, tex);
        glGenTextures(1, &lastTex);
        ...
        glBindTexture(GL_TEXTURE_2D, lastTex );
        ...
        glBindTexture(GL_TEXTURE_2D, tex[contite]);
        ...
}

To copy to clipboard, switch view to plain text mode 

Every time you invoke Plain::initGL() new textures are generated. The images are stored inside OpenGL context (represented by QGLContex) and your program receives only ID numbers of those textures (that's what you store in tex and lastTex).

Now if you invoke initGL() for the second time (within the same context), tex and lastTex will be filled with new values --- the old textures will be still in memory, but you won't be able to access them, since those variables will hold IDs of the new textures. It's not a big problem (except for the resource leak).

But what happens when you invoke initGL() within a different context (just like it happens during renderPixmap())? initGL() will generate new textures, that will be stored in a new context, and it will store their IDs in tex and lastTex (the same ones, since you have only one Plain object). Now when renderPixmap() ends, it destroys that temporary context and the new textures are destroyed too, but tex and lastTex still hold their IDs. The problem is that when you will try to draw something later, you will use invalid texture IDs.

You can solve this problem like this:

void GlWidget::renderPixmap(...)
{
   plain.store();
   QPixmap result( QGLWidget::renderPixmap( ... ) );
   plain.restore();
   return result;
}
...
void Plain::store()
{
   // copy tex and lastTex values somewhere
   // for example onto QValueStack
}
 
void Plain::restore()
{
   // restore tex and lastTex values
}

void GlWidget::renderPixmap(...)
{
   plain.store();
   QPixmap result( QGLWidget::renderPixmap( ... ) );
   plain.restore();
   return result;
}
...
void Plain::store()
{
   // copy tex and lastTex values somewhere
   // for example onto QValueStack
}

void Plain::restore()
{
   // restore tex and lastTex values
}

To copy to clipboard, switch view to plain text mode 

[mickey]

I'm thinking to your solution...and other troubles arise: is there a way to clean the contextGL? (in the same way when app started)
then, I see that renderPixmap render my scene ok but don't render right texture multitextured; is it right? thanks