Improve the test coverage of the C API -- Take 4

compiler-research · Jan 13, 2025 · 0333803 · 0333803
1 parent 40f21ca
commit 0333803
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/lib/Interpreter/CXCppInterOp.cpp b/lib/Interpreter/CXCppInterOp.cpp
@@ -326,18 +326,16 @@ void clang_Interpreter_addIncludePath(CXInterpreter I, const char* dir) {
   getInterpreter(I)->AddIncludePath(dir);
 }
 
+namespace Cpp {
+int Declare(compat::Interpreter& interp, const char* code, bool silent);
+} // namespace Cpp
+
 enum CXErrorCode clang_Interpreter_declare(CXInterpreter I, const char* code,
                                            bool silent) {
   auto* interp = getInterpreter(I);
-  auto& diag = interp->getSema().getDiagnostics();
-
-  const bool is_silent_old = diag.getSuppressAllDiagnostics();
-
-  diag.setSuppressAllDiagnostics(silent);
-  const auto result = interp->declare(code);
-  diag.setSuppressAllDiagnostics(is_silent_old);
 
-  if (result != Cpp::Interpreter::kSuccess)
+  const auto result = Cpp::Declare(*interp, code, silent);
+  if (result)
     return CXError_Failure;
 
   return CXError_Success;

diff --git a/lib/Interpreter/CppInterOp.cpp b/lib/Interpreter/CppInterOp.cpp
@@ -2887,9 +2887,7 @@ namespace Cpp {
   };
   } // namespace
 
-  int Declare(const char* code, bool silent) {
-    auto& I = getInterp();
-
+  int Declare(compat::Interpreter& I, const char* code, bool silent) {
     if (silent) {
       clangSilent diagSuppr(I.getSema().getDiagnostics());
       return I.declare(code);
@@ -2898,6 +2896,10 @@ namespace Cpp {
     return I.declare(code);
   }
 
+  int Declare(const char* code, bool silent) {
+    return Declare(getInterp(), code, silent);
+  }
+
   int Process(const char *code) {
     return getInterp().process(code);
   }