Update changelog #889

MaibornWolff · Dec 11, 2023 · ffbd8a7 · ffbd8a7
1 parent ae47ae2
commit ffbd8a7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,7 +10,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/)
 ### Changed
 
 -   Multiple values for options need to be separated by comma [#3434](https://github.com/MaibornWolff/codecharta/pull/3434)
--   Parameters and options can be specified in any order [#3434](https://github.com/MaibornWolff/codecharta/pull/3434)
+-   Changed the short-form of the `--no-issues` flag in the SourceCodeParser from `-i` to `-ni` [#3434](https://github.com/MaibornWolff/codecharta/pull/3434)
 
 ## [1.121.0] - 2023-12-07
 

diff --git a/...el/src/main/kotlin/de/maibornwolff/codecharta/util/CommaSeparatedParameterPreprocessor.kt b/...el/src/main/kotlin/de/maibornwolff/codecharta/util/CommaSeparatedParameterPreprocessor.kt
@@ -7,15 +7,16 @@ class CommaSeparatedParameterPreprocessor: CommandLine.IParameterPreprocessor {
     override fun preprocess(args: Stack<String>?, commandSpec: CommandLine.Model.CommandSpec?, argSpec: CommandLine.Model.ArgSpec?, info: MutableMap<String, Any>?): Boolean {
         if (args == null) return false
 
-        var argsList = ""
+        val stringBuilder = StringBuilder()
         while (args.peek() != null) {
-            val currentArg = args.pop()
-            argsList += currentArg
-            if (!currentArg.matches(Regex(".*,\\s*$"))) {
+            val arg = args.pop()
+            stringBuilder.append(arg)
+            if (!arg.matches(Regex(".*,\\s*$"))) {
                 break
             }
         }
-        args.push(argsList)
+
+        args.push(stringBuilder.toString())
         return false
     }
 }