From d852563b003ad7bebc7c0664b0044568907676ef Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?M=C3=A1ty=C3=A1s=20Mustoha?= <matyas.mustoha@gmail.com>
Date: Mon, 21 Mar 2022 10:39:03 +0100
Subject: [PATCH] Limit the number of printed metadata file errors

---
 .../pegasus_metadata/PegasusMetadata.cpp      | 25 +++++++++++++------
 .../pegasus_metadata/PegasusMetadata.h        |  7 +++---
 2 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/src/backend/providers/pegasus_metadata/PegasusMetadata.cpp b/src/backend/providers/pegasus_metadata/PegasusMetadata.cpp
index 769eaefa3..585af2a53 100644
--- a/src/backend/providers/pegasus_metadata/PegasusMetadata.cpp
+++ b/src/backend/providers/pegasus_metadata/PegasusMetadata.cpp
@@ -33,6 +33,8 @@
 
 
 namespace {
+constexpr size_t ISSUE_LOG_LIMIT = 100;
+
 QStringList tokenize_by_comma(const QString& str)
 {
     QStringList list = str.split(QLatin1Char(','), Qt::SkipEmptyParts);
@@ -162,19 +164,25 @@ Metadata::Metadata(QString log_tag)
     , rx_uri(QStringLiteral("^[a-zA-Z][a-zA-Z0-9+\\-.]*:.+"))
 {}
 
-void Metadata::print_error(const ParserState& ps, const metafile::Error& err) const
+void Metadata::print_error(ParserState& ps, const metafile::Error& err) const
 {
-    Log::error(m_log_tag, LOGMSG("`%1`, line %2: %3")
-        .arg(::pretty_path(ps.path), QString::number(err.line), err.message));
+    ps.found_issues++;
+    if (ps.found_issues <= ISSUE_LOG_LIMIT) {
+        Log::error(m_log_tag, LOGMSG("`%1`, line %2: %3")
+            .arg(::pretty_path(ps.path), QString::number(err.line), err.message));
+    }
 }
 
-void Metadata::print_warning(const ParserState& ps, const metafile::Entry& entry, const QString& msg) const
+void Metadata::print_warning(ParserState& ps, const metafile::Entry& entry, const QString& msg) const
 {
-    Log::warning(m_log_tag, LOGMSG("`%1`, line %2: %3")
-        .arg(::pretty_path(ps.path), QString::number(entry.line), msg));
+    ps.found_issues++;
+    if (ps.found_issues <= ISSUE_LOG_LIMIT) {
+        Log::warning(m_log_tag, LOGMSG("`%1`, line %2: %3")
+            .arg(::pretty_path(ps.path), QString::number(entry.line), msg));
+    }
 }
 
-const QString& Metadata::first_line_of(const ParserState& ps, const metafile::Entry& entry) const
+const QString& Metadata::first_line_of(ParserState& ps, const metafile::Entry& entry) const
 {
     Q_ASSERT(!entry.key.isEmpty());
     Q_ASSERT(!entry.values.empty());
@@ -551,6 +559,9 @@ std::vector<FileFilter> Metadata::apply_metafile(const QString& metafile_path, S
         Log::error(m_log_tag, LOGMSG("Failed to read metadata file `%1`")
             .arg(::pretty_path(metafile_path)));
     }
+    if (ps.found_issues > ISSUE_LOG_LIMIT) {
+        Log::warning(m_log_tag, LOGMSG("%1 other issues omitted").arg(QString::number(ps.found_issues - ISSUE_LOG_LIMIT)));
+    }
 
     return std::move(ps.filters);
 }
diff --git a/src/backend/providers/pegasus_metadata/PegasusMetadata.h b/src/backend/providers/pegasus_metadata/PegasusMetadata.h
index b1bcaa8ae..8dfeae022 100644
--- a/src/backend/providers/pegasus_metadata/PegasusMetadata.h
+++ b/src/backend/providers/pegasus_metadata/PegasusMetadata.h
@@ -46,6 +46,7 @@ struct ParserState {
     model::Collection* cur_coll = nullptr;
     std::vector<FileFilter> filters;
     std::vector<model::Collection*> all_colls;
+    size_t found_issues = 0;
 
     explicit ParserState(const QString&);
     NO_COPY_NO_MOVE(ParserState)
@@ -77,10 +78,10 @@ class Metadata {
     const QRegularExpression rx_uri;
 
 
-    void print_error(const ParserState& ps, const metafile::Error&) const;
-    void print_warning(const ParserState& ps, const metafile::Entry&, const QString&) const;
+    void print_error(ParserState&, const metafile::Error&) const;
+    void print_warning(ParserState&, const metafile::Entry&, const QString&) const;
 
-    const QString& first_line_of(const ParserState& ps, const metafile::Entry&) const;
+    const QString& first_line_of(ParserState&, const metafile::Entry&) const;
     void replace_newlines(QString&) const;
 
     void apply_collection_entry(ParserState&, const metafile::Entry&) const;