chore(opossum): Prefer add functions when building collections

Avoid any confusion what `this` refers to for these (extension) functions. Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
oss-review-toolkit · Dec 16, 2024 · 03d9166 · 03d9166
1 parent 6db8eae
commit 03d9166
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/model/src/main/kotlin/utils/SortedCollectionConverters.kt b/model/src/main/kotlin/utils/SortedCollectionConverters.kt
@@ -105,16 +105,16 @@ class SnippetFindingSortedSetConverter : StdConverter<Set<SnippetFinding>, Sorte
 
 private fun Provenance.getSortKey(): String =
     buildList {
-        this += javaClass.canonicalName
+        add(javaClass.canonicalName)
 
         when (this@getSortKey) {
             is RepositoryProvenance -> {
-                this += vcsInfo.type.toString()
-                this += vcsInfo.url
+                add(vcsInfo.type.toString())
+                add(vcsInfo.url)
             }
 
             is ArtifactProvenance -> {
-                this += sourceArtifact.url
+                add(sourceArtifact.url)
             }
 
             else -> {

diff --git a/plugins/package-managers/conan/src/main/kotlin/Conan.kt b/plugins/package-managers/conan/src/main/kotlin/Conan.kt
@@ -314,7 +314,7 @@ class Conan(
                     val dependencies = parseDependencyTree(pkgInfos, pkgInfo.requires, workingDir) +
                         parseDependencyTree(pkgInfos, pkgInfo.buildRequires, workingDir)
 
-                    this += PackageReference(id, dependencies = dependencies)
+                    add(PackageReference(id, dependencies = dependencies))
                 }
             }
         }

diff --git a/plugins/reporters/opossum/src/main/kotlin/OpossumReporter.kt b/plugins/reporters/opossum/src/main/kotlin/OpossumReporter.kt
@@ -443,6 +443,6 @@ class OpossumReporter(
 private fun DependencyNode.getDependencies(): List<DependencyNode> =
     buildList {
         visitDependencies { dependencyNodes ->
-            this += dependencyNodes.map { it.getStableReference() }
+            addAll(dependencyNodes.map { it.getStableReference() })
         }
     }