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.

Thanks, I've applied all your fixes.



In Visual Studio this #pragma is used for custom code folding. There used to be lots of them, but I didn't really use the feature and they generatd GCC warnings so I took them out. Seems I missed one.