Hi Orson,
I am back on testing this openGL issues.
I implemented your context manager in my branch.
It looks good the implementation (still need a bit style clean up your source
code file, eg: gl_context_mgr.cpp line 86)
I didn't tried it properly on Windows yet.
I am debugging (on debug mode) it on Linux.
Running in debug mode, the only issue that remains is the one with the windows
is not visible when the context is set.
For instance, now every time I run pcbnew in debug more, just open and close it
will make an ASSERT while closing the GAL.
The windows is probably already gone at that time..
I also having some difficulties in the module editor.
I implemented on my branch a object->close(); feature so I can try early to
destroy the context, hopefully while the windows is still visible.
I found for example if I close the windows (of module editor) by pressing X it
will follow a path.. if I close is by "Cancel" it will follow other path..
So this get difficult to solve..
Anyway, some of this issues now are in the end to try to destroy the context
because using your lock feature, it is trying to set the context always.
(Before that was not true, or we implemented it different for Linux vs Windows)
I have one suggestion for your context manager.
In the lock, maybe we can try to set the context just if it changed.
>From our experience and from the wxwidget (opengl) samples, they delete the
>context without set it.
I also found this on some samples, on the OnPaint event:
// This is normally only necessary if there is more than one wxGLCanvas
// or more than one wxGLContext in the application.
SetCurrent(*m_glRC);
So it may mean that the SetCurrent does not need to take place on all paint
events.
Also, in the end, when destroy it, if it didn't change, there is no need to
SetCurrent again and it will not ASSERT .. (if the context didn't change.. so I
expect that it is still on a valid state and can be used..)
Hope you can have a thought about this.. cheers!
Regards,
Mario Luzeiro
_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to : [email protected]
Unsubscribe : https://launchpad.net/~kicad-developers
More help : https://help.launchpad.net/ListHelp
