commit 1ec2fcd16e3c28e1532829d9a4f2cf29f8ebf559
author Mark Dickinson <dickinsm@gmail.com> Fri Jun 20 14:53:43 2008 +0000
committer Mark Dickinson <dickinsm@gmail.com> Fri Jun 20 14:53:43 2008 +0000
tree 2acd565c41af554d9a7ab7b60630d7d456b902d5
parent 7b2e2df7406e8d6d29e98388964145b9a5de8316

Issue #3004:  Minor fix to slice.indices().  slice(-10).indices(9) now
returns (0, 0, 1) instead of (0, -1, 1), and slice(None, 10, -1).indices(10)
returns (9, 9, -1) instead of (9, 10, -1).

Lib/test/test_slice.py

diff --git a/Lib/test/test_slice.py b/Lib/test/test_slice.py
index 83f051f..8c90c10 100644
--- a/Lib/test/test_slice.py
+++ b/Lib/test/test_slice.py
@@ -79,6 +79,20 @@
         self.assertEqual(slice(None,  None, -1).indices(10), (9, -1, -1))
         self.assertEqual(slice(None,  None, -2).indices(10), (9, -1, -2))
         self.assertEqual(slice(3,     None, -2).indices(10), (3, -1, -2))
+        # issue 3004 tests
+        self.assertEqual(slice(None, -9).indices(10), (0, 1, 1))
+        self.assertEqual(slice(None, -10).indices(10), (0, 0, 1))
+        self.assertEqual(slice(None, -11).indices(10), (0, 0, 1))
+        self.assertEqual(slice(None, -10, -1).indices(10), (9, 0, -1))
+        self.assertEqual(slice(None, -11, -1).indices(10), (9, -1, -1))
+        self.assertEqual(slice(None, -12, -1).indices(10), (9, -1, -1))
+        self.assertEqual(slice(None, 9).indices(10), (0, 9, 1))
+        self.assertEqual(slice(None, 10).indices(10), (0, 10, 1))
+        self.assertEqual(slice(None, 11).indices(10), (0, 10, 1))
+        self.assertEqual(slice(None, 8, -1).indices(10), (9, 8, -1))
+        self.assertEqual(slice(None, 9, -1).indices(10), (9, 9, -1))
+        self.assertEqual(slice(None, 10, -1).indices(10), (9, 9, -1))
+
         self.assertEqual(
             slice(-100,  100     ).indices(10),
             slice(None).indices(10)