feat: uniform logging level control across CorpusImpl

Author: fpelliccioni

src/lib/Lib/CorpusImpl.cpp

@@ -101,7 +101,6 @@ format_duration(
 mrdocs::Expected<std::unique_ptr<Corpus>>
 CorpusImpl::
 build(
-    report::Level reportLevel,
     std::shared_ptr<ConfigImpl const> const& config,
     tooling::CompilationDatabase const& compilations)
 {
@@ -159,7 +158,7 @@ build(
     // ------------------------------------------
     // Traverse the AST for all translation units.
     // This operation happens on a thread pool.
-    report::print(reportLevel, "Extracting declarations");
+    report::info("Extracting declarations");
 
     // Get a copy of the filename strings
     std::vector<std::string> files = compilations.getAllFiles();
@@ -187,15 +186,14 @@ build(
             taskGroup.async(
             [&, idx = ++index, path = std::move(file)]()
             {
-                report::log(reportLevel,
-                    "[{}/{}] \"{}\"", idx, files.size(), path);
+                report::debug("[{}/{}] \"{}\"", idx, files.size(), path);
                 processFile(path);
             });
         }
         errors = taskGroup.wait();
     }
     // Print diagnostics totals
-    context.reportEnd(reportLevel);
+    context.reportEnd(report::Level::info);
 
     // ------------------------------------------
     // Report warning and error totals
@@ -216,7 +214,7 @@ build(
         return Unexpected(results.error());
     corpus->info_ = std::move(results.value());
 
-    report::log(reportLevel,
+    report::info(
         "Extracted {} declarations in {}",
         corpus->info_.size(),
         format_duration(clock_type::now() - start_time));

src/lib/Lib/CorpusImpl.hpp

@@ -79,7 +79,6 @@ class CorpusImpl : public Corpus
         instance will be already populated. They will
         not need to call this function directly.
 
-        @param reportLevel Error reporting level.
         @param config A shared pointer to the configuration.
         @param compilations A compilations database for the input files.
     */
@@ -88,7 +87,6 @@ class CorpusImpl : public Corpus
     static
     mrdocs::Expected<std::unique_ptr<Corpus>>
     build(
-        report::Level reportLevel,
         std::shared_ptr<ConfigImpl const> const& config,
         tooling::CompilationDatabase const& compilations);
 

src/test/TestRunner.cpp

@@ -112,9 +112,9 @@ handleFile(
     MrDocsCompilationDatabase compilations(
         llvm::StringRef(parentDir), SingleFileDB(filePath), config, defaultIncludePaths);
 
+    report::setMinimumLevel(report::Level::warn);
     // Build Corpus
-    auto corpus = CorpusImpl::build(
-        report::Level::debug, config, compilations);
+    auto corpus = CorpusImpl::build(config, compilations);
     if(! corpus)
         return report::error("{}: \"{}\"", corpus.error(), filePath);
 

src/tool/GenerateAction.cpp

@@ -186,8 +186,7 @@ DoGenerateAction(
     // --------------------------------------------------------------
     MRDOCS_TRY(
         std::unique_ptr<Corpus> corpus,
-        CorpusImpl::build(
-            report::Level::info, config, compilationDatabase));
+        CorpusImpl::build(config, compilationDatabase));
     if (corpus->empty())
     {
         report::warn("Corpus is empty, not generating docs");