This is an automated email from the ASF dual-hosted git repository.

gnodet pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven.git


The following commit(s) were added to refs/heads/master by this push:
     new 3c65873b58 [MNG-7970] Remove extra space in MavenSimpleLogger stack 
trace formatter (#1342)
3c65873b58 is described below

commit 3c65873b584945c034aeb4ba51a5793c6582b19d
Author: Michael Bien <[email protected]>
AuthorDate: Fri Dec 15 07:39:42 2023 +0100

    [MNG-7970] Remove extra space in MavenSimpleLogger stack trace formatter 
(#1342)
    
    Regular formatted stack traces do not have a space before the file/line
    section.
    
    before:
    at java.util.zip.ZipFile$Source.get (ZipFile.java:1426)
    
    after:
    at java.util.zip.ZipFile$Source.get(ZipFile.java:1426)
---
 .../src/main/java/org/slf4j/impl/MavenSimpleLogger.java                 | 2 +-
 .../src/test/java/org/slf4j/impl/MavenSimpleLoggerTest.java             | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLogger.java 
b/maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLogger.java
index 12a871cc63..988b2239bc 100644
--- a/maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLogger.java
+++ b/maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLogger.java
@@ -83,7 +83,7 @@ public class MavenSimpleLogger extends SimpleLogger {
             builder.a(e.getClassName());
             builder.a(".");
             builder.a(e.getMethodName());
-            builder.a(" (");
+            builder.a("(");
             builder.strong(getLocation(e));
             builder.a(")");
             stream.println(builder);
diff --git 
a/maven-slf4j-provider/src/test/java/org/slf4j/impl/MavenSimpleLoggerTest.java 
b/maven-slf4j-provider/src/test/java/org/slf4j/impl/MavenSimpleLoggerTest.java
index da2d0b041d..b51decb6b9 100644
--- 
a/maven-slf4j-provider/src/test/java/org/slf4j/impl/MavenSimpleLoggerTest.java
+++ 
b/maven-slf4j-provider/src/test/java/org/slf4j/impl/MavenSimpleLoggerTest.java
@@ -51,7 +51,7 @@ class MavenSimpleLoggerTest {
 
         Class<?> testClass = testInfo.getTestClass().get();
         String testMethodName = testInfo.getTestMethod().get().getName();
-        String testClassStackTraceLinePattern = "at " + testClass.getName() + 
"." + testMethodName + " \\("
+        String testClassStackTraceLinePattern = "at " + testClass.getName() + 
"." + testMethodName + "\\("
                 + testClass.getSimpleName() + ".java:\\d+\\)";
         List<String> expectedLines = Arrays.asList(
                 "java.lang.RuntimeException: top-level",

Reply via email to