Print warnings only ones

bindgen/CMakeLists.txt

@@ -48,6 +48,8 @@ add_executable(bindgen
   ir/Define.cpp
   ir/LiteralDefine.cpp
   ir/LiteralDefine.h
+  utils/WarningManager.cpp
+  utils/WarningManager.h
 )
 
 set_target_properties(bindgen

bindgen/TypeTranslator.cpp

@@ -137,16 +137,15 @@ std::string TypeTranslator::Translate(const clang::QualType &qtpe,
     // Warning / Sanity checks
 
     if (qtpe.isConstQualified() || (ctx && qtpe.isConstant(*ctx))) {
-        llvm::errs() << "Warning: Const qualifier not supported\n";
-        llvm::errs().flush();
+        warningManager.printWarning("Warning: Const qualifier not supported");
     }
     if (qtpe.isVolatileQualified()) {
-        llvm::errs() << "Warning: Volatile qualifier not supported\n";
-        llvm::errs().flush();
+        warningManager.printWarning(
+            "Warning: Volatile qualifier not supported");
     }
     if (qtpe.isRestrictQualified()) {
-        llvm::errs() << "Warning: Restrict qualifier not supported\n";
-        llvm::errs().flush();
+        warningManager.printWarning(
+            "Warning: Restrict qualifier not supported");
     }
 
     const clang::Type *tpe = qtpe.getTypePtr();

bindgen/TypeTranslator.h

@@ -1,5 +1,6 @@
 #pragma once
 
+#include "utils/WarningManager.h"
 #include <clang/AST/AST.h>
 #include <clang/AST/ASTContext.h>
 #include <clang/Tooling/Tooling.h>
@@ -8,6 +9,7 @@ class TypeTranslator {
   private:
     clang::ASTContext *ctx;
     std::map<std::string, std::string> typeMap;
+    WarningManager warningManager;
 
   public:
     explicit TypeTranslator(clang::ASTContext *ctx);

bindgen/utils/WarningManager.cpp

@@ -0,0 +1,11 @@
+#include "WarningManager.h"
+
+#include <llvm/Support/raw_ostream.h>
+
+void WarningManager::printWarning(std::string warning) {
+    if (printedWarnings.find(warning) == printedWarnings.end()) {
+        llvm::errs() << warning << "\n";
+        llvm::errs().flush();
+        printedWarnings.insert(warning);
+    }
+}

bindgen/utils/WarningManager.h

@@ -0,0 +1,18 @@
+#ifndef SCALA_NATIVE_BINDGEN_WARNINGMANAGER_H
+#define SCALA_NATIVE_BINDGEN_WARNINGMANAGER_H
+
+#include <string>
+#include <unordered_set>
+
+/**
+ * Ignores duplicate warnings
+ */
+class WarningManager {
+  public:
+    void printWarning(std::string warning);
+
+  private:
+    std::unordered_set<std::string> printedWarnings;
+};
+
+#endif // SCALA_NATIVE_BINDGEN_WARNINGMANAGER_H