<div dir="ltr"><div><div><div><div>Hi Jannik and Ulrich,<br><br></div>You are right it's a bug, the missing const is an error, rather than the missing virtual. I've fixed this and rewritten the implementation of ShaderCompose::releaseGLObjects() const. <br><br></div>I need to do a build and test before checking it in.<br><br></div>Thankfully ShaderComposer is a not widely used, and deprecated thanks to #pragmatic shader composition so this error shouldn't effect too many users.<br><br></div>Robert.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 18 April 2016 at 20:02, Ulrich Hertlein <span dir="ltr"><<a href="mailto:u.hertlein@sandbox.de" target="_blank">u.hertlein@sandbox.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi guys,<br>
<span class=""><br>
On 18/04/2016 20:22, Robert Osfield wrote:<br>
> On 18 April 2016 at 17:39, Andy Skinner <<a href="mailto:Andy.Skinner@mathworks.com">Andy.Skinner@mathworks.com</a><br>
</span><span class="">> <mailto:<a href="mailto:Andy.Skinner@mathworks.com">Andy.Skinner@mathworks.com</a>>> wrote:<br>
><br>
> We are getting a warning for ShaderComposer::releaseGLObjects: 'void<br>
> osg::ShaderComposer::releaseGLObjects(osg::State *)' : member function does not<br>
> override any base class virtual member function osg::Object::releaseGLObjects is<br>
> virtual, but osg::ShaderComposer::releaseGLObjects is not. Is there a reason why?<br>
><br>
> Sounds like a dodgy compiler warning.<br>
<br>
</span>I think not, it looks like an actual bug:<br>
<br>
osg::Object<br>
virtual void releaseGLObjects(osg::State* = 0) const {}<br>
<br>
osg::ShaderComposer<br>
void releaseGLObjects(osg::State* state);<br>
<br>
The ShaderComposer method is non-const and therefore does not override the osg::Object method!<br>
<br>
Can we use the 'override' keyword yet?<br>
<br>
Cheers,<br>
/ulrich<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<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>