Hello Robert,
I've now checked in a number of warning fixes, out of over 1000 warnings that should be removed (mainly repeats) I think just four were actual bugs fixes, whilst they might be minor bugs it's still nice to see them caught and hopefully fixed . The bulk left are unused parameter and assignment in conditionals.
I just did an update and it doesn't seem you fixed the "warning C4512: 'classname' : assignment operator could not be generated" warning? That's the most frequent... I'd like to see it fixed or suppressed if you don't feel like adding a dummy assignment operator...
My though now is to add suppression of C4706 into include/osg/Export and then review each of the existing suppressed warnings to see if we can remove ones that no longer effect us. The warning codes used date back to VS 6.0 days, and things have moved on a lot in compiler and on the OSG side to warrant a revist of which warnings get suppressed.
I'm fine with suppressing C4706 as it's true that it's a warning that only tells you that you *might* be doing something unsafe... Whether it's actually safe or not is totally context-dependent.
As for the other warnings, I can have a look at their descriptions and see, but that might be more your alley since it requires a judgement call. As I mentioned, just use <warning number> site:microsoft.com to look up the documentation on a warning type.
Thanks, 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
