Merge pull request #98 from jamezp/LOGMGR-141

[LOGMGR-141] Fix ending deletion length when truncating format parts.
jboss-logging · Jun 28, 2016 · ef5f5f4 · ef5f5f4
2 parents 521ea2f + c728e14
commit ef5f5f4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/main/java/org/jboss/logmanager/formatters/Formatters.java b/src/main/java/org/jboss/logmanager/formatters/Formatters.java
@@ -210,7 +210,7 @@ public void render(final StringBuilder builder, final ExtLogRecord record) {
                 final int overflow = writtenLen - maximumWidth;
                 if (overflow > 0) {
                     if (truncateBeginning) {
-                        builder.delete(oldLen, overflow + 1);
+                        builder.delete(oldLen, oldLen + overflow);
                     }
                     builder.setLength(newLen - overflow);
                 } else {

diff --git a/src/test/java/org/jboss/logmanager/formatters/PatternFormatterTests.java b/src/test/java/org/jboss/logmanager/formatters/PatternFormatterTests.java
@@ -184,6 +184,11 @@ public void truncation() throws Exception {
 
         formatter = new PatternFormatter("%-5.10m");
         Assert.assertEquals("test ", formatter.format(record));
+
+        // Exact length truncation
+        final String msg = "test message";
+        formatter = new PatternFormatter("%c %-5.-7m");
+        Assert.assertEquals(CATEGORY + " message", formatter.format(createLogRecord(msg)));
     }
 
     @Test