Hi Ralf, I have just registered myself with the Coverity scan for the OSG, pending approval.
I haven't dived into your changes yet. I'm inclined to do them incrementally rather than all 269 together as there is always a chance that fixes to warning that these tools generated actually introduce there own bugs. The smaller chunk of changes we review and merge at a time the higher the likelyhood of catching errors before checking in and easier it is too roll things back if there is a problem. Do you have github fork of the OSG where you've made these changes? Robert. On 14 April 2016 at 14:50, Ralf Habacker <[email protected]> wrote: > Hi Robert > > at https://scan.coverity.com/projects/openscenegraph?tab=overview the > openscenegraph sources has been reviewed by Coverity, a static code > analyser. > > The appended patch fixes 269 reported issues in the ive plugin of kind: > > CID xxx (#1 of 1): Failed to override method (BAD_OVERRIDE) > bad_override: Method ive::xxx::read hides but does not override > ive::ReadWrite::read because some type qualifiers do not match. > CID xxx (#1 of 1): Failed to override method > (BAD_OVERRIDE)bad_override: Method ive::xxx::write hides but does not > override ive::ReadWrite::write because some type qualifiers do not match. > > Regards > Ralf > > _______________________________________________ > osg-submissions mailing list > [email protected] > http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org > _______________________________________________ osg-submissions mailing list [email protected] http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
