[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();
