Extra C++ warning flags with clang and g++.

Fixes 223.

2 files changed, 23 insertions, 9 deletions

diff --git a/cpp/src/slice2js/Gen.cpp b/cpp/src/slice2js/Gen.cpp
index 10b571b6d83..58c6ee1e51b 100644
--- a/cpp/src/slice2js/Gen.cpp
+++ b/cpp/src/slice2js/Gen.cpp
@@ -17,6 +17,13 @@
 #include <Slice/FileTracker.h>
 #include <Slice/Util.h>
 
+// TODO: fix this warning!
+#if defined(__clang__)
+#   pragma clang diagnostic ignored "-Wshadow"
+#elif defined(__GNUC__)
+#   pragma GCC diagnostic ignored "-Wshadow"
+#endif
+
 using namespace std;
 using namespace Slice;
 using namespace IceUtil;
@@ -419,7 +426,7 @@ Slice::JsVisitor::writeInitDataMembers(const DataMemberList& dataMembers)
 }
 
 string
-Slice::JsVisitor::getValue(const string& scope, const TypePtr& type)
+Slice::JsVisitor::getValue(const string& /*scope*/, const TypePtr& type)
 {
     assert(type);
 
@@ -479,7 +486,7 @@ Slice::JsVisitor::getValue(const string& scope, const TypePtr& type)
 }
 
 string
-Slice::JsVisitor::writeConstantValue(const string& scope, const TypePtr& type, const SyntaxTreeBasePtr& valueType,
+Slice::JsVisitor::writeConstantValue(const string& /*scope*/, const TypePtr& type, const SyntaxTreeBasePtr& valueType,
                                      const string& value)
 {
     ostringstream os;
@@ -642,7 +649,7 @@ Slice::Gen::Gen(const string& base, const vector<string>& includePaths, const st
     printGeneratedHeader(_out, _fileBase + ".ice");
 }
 
-Slice::Gen::Gen(const string& base, const vector<string>& includePaths, const string& dir, bool typeScript,
+Slice::Gen::Gen(const string& base, const vector<string>& includePaths, const string& /*dir*/, bool typeScript,
                 ostream& out) :
     _out(out),
     _includePaths(includePaths),
@@ -815,14 +822,14 @@ Slice::Gen::RequireVisitor::visitClassDefStart(const ClassDefPtr& p)
 }
 
 bool
-Slice::Gen::RequireVisitor::visitStructStart(const StructPtr& p)
+Slice::Gen::RequireVisitor::visitStructStart(const StructPtr&)
 {
     _seenStruct = true; // Set regardless of whether p->isLocal()
     return false;
 }
 
 void
-Slice::Gen::RequireVisitor::visitOperation(const OperationPtr& p)
+Slice::Gen::RequireVisitor::visitOperation(const OperationPtr&)
 {
     _seenOperation = true;
 }
@@ -879,7 +886,7 @@ Slice::Gen::RequireVisitor::visitDictionary(const DictionaryPtr& dict)
 }
 
 void
-Slice::Gen::RequireVisitor::visitEnum(const EnumPtr& p)
+Slice::Gen::RequireVisitor::visitEnum(const EnumPtr&)
 {
     _seenEnum = true;
 }
@@ -1156,7 +1163,7 @@ Slice::Gen::TypesVisitor::visitModuleStart(const ModulePtr& p)
 }
 
 void
-Slice::Gen::TypesVisitor::visitModuleEnd(const ModulePtr& p)
+Slice::Gen::TypesVisitor::visitModuleEnd(const ModulePtr&)
 {
 }
 
@@ -2437,7 +2444,7 @@ Slice::Gen::TypeScriptAliasVisitor::addAlias(const string& type, const string& p
 }
 
 bool
-Slice::Gen::TypeScriptAliasVisitor::visitModuleStart(const ModulePtr& p)
+Slice::Gen::TypeScriptAliasVisitor::visitModuleStart(const ModulePtr&)
 {
     return true;
 }
@@ -2539,7 +2546,7 @@ Slice::Gen::TypeScriptAliasVisitor::visitDictionary(const DictionaryPtr& dict)
 }
 
 void
-Slice::Gen::TypeScriptAliasVisitor::writeAlias(const UnitPtr& p)
+Slice::Gen::TypeScriptAliasVisitor::writeAlias(const UnitPtr&)
 {
     if(!_aliases.empty())
     {
diff --git a/cpp/src/slice2js/JsUtil.cpp b/cpp/src/slice2js/JsUtil.cpp
index 55e47de850b..607c7bc9339 100644
--- a/cpp/src/slice2js/JsUtil.cpp
+++ b/cpp/src/slice2js/JsUtil.cpp
@@ -23,6 +23,13 @@
 #include <unistd.h>
 #endif
 
+// TODO: fix this warning!
+#if defined(__clang__)
+#   pragma clang diagnostic ignored "-Wshadow"
+#elif defined(__GNUC__)
+#   pragma GCC diagnostic ignored "-Wshadow"
+#endif
+
 using namespace std;
 using namespace Slice;
 using namespace IceUtil;