Only print report:exclude debug statement for files with unused decla…

…rations. (#382)
peripheryapp · Jul 23, 2021 · 699ffa4 · 699ffa4
1 parent bb67c0f
commit 699ffa4
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 8 deletions.
diff --git a/Sources/Frontend/Formatters/OutputDeclarationFilter.swift b/Sources/Frontend/Formatters/OutputDeclarationFilter.swift
@@ -1,4 +1,5 @@
 import Foundation
+import SystemPackage
 import Shared
 import PeripheryKit
 
@@ -18,12 +19,21 @@ final class OutputDeclarationFilter: Injectable {
     func filter(_ declarations: [ScanResult]) -> [ScanResult] {
         let excludedSourceFiles = configuration.reportExcludeSourceFiles
 
-        excludedSourceFiles.forEach {
-            logger.debug("[report:exclude] \($0.string)")
-        }
+        var reportedExclusions: Set<FilePath> = []
 
         return declarations.filter {
-            !excludedSourceFiles.contains($0.declaration.location.file.path)
+            let path = $0.declaration.location.file.path
+
+            if excludedSourceFiles.contains(path) {
+                if !reportedExclusions.contains(path) {
+                    self.logger.debug("[report:exclude] \(path)")
+                    reportedExclusions.insert(path)
+                }
+
+                return false
+            }
+
+            return true
         }
     }
 }
diff --git a/Sources/Shared/Configuration.swift b/Sources/Shared/Configuration.swift
@@ -277,13 +277,13 @@ public final class Configuration: Singleton {
         }
     }
 
-    public var indexExcludeSourceFiles: Set<FilePath> {
+    public lazy var indexExcludeSourceFiles: Set<FilePath> = {
         Set(indexExclude.flatMap { FilePath.glob($0) })
-    }
+    }()
 
-    public var reportExcludeSourceFiles: Set<FilePath> {
+    public lazy var reportExcludeSourceFiles: Set<FilePath> = {
         Set(reportExclude.flatMap { FilePath.glob($0) })
-    }
+    }()
 
     // MARK: - Private