[Cleanup] Redundant substring() operation Removes the second substring() parameter if this parameter is the length of the string. It's the default value. Found during testing of Bug 569800 (new JDT cleanup for this case). Change-Id: Id5d925b4436951383b8c69b807e9afd47d769d76 Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>
diff --git a/tests/org.eclipse.core.tests.runtime/src/org/eclipse/core/tests/internal/runtime/PlatformLogReader.java b/tests/org.eclipse.core.tests.runtime/src/org/eclipse/core/tests/internal/runtime/PlatformLogReader.java index ffd8478..b7db1e7 100644 --- a/tests/org.eclipse.core.tests.runtime/src/org/eclipse/core/tests/internal/runtime/PlatformLogReader.java +++ b/tests/org.eclipse.core.tests.runtime/src/org/eclipse/core/tests/internal/runtime/PlatformLogReader.java
@@ -245,7 +245,7 @@ return ""; } StringBuilder buffer = new StringBuilder(); - buffer.append(currentLine.substring(KEYWORD_MESSAGE.length() + 1, currentLine.length())); + buffer.append(currentLine.substring(KEYWORD_MESSAGE.length() + 1)); currentLine = reader.readLine(); buffer.append(readText()); return buffer.toString();