From a848477d1a1f75b49db7227186ac64968cdae498 Mon Sep 17 00:00:00 2001 From: crazyhzm Date: Fri, 26 Apr 2024 23:26:17 +0800 Subject: [PATCH] Simplified redundant judgment Signed-off-by: crazyhzm --- .../org/apache/maven/cli/CleanArgument.java | 19 +++---------------- 1 file changed, 3 insertions(+), 16 deletions(-) diff --git a/maven-embedder/src/main/java/org/apache/maven/cli/CleanArgument.java b/maven-embedder/src/main/java/org/apache/maven/cli/CleanArgument.java index c725faa8c5ad..ad4566dd4710 100644 --- a/maven-embedder/src/main/java/org/apache/maven/cli/CleanArgument.java +++ b/maven-embedder/src/main/java/org/apache/maven/cli/CleanArgument.java @@ -47,24 +47,11 @@ public static String[] cleanArgs(String[] args) { // this has to be a separate "if" statement, to capture the case of: "-Dfoo=bar" if (addedToBuffer && arg.endsWith("\"")) { - String cleanArgPart = arg.substring(0, arg.length() - 1); - // if we're building an argument, keep doing so. - if (currentArg != null) { - // if this is the case of "-Dfoo=bar", then we need to adjust the buffer. - if (addedToBuffer) { - currentArg.setLength(currentArg.length() - 1); - } - // otherwise, we trim the trailing " and append to the buffer. - else { - // TODO introducing a space here...not sure what else to do but collapse whitespace - currentArg.append(' ').append(cleanArgPart); - } + // if this is the case of "-Dfoo=bar", then we need to adjust the buffer. + currentArg.setLength(currentArg.length() - 1); - cleaned.add(currentArg.toString()); - } else { - cleaned.add(cleanArgPart); - } + cleaned.add(currentArg.toString()); currentArg = null; addedToBuffer = false;