Fix #39: Enable compiler warnings

Author: jonas

CMakeLists.txt

@@ -6,10 +6,10 @@ find_package(Clang REQUIRED CONFIG)
 message(STATUS "Found LLVM ${LLVM_PACKAGE_VERSION}")
 message(STATUS "Using LLVMConfig.cmake in: ${LLVM_DIR}")
 
-include_directories(${LLVM_INCLUDE_DIRS})
+include_directories(SYSTEM ${LLVM_INCLUDE_DIRS})
 add_definitions(${LLVM_DEFINITIONS})
 
-add_compile_options(-fexceptions -std=c++11)
+add_compile_options(-fexceptions -std=c++11 -Wall -Wconversion -Werror)
 
 add_executable(bindgen
   Main.cpp

Utils.h

@@ -19,8 +19,8 @@ inline std::string uint64ToScalaNat(uint64_t v, std::string accumulator = "") {
     if (v == 0)
         return accumulator;
 
-    int last_digit = v % 10;
-    int rest = v / 10;
+    auto last_digit = v % 10;
+    auto rest = v / 10;
 
     if (accumulator.empty()) {
         return uint64ToScalaNat(rest, "_" + std::to_string(last_digit));
@@ -44,11 +44,11 @@ inline bool typeEquals(const clang::Type* tpe1, const std::string* tpe2){
     return false;
 }
 
-static std::array<std::string, 39> reserved_words = {"abstract", "case", "catch", "class", "def", "do", "else", "extends",
+static std::array<std::string, 39> reserved_words = {{"abstract", "case", "catch", "class", "def", "do", "else", "extends",
                                       "false", "final", "finally", "for", "forSome", "if", "implicit", "import",
                                       "lazy", "match", "new", "null", "object", "override", "package", "private",
                                       "protected", "return", "sealed", "super", "this", "throw", "trait", "try",
-                                      "true", "type", "val", "var", "while", "with", "yield"};
+                                      "true", "type", "val", "var", "while", "with", "yield"}};
 
 inline std::string handleReservedWords(std::string name, std::string suffix = "") {
     auto found = std::find(reserved_words.begin(), reserved_words.end(), name);

ir/Function.cpp

@@ -13,13 +13,12 @@ Function::Function(std::string name, std::vector<Parameter> parameters,
 llvm::raw_ostream &operator<<(llvm::raw_ostream &s, const Function &func) {
     s << "  def " << handleReservedWords(func.name)
       << "(";
-    auto paramsCount = static_cast<int>(func.parameters.size());
-    for (int i = 0; i < paramsCount; ++i) {
+    for (std::vector<Parameter>::size_type i = 0; i < func.parameters.size(); i++) {
         const Parameter &param = func.parameters[i];
         s << handleReservedWords(param.getName())
           << ": "
           << param.getType();
-        if (i < paramsCount - 1) {
+        if (i + 1 != func.parameters.size()) {
             s << ", ";
         }
     }

ir/Struct.cpp

@@ -24,10 +24,9 @@ TypeDef Struct::generateTypeDef() const {
 
 std::string Struct::getFieldsTypes() const {
     std::stringstream s;
-    auto fieldsCount = static_cast<int>(fields.size());
-    for (int i = 0; i < fieldsCount; i++) {
+    for (std::vector<Field>::size_type i = 0; i < fields.size(); i++) {
         s << fields[i].getType();
-        if (i < fieldsCount - 1) {
+        if (i + 1 != fields.size()) {
             s << ", ";
         }
     }

scripts/test.sh

@@ -10,8 +10,9 @@ if [[ ! -e target/.llvm-version ]] || [[ "$(<target/.llvm-version)" != "${LLVM_V
   mkdir -p target
   echo "${LLVM_VERSION:-}" > target/.llvm-version
   (cd target && cmake ..)
-  make -C target
 fi
 
+make -C target
+
 cd tests
 sbt "${@:-test}"

visitor/TreeConsumer.h

@@ -17,15 +17,11 @@ class TreeConsumer : public clang::ASTConsumer {
 
     clang::SourceManager &smanager;
 
-    /* pointer to ir that is stored in ScalaFrontendActionFactory */
-    IR *ir;
-
 public:
 
     explicit TreeConsumer(clang::CompilerInstance *CI, IR *ir)
             : visitor(CI, ir),
-              smanager(CI->getASTContext().getSourceManager()),
-              ir(ir) {}
+              smanager(CI->getASTContext().getSourceManager()) {}
 
     bool HandleTopLevelDecl(clang::DeclGroupRef DG) override {
         // a DeclGroupRef may have multiple Decls, so we iterate through each one