<div dir="ltr"><div><div><div>Hi Pjotr,<br><br></div>Thanks for testing with VS2015 and reporting the warnings.  I don't have VS2015 so can't test any fixes myself, could you have a bash at fixing the warnings and send me the changes, I can then review.<br><br></div><div>Currently I'm working on addressing warnings generated by clang 3.6, it's pretty strict but hasn't reported the same warnings as VS2015.<br></div><div><br></div>Cheers,<br></div>Robert.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 23 July 2015 at 11:12, Pjotr Svetachov <span dir="ltr"><<a href="mailto:pjotrsvetachov@gmail.com" target="_blank">pjotrsvetachov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Robert,<br>
<br>
Testing with the newly released VS2015 produced these 4 warnings for each file that includes (directly or indirectly) osg/GraphicsThread this will lead to hundreds of warnings:<br>
1>I:\Libraries\OpenSceneGraphTrunk\include\osg/OperationThread(55): warning C4589: Constructor of abstract class 'osg::Operation' ignores initializer for virtual base class 'osg::Referenced'<br>
1>  I:\Libraries\OpenSceneGraphTrunk\include\osg/OperationThread(55): note: virtual base classes are only initialized by the most-derived type<br>
1>I:\Libraries\OpenSceneGraphTrunk\include\osg/OperationThread(81): warning C4589: Constructor of abstract class 'osg::Operation' ignores initializer for virtual base class 'osg::Referenced'<br>
1>  I:\Libraries\OpenSceneGraphTrunk\include\osg/OperationThread(81): note: virtual base classes are only initialized by the most-derived type<br>
1>I:\Libraries\OpenSceneGraphTrunk\include\osg/OperationThread(85): warning C4589: Constructor of abstract class 'osg::Operation' ignores initializer for virtual base class 'osg::Referenced'<br>
1>  I:\Libraries\OpenSceneGraphTrunk\include\osg/OperationThread(85): note: virtual base classes are only initialized by the most-derived type<br>
1>I:\Libraries\OpenSceneGraphTrunk\include\osg/GraphicsThread(45): warning C4589: Constructor of abstract class 'osg::GraphicsOperation' ignores initializer for virtual base class 'osg::Referenced'<br>
1>  I:\Libraries\OpenSceneGraphTrunk\include\osg/GraphicsThread(45): note: virtual base classes are only initialized by the most-derived type<br>
1>  I:\Libraries\OpenSceneGraphTrunk\include\osg/GraphicsThread(45): note: This diagnostic occurred in the compiler generated function 'osg::GraphicsOperation::GraphicsOperation(const osg::GraphicsOperation &)'<br>
<br>
Trying to find out what is really happening I stumbled on this:<br>
<a href="http://stackoverflow.com/questions/10534228/order-of-constructor-call-in-virtual-inheritance" rel="noreferrer" target="_blank">http://stackoverflow.com/questions/10534228/order-of-constructor-call-in-virtual-inheritance</a><br>
<br>
The first three warnings come from this:<br>
The problem is that the Operation class is virtually inheriting from the class Referenced. The constructors and copy constructors of the class Operation call Referenced(true). Now, with virtual inheritance the constructors of virtual base classes are always called from the most derived class. Now what Visual Studio is complaining about is that Operations is a abstract class so you can not construct an instance of it directly. You need to inherit from it, implement the abstract methods and construct the derived class. So the call to Referenced(true) will never happen inside the Operation constructors. Instead you need to add this call to the most derived classes of Operation. If you don't do this it will call the default constructor Referenced::Referenced() instead of Referenced::Referenced(true).<br>
I did a search to see whats inheriting from Operation and found a few cases. For example BlockAndFlushOperation and ReleaseContext_Block_MakeCurrentOperation do not call Referenced(true) in their constructors so then the default constructor Referenced() is called.<br>
<br>
The 4th warning comes from that visual studio makes a default constructor for GraphicsOperation that calls Referenced() and then complains about it, this is probably a bug in vs2015, see also: <a href="https://connect.microsoft.com/VisualStudio/feedback/details/1570496/vs-2015-generates-a-copy-constructor-and-then-complains-about-it" rel="noreferrer" target="_blank">https://connect.microsoft.com/VisualStudio/feedback/details/1570496/vs-2015-generates-a-copy-constructor-and-then-complains-about-it</a><br>
<br>
The same is happening for the class ViewerBase because it is using virtual inheritance to derive from Object and calling Object(true) from the constructor.<br>
For ViewerBase I even got an error:<br>
1>I:\Libraries\OpenSceneGraphTrunk\include\osgViewer/ViewerBase(340): error C2249: 'osg::Object::operator =': no accessible path to private member declared in virtual base 'osg::Object'<br>
1>  I:\Libraries\OpenSceneGraphTrunk\include\osg/Object(238): note: see declaration of 'osg::Object::operator ='<br>
1>  I:\Libraries\OpenSceneGraphTrunk\include\osg/Object(56): note: see declaration of 'osg::Object'<br>
1>  I:\Libraries\OpenSceneGraphTrunk\include\osgViewer/ViewerBase(340): note: This diagnostic occurred in the compiler generated function 'osgViewer::ViewerBase &osgViewer::ViewerBase::operator =(const osgViewer::ViewerBase &)'<br>
But this looks like the bug in visual studio from above but this time it makes a default copy operator for ViewerBase that tries to call the private copy operator of Object from the generated default copy operator. It might be an bug in the compiler or it might be by a genuine error (I don't know the specs of c++ that well:) ) but a workaround for this error is to add:<br>
<br>
Code:<br>
<br>
private:<br>
ViewerBase& operator = (const ViewerBase&) { return *this; }<br>
<br>
<br>
<br>
At the end of the ViewerBase class. (At least if that is the intended behavior)<br>
<br>
<br>
Cheers,<br>
Pjotr<br>
<br>
------------------<br>
Read this topic online here:<br>
<a href="http://forum.openscenegraph.org/viewtopic.php?p=64443#64443" rel="noreferrer" target="_blank">http://forum.openscenegraph.org/viewtopic.php?p=64443#64443</a><br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
<br>
<br>
_______________________________________________<br>
osg-users mailing list<br>
<a href="mailto:osg-users@lists.openscenegraph.org">osg-users@lists.openscenegraph.org</a><br>
<a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org</a><br>
</div></div></blockquote></div><br></div>