From: John P. <jwp...@gm...> - 2013-01-31 15:49:38
|
On Thu, Jan 31, 2013 at 8:45 AM, Kirk, Benjamin (JSC-EG311) <ben...@na...> wrote: > On Jan 31, 2013, at 9:41 AM, John Peterson <pet...@cf...> wrote: > >> BTW, I'm not sure we should ever add -Wshadow to default compiler >> flags for libmesh! Any apps compiling using libmesh_CXXFLAGS would >> suddenly explode with new (usually innocuous) warnings > > If we don't we will continue to write shadowed code ourselves… Very true. > So I propose we add them to the git repo, potentially removing them for a particular release? How about a buildbot configuration with -Wshadow turned on? It's something we can let go for a while and then go back and fix later. > what really makes me nuts is when things get triggered in headers we include… Yes, VTK seems to be really stupid about this. Still #including iostream.h or something ridiculous. -- John |