Hi Justin,

My guess is that we can probably relax the use of private: in this context.

I've moved the private sections into the protected section, and
re-ordered the class interface so that the public section is first,
along the lines of the rest of the OSG.  An svn update will provide
this change.  Could you test this out.

Thanks,
Robert.


On Tue, Jul 15, 2008 at 8:49 PM, Vican, Justin E. <[EMAIL PROTECTED]> wrote:
> Hi Jeremy, J-S, Robert,
> I just updated to the latest and greatest (r8597), and the new osgWidget code 
> is throwing the following errors when I try to compile:
> 64Bit Linux, RHEL4, gcc3.4.6
>
>
> src/osgWidget/Window.cpp: In member function `bool 
> osgWidget::Window::EmbeddedWindow::setWindow(osgWidget::Window*)':
> include/osgWidget/Widget:67: error: 
> `osgWidget::Window*osgWidget::Widget::_parent' is private
> src/osgWidget/Window.cpp:118: error: within this context
> include/osgWidget/Widget:67: error: 
> `osgWidget::Window*osgWidget::Widget::_parent' is private
> src/osgWidget/Window.cpp:118: error: within this context
> include/osgWidget/Widget:112: error: `osgWidget::WindowManager* 
> osgWidget::Widget::_getWindowManager() const' is private
> src/osgWidget/Window.cpp:120: error: within this context
> include/osgWidget/Widget:112: error: `osgWidget::WindowManager* 
> osgWidget::Widget::_getWindowManager() const' is private
> src/osgWidget/Window.cpp:120: error: within this context
> src/osgWidget/Window.cpp: In member function `void 
> osgWidget::Window::_removeFromGeode(osgWidget::Widget*)':
> src/osgWidget/Window.cpp:413: warning: converting of negative value 
> `-0x00000000000000001' to `unsigned int'
> gmake[3]: *** [src/osgWidget/CMakeFiles/osgWidget.dir/Window.o] Error 1
> gmake[3]: Leaving directory `BUILD/Linux/x86_64'
> gmake[2]: *** [src/osgWidget/CMakeFiles/osgWidget.dir/all] Error 2
> gmake[2]: Leaving directory `BUILD/Linux/x86_64'
> gmake[1]: *** [all] Error 2
> gmake[1]: Leaving directory `BUILD/Linux/x86_64'
> gmake: *** [build] Error 2
>
>
> The following minor modifications seem to resolve the errors.  Is this an 
> acceptable modification?
>
>
> #if 0 // SVN r8597
>    if(_parent) parented(_parent);
>    WindowManager* wm = _getWindowManager();
> #else // My modifications:
>    if (getParent()) parented(getParent());
>    WindowManager* wm = getWindowManager();
> #endif
>
>
>
> Thanks,
> Justin
>
> P.S.  Other than that, the source tree seems to be building fine ... 44% done 
> ... will update after coffee :)
>
>
> -----Original Message-----
> From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Jean-Sébastien 
> Guay
> Sent: Tuesday, July 15, 2008 3:21 PM
> To: OpenSceneGraph Users
> Subject: Re: [osg-users] osgWidget now checked into OSG SVN trunk
>
> Hi Jeremy,
>
>> It's all up to the ResizeHandler though; if you wanted to ALWAYS work in
>> some coordinate space like 1024x768, it should be possible just by using
>> (or perhaps not even using) a different ResizeHandler. The one I created
>> in ViewerEventHandler, however, tries to make sure the WindowManager
>> object always knows the exact pixel dimensions of the View.
>
> The question is will it be automatic? If, for a HUD, you always want to
>
> have pixel-exact mapping, then it should be automatic and not depend on
> what size the user sets his window to.
>
> I'll leave the details to you, as you're much better placed to
> understand what's involved. All I'm saying is that osgViewer supports
> users setting environment variables or passing command-line arguments to
> control how the window appears, and I think that in examples, these
> should not be overridden by hard-coding a window position/size.
>
> J-S
> --
> ______________________________________________________
> Jean-Sebastien Guay    [EMAIL PROTECTED]
>                                http://www.cm-labs.com/
>                         http://whitestar02.webhop.org/
> _______________________________________________
> osg-users mailing list
> [email protected]
> http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
> _______________________________________________
> osg-users mailing list
> [email protected]
> http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
>
_______________________________________________
osg-users mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org

Reply via email to