diff --git a/pmd/etc/changelog.txt b/pmd/etc/changelog.txt index 04db7a3aef..d1ef5580ec 100644 --- a/pmd/etc/changelog.txt +++ b/pmd/etc/changelog.txt @@ -7,6 +7,7 @@ Fixed bug 1043: node.getEndLine() always returns 0 (ECMAscript) Fixed bug 1044: Unknown option: -excludemarker Fixed bug 1047: False Positive in 'for' loops for LocalVariableCouldBeFinal in 5.0.1 Fixed bug 1048: CommentContent Rule, String Index out of range Exception +Fixed bug 1055: Please add a colon in the ant output after line,column for Oracle JDeveloper IDE usage Fixed bug 1059: Change rule name "Use Singleton" should be "Use Utility class" Fixed bug 1060: GodClassRule >>> wrong method diff --git a/pmd/src/main/java/net/sourceforge/pmd/renderers/TextRenderer.java b/pmd/src/main/java/net/sourceforge/pmd/renderers/TextRenderer.java index f606244b29..c938d5eb4f 100644 --- a/pmd/src/main/java/net/sourceforge/pmd/renderers/TextRenderer.java +++ b/pmd/src/main/java/net/sourceforge/pmd/renderers/TextRenderer.java @@ -45,7 +45,7 @@ public class TextRenderer extends AbstractIncrementingRenderer { RuleViolation rv = violations.next(); buf.append(rv.getFilename()); buf.append(':').append(Integer.toString(rv.getBeginLine())); - buf.append('\t').append(rv.getDescription()).append(PMD.EOL); + buf.append(":\t").append(rv.getDescription()).append(PMD.EOL); writer.write(buf.toString()); } } diff --git a/pmd/src/test/java/net/sourceforge/pmd/renderers/TextRendererTest.java b/pmd/src/test/java/net/sourceforge/pmd/renderers/TextRendererTest.java index 21940e529f..cbff54e6fc 100644 --- a/pmd/src/test/java/net/sourceforge/pmd/renderers/TextRendererTest.java +++ b/pmd/src/test/java/net/sourceforge/pmd/renderers/TextRendererTest.java @@ -12,7 +12,7 @@ public class TextRendererTest extends AbstractRendererTst { @Override public String getExpected() { - return "n/a:1\tmsg" + PMD.EOL; + return "n/a:1:\tmsg" + PMD.EOL; } @Override @@ -22,7 +22,7 @@ public class TextRendererTest extends AbstractRendererTst { @Override public String getExpectedMultiple() { - return "n/a:1\tmsg" + PMD.EOL + "n/a:1\tmsg" + PMD.EOL; + return "n/a:1:\tmsg" + PMD.EOL + "n/a:1:\tmsg" + PMD.EOL; } @Override