Merge pull request adafruit#675 from Rosuav/seq_simplify

pfalcon · pfalcon · commit 62f7ba7a815f · 2014-06-09T23:40:04.000+03:00
Remove unnecessary bounds check from mp_seq_get_fast_slice_indexes.
diff --git a/py/sequence.c b/py/sequence.c
@@ -80,10 +80,6 @@ bool mp_seq_get_fast_slice_indexes(machine_uint_t len, mp_obj_t slice, mp_bound_
     }
     if (stop < 0) {
         stop = len + stop;
-        // CPython returns empty sequence in such case
-        if (stop < 0) {
-            stop = start;
-        }
     } else if (stop > len) {
         stop = len;
     }

Original file line number	Diff line number	Diff line change
`@@ -80,10 +80,6 @@ bool mp_seq_get_fast_slice_indexes(machine_uint_t len, mp_obj_t slice, mp_bound_`
`80`	`80`	`}`
`81`	`81`	`if (stop < 0) {`
`82`	`82`	`stop = len + stop;`
`83`		`- // CPython returns empty sequence in such case`
`84`		`- if (stop < 0) {`
`85`		`- stop = start;`
`86`		`- }`
`87`	`83`	`} else if (stop > len) {`
`88`	`84`	`stop = len;`
`89`	`85`	`}`