Re: [osg-users] Crash in osgViewerWX
Paul Melis wrote: Paul Melis wrote: I don't think there's an easy way out for this one, as wxWidgets uses a different memory management strategy than OSG. Even removing the GraphicsWindowWX as child of the main frame just before exit forces destruction. IMHO GraphicsWindowWX shouldn't inherit from both a wxWidgets class and an OSG class, as the former uses explicit memory management, while the latter uses an implicit style (through osg::Referenced and osg::ref_ptr). A better solution would probably be to split GraphicsWindowWX into a class derived from wxGLCanvas and one derived from osgViewer::GraphicsWindow. The latter would then need to define some custom events for the different events handled by GraphicsWindow and use wxPostEvent to post these to the derived canvas class. But I'd need to try it out to see how this works out. Here's the osgviewerWX example reworked to do the split described above. I'll submit it to osg-submissions later. Perhaps this can help you now. Replying to myself... It actually doesn't use events to communicate between the two parts, but simple method calls. In case GraphicsWindowWX's methods can be called from a different thread than the main GUI loop it will need to use wxPostEvent. Wether this happens in practice might be related to the active threading mode, but I'm not really sure. Paul ___ osg-users mailing list osg-users@lists.openscenegraph.org http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
Re: [osg-users] Crash in osgViewerWX
Hi,
Assuming I understand you correctly you added something like this to
class GraphicsWindowWX:
void setViewer(osgViewer::Viewer *v) { _viewer = v; }
void setView() { _viewer-getCamera()-setViewMatrixAsLookAt(
osg::Vec3(450.,-450.,1200.),osg::Vec3(450.,450.,0.),osg::Vec3(0.,0.,1.)); };
osg::ref_ptrosgViewer::Viewer _viewer;
You also added the setView method() but that's not necessary to cause a
crash, as the above reproduces something similar to what you get, I think.
The problem is that the GraphicsWindowWX is destructed twice on exit.
Once because it is a child window of the top-level wxFrame, which
automatically destroys its children on exit. And once because
osg::Camera holds a reference to it whose count goes to zero when the
Camera is destructed. So the ref_ptr to osgViewer::Viewer is unref()'ed
once too often, causing other problems with destruction of instances the
Viewer holds references to:
(gdb) bt
#0 0x005d4715 in __gnu_cxx::__exchange_and_add () from
/usr/lib/libstdc++.so.6
#1 0x00e659a9 in ~View (this=0x8e7322c) at
/usr/lib/gcc/i386-redhat-linux/4.0.2/../../../../include/c++/4.0.2/bits/basic_string.h:227
#2 0x0805357b in osg::Referenced::unref ()
#3 0x08054862 in osg::ref_ptrosgViewer::Viewer::~ref_ptr ()
#4 0x08051de6 in GraphicsWindowWX::~GraphicsWindowWX ()
#5 0x00d0a78d in ~Camera (this=0x8e73380) at
/home/paul/c/osg-2.4.0/include/osg/Referenced:155
#6 0x0805357b in osg::Referenced::unref ()
#7 0x00e66cd3 in ~View (this=0x8e7322c, __vtt_parm=0x2ab550) at
/home/paul/c/osg-2.4.0/include/osg/ref_ptr:54
#8 0x00267f71 in ~View (this=0x8e7322c, __vtt_parm=0x2ab54c) at
/home/paul/c/osg-2.4.0/src/osgViewer/View.cpp:169
#9 0x0027b3eb in ~Viewer (this=0x8e731e8) at
/home/paul/c/osg-2.4.0/src/osgViewer/Viewer.cpp:172
#10 0x0805357b in osg::Referenced::unref ()
#11 0x08054862 in osg::ref_ptrosgViewer::Viewer::~ref_ptr ()
#12 0x08051de6 in GraphicsWindowWX::~GraphicsWindowWX ()
#13 0x00b6b439 in wxWindowBase::DestroyChildren () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#14 0x00a815b8 in wxWindow::~wxWindow () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#15 0x00b66032 in wxTopLevelWindowBase::~wxTopLevelWindowBase () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#16 0x00a77326 in wxTopLevelWindowGTK::~wxTopLevelWindowGTK () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#17 0x00b220dd in wxFrameBase::~wxFrameBase () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#18 0x00acd75f in wxFrame::~wxFrame () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#19 0x080549e6 in MainFrame::~MainFrame ()
#20 0x00afdf57 in wxAppBase::DeletePendingObjects () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#21 0x00afdfb7 in wxAppBase::OnIdle () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#22 0x0030d55d in wxAppConsole::HandleEvent () from
/usr/lib/libwx_baseu-2.6.so.0
#23 0x0039649f in wxEvtHandler::ProcessEventIfMatches () from
/usr/lib/libwx_baseu-2.6.so.0
#24 0x00396670 in wxEventHashTable::HandleEvent () from
/usr/lib/libwx_baseu-2.6.so.0
#25 0x00396821 in wxEvtHandler::ProcessEvent () from
/usr/lib/libwx_baseu-2.6.so.0
#26 0x00afe210 in wxAppBase::ProcessIdle () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#27 0x00a53003 in ?? () from /usr/lib/libwx_gtk2u_core-2.6.so.0
#28 0x006a0730 in ?? () from /usr/lib/libglib-2.0.so.0
#29 0x0069e4ce in g_main_context_dispatch () from /usr/lib/libglib-2.0.so.0
#30 0x006a14d6 in ?? () from /usr/lib/libglib-2.0.so.0
#31 0x006a17c3 in g_main_loop_run () from /usr/lib/libglib-2.0.so.0
#32 0x02109a46 in gtk_main () from /usr/lib/libgtk-x11-2.0.so.0
#33 0x00a6c204 in wxEventLoop::Run () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#34 0x00afdaaa in wxAppBase::MainLoop () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#35 0x00afdb8f in wxAppBase::OnRun () from
/usr/lib/libwx_gtk2u_core-2.6.so.0
#36 0x00341a44 in wxEntry () from /usr/lib/libwx_baseu-2.6.so.0
#37 0x00341afa in wxEntry () from /usr/lib/libwx_baseu-2.6.so.0
#38 0x08050c81 in main ()
I don't think there's an easy way out for this one, as wxWidgets uses a
different memory management strategy than OSG. Even removing the
GraphicsWindowWX as child of the main frame just before exit forces
destruction. IMHO GraphicsWindowWX shouldn't inherit from both a
wxWidgets class and an OSG class, as the former uses explicit memory
management, while the latter uses an implicit style (through
osg::Referenced and osg::ref_ptr).
A better solution would probably be to split GraphicsWindowWX into a
class derived from wxGLCanvas and one derived from
osgViewer::GraphicsWindow. The latter would then need to define some
custom events for the different events handled by GraphicsWindow and use
wxPostEvent to post these to the derived canvas class. But I'd need to
try it out to see how this works out.
Paul
Jose Marti wrote:
i've used the osgViewerWX example for my program and I'd like to
change the camera view inside the GraphicsWindowsWX class. I've done this:
- I pass the viewer to GraphicsWindowsWX in function
Re: [osg-users] Crash in osgViewerWX
Paul Melis wrote:
I don't think there's an easy way out for this one, as wxWidgets uses
a different memory management strategy than OSG. Even removing the
GraphicsWindowWX as child of the main frame just before exit forces
destruction. IMHO GraphicsWindowWX shouldn't inherit from both a
wxWidgets class and an OSG class, as the former uses explicit memory
management, while the latter uses an implicit style (through
osg::Referenced and osg::ref_ptr).
A better solution would probably be to split GraphicsWindowWX into a
class derived from wxGLCanvas and one derived from
osgViewer::GraphicsWindow. The latter would then need to define some
custom events for the different events handled by GraphicsWindow and
use wxPostEvent to post these to the derived canvas class. But I'd
need to try it out to see how this works out.
Here's the osgviewerWX example reworked to do the split described above.
I'll submit it to osg-submissions later. Perhaps this can help you now.
Paul
Paul
Jose Marti wrote:
i've used the osgViewerWX example for my program and I'd like to
change the camera view inside the GraphicsWindowsWX class. I've done
this:
- I pass the viewer to GraphicsWindowsWX in function wxOsgApp::OnInit().
frame-SetViewer(viewer.get());
gw-setViewer(viewer.get());
Where does setViewer() come from, and what does it do?
And I set the new camera view in class GraphicsWindowWX.
void GraphicsWindowWX::setView()
{
_viewer-getCamera()-setViewMatrixAsLookAt(
osg::Vec3(450.,-450.,1200.),osg::Vec3(450.,450.,0.),osg::Vec3(0.,0.,1.));
}
Where is setView() called?
It works fine, but when I close the program, it crashes and i got
this message in the console window:
'Warning: deleting still referenced object 08E57B70 of tye 'class
osg::Referenced *'
the final reference count was 1189214347, memory
corruption possible.
What am I doing wrong??
Hard to say, as you don't provide enoug details to go on.
But if you're storing the osgViewer::Viewer instance in
GraphicsWindowWX (which seems to be what you do).
Paul
___
osg-users mailing list
osg-users@lists.openscenegraph.org
http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
// For compilers that support precompilation, includes wx.h.
#include wx/wxprec.h
#ifdef __BORLANDC__
#pragma hdrstop
#endif
#ifndef WX_PRECOMP
#include wx/wx.h
#endif
//#ifdef WIN32
//#include wx/msw/wx.rc
//#endif
#include osgviewerWX.h
#include osgViewer/ViewerEventHandlers
#include osgGA/TrackballManipulator
#include osgDB/ReadFile
#include wx/image.h
#include wx/menu.h
#include iostream
// `Main program' equivalent, creating windows and returning main app frame
bool wxOsgApp::OnInit()
{
if (argc2)
{
std::cout wxString(argv[0]).mb_str() : requires filename argument. std::endl;
return false;
}
int width = 800;
int height = 600;
// Create the main frame window
MainFrame *frame = new MainFrame(NULL, wxT(wxWidgets OSG Sample),
wxDefaultPosition, wxSize(width, height));
// create osg canvas
//- initialize
int *attributes = new int[6];
attributes[0] = int(WX_GL_DOUBLEBUFFER);
attributes[1] = WX_GL_RGBA;
attributes[2] = WX_GL_DEPTH_SIZE;
attributes[3] = 8;
attributes[4] = WX_GL_STENCIL_SIZE;
attributes[5] = 8;
OSGCanvas *canvas = new OSGCanvas(frame, wxID_ANY, wxDefaultPosition,
wxSize(width, height), wxSUNKEN_BORDER, wxT(osgviewerWX), attributes);
GraphicsWindowWX* gw = new GraphicsWindowWX(canvas);
canvas-SetGraphicsWindow(gw);
canvas-UseCursor(false);
osgViewer::Viewer *viewer = new osgViewer::Viewer;
viewer-getCamera()-setGraphicsContext(gw);
viewer-getCamera()-setViewport(0,0,width,height);
viewer-addEventHandler(new osgViewer::StatsHandler);
viewer-setThreadingModel(osgViewer::Viewer::SingleThreaded);
// load the scene.
wxString fname(argv[1]);
osg::ref_ptrosg::Node loadedModel = osgDB::readNodeFile(std::string(fname.mb_str()));
if (!loadedModel)
{
std::cout argv[0] : No data loaded. std::endl;
return false;
}
viewer-setSceneData(loadedModel.get());
viewer-setCameraManipulator(new osgGA::TrackballManipulator);
frame-SetViewer(viewer);
/* Show the frame */
frame-Show(true);
return true;
}
IMPLEMENT_APP(wxOsgApp)
BEGIN_EVENT_TABLE(MainFrame, wxFrame)
EVT_IDLE(MainFrame::OnIdle)
END_EVENT_TABLE()
/* My frame constructor */
MainFrame::MainFrame(wxFrame *frame, const wxString title, const wxPoint pos,
const wxSize size, long style)
: wxFrame(frame, wxID_ANY, title, pos, size, style)
{
}
void MainFrame::SetViewer(osgViewer::Viewer *viewer)
{
_viewer = viewer;
}
void MainFrame::OnIdle(wxIdleEvent event)
{
_viewer-frame();
event.RequestMore();
}
BEGIN_EVENT_TABLE(OSGCanvas, wxGLCanvas)
EVT_SIZE
[osg-users] Crash in osgViewerWX
hi,
i've used the osgViewerWX example for my program and I'd like to change the
camera view inside the GraphicsWindowsWX class. I've done this:
- I pass the viewer to GraphicsWindowsWX in function wxOsgApp::OnInit().
frame-SetViewer(viewer.get());
gw-setViewer(viewer.get());
And I set the new camera view in class GraphicsWindowWX.
void GraphicsWindowWX::setView()
{
_viewer-getCamera()-setViewMatrixAsLookAt(
osg::Vec3(450.,-450.,1200.),osg::Vec3(450.,450.,0.),osg::Vec3(0.,0.,1.));
}
It works fine, but when I close the program, it crashes and i got this message
in the console window:
'Warning: deleting still referenced object 08E57B70 of tye 'class
osg::Referenced *'
the final reference count was 1189214347, memory corruption
possible.
What am I doing wrong??
_
La vida de los famosos al desnudo en MSN Entretenimiento
http://entretenimiento.es.msn.com/___
osg-users mailing list
osg-users@lists.openscenegraph.org
http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
