Author: olamy
Date: Mon Dec 31 14:24:12 2012
New Revision: 1427137

URL: http://svn.apache.org/viewvc?rev=1427137&view=rev
Log:
use indexOf(char)

Modified:
    
archiva/redback/redback-components/trunk/spring-quartz/src/main/java/org/apache/archiva/redback/components/scheduler/CronExpressionValidator.java

Modified: 
archiva/redback/redback-components/trunk/spring-quartz/src/main/java/org/apache/archiva/redback/components/scheduler/CronExpressionValidator.java
URL: 
http://svn.apache.org/viewvc/archiva/redback/redback-components/trunk/spring-quartz/src/main/java/org/apache/archiva/redback/components/scheduler/CronExpressionValidator.java?rev=1427137&r1=1427136&r2=1427137&view=diff
==============================================================================
--- 
archiva/redback/redback-components/trunk/spring-quartz/src/main/java/org/apache/archiva/redback/components/scheduler/CronExpressionValidator.java
 (original)
+++ 
archiva/redback/redback-components/trunk/spring-quartz/src/main/java/org/apache/archiva/redback/components/scheduler/CronExpressionValidator.java
 Mon Dec 31 14:24:12 2012
@@ -192,15 +192,15 @@ public final class CronExpressionValidat
             return true;
         }
 
-        if ( dayOfWeekField.indexOf( "L" ) >= 0 )
+        if ( dayOfWeekField.indexOf( 'L' ) >= 0 )
         {
             return checkFieldWithLetter( dayOfWeekField, "L", 1, 7, -1, -1 );
         }
-        else if ( dayOfWeekField.indexOf( "C" ) >= 0 )
+        else if ( dayOfWeekField.indexOf( 'C' ) >= 0 )
         {
             return checkFieldWithLetter( dayOfWeekField, "C", 1, 7, -1, -1 );
         }
-        else if ( dayOfWeekField.indexOf( "#" ) >= 0 )
+        else if ( dayOfWeekField.indexOf( '#' ) >= 0 )
         {
             return checkFieldWithLetter( dayOfWeekField, "#", 1, 7, 1, 5 );
         }
@@ -217,7 +217,7 @@ public final class CronExpressionValidat
 
     private boolean checkField( String secondsField, int minimal, int maximal )
     {
-        if ( secondsField.indexOf( "-" ) > -1 )
+        if ( secondsField.indexOf( '-' ) > -1 )
         {
             String startValue = secondsField.substring( 0, 
secondsField.indexOf( "-" ) );
             String endValue = secondsField.substring( secondsField.indexOf( 
"-" ) + 1 );
@@ -239,15 +239,15 @@ public final class CronExpressionValidat
                 return false;
             }
         }
-        else if ( secondsField.indexOf( "," ) > -1 )
+        else if ( secondsField.indexOf( ',' ) > -1 )
         {
             return checkListField( secondsField, minimal, maximal );
         }
-        else if ( secondsField.indexOf( "/" ) > -1 )
+        else if ( secondsField.indexOf( '/' ) > -1 )
         {
             return checkIncrementField( secondsField, minimal, maximal );
         }
-        else if ( secondsField.indexOf( "*" ) != -1 )
+        else if ( secondsField.indexOf( '*' ) != -1 )
         {
             return true;
         }
@@ -357,9 +357,9 @@ public final class CronExpressionValidat
 
     private boolean checkIncrementField( String value, int minimal, int 
maximal )
     {
-        String start = value.substring( 0, value.indexOf( "/" ) );
+        String start = value.substring( 0, value.indexOf( '/' ) );
 
-        String increment = value.substring( value.indexOf( "/" ) + 1 );
+        String increment = value.substring( value.indexOf( '/' ) + 1 );
 
         if ( !"*".equals( start ) )
         {


Reply via email to