<div dir="ltr">Hi Andy,<br><div class="gmail_extra"><br><div class="gmail_quote">On 26 April 2016 at 20:31, Andy Skinner <span dir="ltr"><<a href="mailto:Andy.Skinner@mathworks.com" target="_blank">Andy.Skinner@mathworks.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div link="blue" vlink="purple" lang="EN-US">
<div>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">Thanks for that fix.  There is a remaining issue, and I'm not sure if you would see it as a bug or a dodgy compiler warning.  :)<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">We get a similar message about osgUtil::CullVisitor::clone().<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">NodeVisitor uses META_Object, which brings in:<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">virtual osg::Object* clone(const osg::CopyOp& copyop) const { return new name (*this,copyop); }<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">But CullVisitor defines clone as:<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">        virtual CullVisitor* clone() const { return new CullVisitor(*this); }<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">They differ in whether they take an argument.  I believe we've discussed this kind of thing before when considering using "using", and it was, if I remember correctly,
 a dodgy compiler warning then.<u></u><u></u></span></p><br></div></div></blockquote><div><br></div><div>The osg::ShaderComposer warning did highlight a bug, but this warning isn't highlighting an actual error.  If one calls clone(osg::CopyOp) then you'll still git the correct clone() method.  It does highlight what is not ideal coding style, and in this case it's a historical reason why there is this clone() convenience method.  These days a osg::clone(object) would do just as well without requiring the extra method, but this is a relatively modern addition.<br><br></div><div>Could you try doing adding:<br><br></div><div>   using NodeVisitor::clone;<br><br></div><div>To the include/osgUtil/CullVisitor and see if that quietens the warning.<br><br></div><div>Robert.<br></div><div><br> </div></div></div></div>