When I compile PolyVox using g++ 4.6 and "-Wall -Wextra" (always a good idea) I get a bunch of warnings.
GradientEstimators.h contains a "#pragma" that g++ warns about. I think this pragma is entirely useless regarding code generation, so it can go.
SurfaceMesh.h / SurfaceMesh.inl contain a function "isEmpty()" which returns a "const bool". g++ warns about this, "bool" as return type is enought. As it is neither a reference nor a pointer, defining constness on it is a bit weird indeed.
AStarPathfinderImpl.h defines the member variables parent, gVal, hVal in this order. But the constructor initializes them in another order, which produces a warning as well. The cure is easy: make definition and initialization order the same.
| Attachments: |
fixes.patch [1.98 KiB]
Downloaded 243 times |