Fix for problem reported by Neal Norwitz. Tighten up calculation of
slicelength. Include his test case.
diff --git a/Objects/sliceobject.c b/Objects/sliceobject.c
index 7499d31..9a268b7 100644
--- a/Objects/sliceobject.c
+++ b/Objects/sliceobject.c
@@ -151,13 +151,15 @@
if (*stop < 0) *stop = -1;
if (*stop > length) *stop = length;
}
-
- if (*step < 0) {
+
+ if ((*stop - *start)*(*step) <= 0) {
+ *slicelength = 0;
+ }
+ else if (*step < 0) {
*slicelength = (*stop-*start+1)/(*step)+1;
} else {
*slicelength = (*stop-*start-1)/(*step)+1;
}
- if (*slicelength < 0) *slicelength = 0;
return 0;
}