Remove use of redundant String.substring() parameter.
diff --git a/src/main/java/org/apache/commons/ognl/ASTAdd.java b/src/main/java/org/apache/commons/ognl/ASTAdd.java
index f0d7777..b7aaae8 100644
--- a/src/main/java/org/apache/commons/ognl/ASTAdd.java
+++ b/src/main/java/org/apache/commons/ognl/ASTAdd.java
@@ -215,7 +215,7 @@
// dirty fix for overly aggressive casting dot operations
if ( rootExpr.endsWith( "." ) && chain != null && chain.startsWith( ")." ) )
{
- chain = chain.substring( 1, chain.length() );
+ chain = chain.substring( 1 );
}
expr = rootExpr + ( chain != null ? chain + "." : "" ) + expr;
@@ -235,7 +235,7 @@
if ( !ASTProperty.class.isInstance( children[i].jjtGetChild( 0 ) ) && rootExpr.endsWith( ")" )
&& expr.startsWith( ")" ) )
{
- expr = expr.substring( 1, expr.length() );
+ expr = expr.substring( 1 );
}
expr = rootExpr + expr;
diff --git a/src/test/java/org/apache/commons/ognl/internal/ConcurrentHashMapCacheTest.java b/src/test/java/org/apache/commons/ognl/internal/ConcurrentHashMapCacheTest.java
index 9614c34..a7861d1 100644
--- a/src/test/java/org/apache/commons/ognl/internal/ConcurrentHashMapCacheTest.java
+++ b/src/test/java/org/apache/commons/ognl/internal/ConcurrentHashMapCacheTest.java
@@ -89,7 +89,7 @@
for ( Method method : methods )
{
String name = method.getName( );
- boolean isGet = name.substring( 3, name.length( ) ).equalsIgnoreCase( key.getMethodName( ) );
+ boolean isGet = name.substring( 3 ).equalsIgnoreCase( key.getMethodName( ) );
if ( isGet )
{
list.add( method );
