Fix range-with-step check for boundary condition (#6)

Author: varunbpatil

gronx_test.go

@@ -129,6 +129,7 @@ func testcases() []Case {
 		{"0 0 * * 2-7", "2011-06-18 23:09:00", false},
 		{"0 0 * * 4-7", "2011-07-19 00:00:00", false},
 		{"0-12/4 * * * *", "2011-06-20 12:04:00", true},
+		{"0-10/2 * * * *", "2011-06-20 12:12:00", false},
 		{"4-59/2 * * * *", "2011-06-20 12:04:00", true},
 		{"4-59/2 * * * *", "2011-06-20 12:06:00", true},
 		{"4-59/3 * * * *", "2011-06-20 12:06:00", false},

validator.go

@@ -53,16 +53,12 @@ func inRange(val int, s string) (bool, error) {
 }
 
 func inStepRange(val, start, end, step int) bool {
-	for {
-		if start == val {
+	for i := start; i <= end && i <= val; i += step {
+		if i == val {
 			return true
 		}
-		if start > end {
-			return false
-		}
-
-		start += step
 	}
+	return false
 }
 
 func isValidMonthDay(val string, last int, ref time.Time) (bool, error) {