[Docutils-develop] [PATCH 17/39] py3: Replace 'types.ClassType' with type, 'types.SliceType' with slice

[Stephen F. <st...@th...>]

These have been removed in Python 3, but in Python 2.7 they were only
aliases anyway. Switch them out.

Signed-off-by: Stephen Finucane <st...@th...ru>
---
 docutils/docutils/nodes.py        | 11 +++++------
 docutils/docutils/statemachine.py |  5 ++---
 docutils/test/test_nodes.py       |  4 ++--
 3 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/docutils/docutils/nodes.py b/docutils/docutils/nodes.py
index 01f32238d..cc8526e0f 100644
--- a/docutils/docutils/nodes.py
+++ b/docutils/docutils/nodes.py
@@ -27,7 +27,6 @@ import sys
 import os
 import re
 import warnings
-import types
 import unicodedata
 
 if sys.version_info >= (3, 0):
@@ -257,11 +256,11 @@ class Node(object):
         if include_self and descend and not siblings:
             if condition is None:
                 return self._all_traverse()
-            elif isinstance(condition, (types.ClassType, type)):
+            elif isinstance(condition, type):
                 return self._fast_traverse(condition)
         # Check if `condition` is a class (check for TypeType for Python
         # implementations that use only new-style classes, like PyPy).
-        if isinstance(condition, (types.ClassType, type)):
+        if isinstance(condition, type):
             node_class = condition
             def condition(node, node_class=node_class):
                 return isinstance(node, node_class)
@@ -590,7 +589,7 @@ class Element(Node):
             return self.attributes[key]
         elif isinstance(key, int):
             return self.children[key]
-        elif isinstance(key, types.SliceType):
+        elif isinstance(key, slice):
             assert key.step in (None, 1), 'cannot handle slice with stride'
             return self.children[key.start:key.stop]
         else:
@@ -603,7 +602,7 @@ class Element(Node):
         elif isinstance(key, int):
             self.setup_child(item)
             self.children[key] = item
-        elif isinstance(key, types.SliceType):
+        elif isinstance(key, slice):
             assert key.step in (None, 1), 'cannot handle slice with stride'
             for node in item:
                 self.setup_child(node)
@@ -617,7 +616,7 @@ class Element(Node):
             del self.attributes[key]
         elif isinstance(key, int):
             del self.children[key]
-        elif isinstance(key, types.SliceType):
+        elif isinstance(key, slice):
             assert key.step in (None, 1), 'cannot handle slice with stride'
             del self.children[key.start:key.stop]
         else:
diff --git a/docutils/docutils/statemachine.py b/docutils/docutils/statemachine.py
index 24c54cb32..9579a6a57 100644
--- a/docutils/docutils/statemachine.py
+++ b/docutils/docutils/statemachine.py
@@ -109,7 +109,6 @@ __docformat__ = 'restructuredtext'
 
 import sys
 import re
-import types
 import unicodedata
 from docutils import utils
 from docutils.utils.error_reporting import ErrorOutput
@@ -1145,7 +1144,7 @@ class ViewList(object):
     # just works.
 
     def __getitem__(self, i):
-        if isinstance(i, types.SliceType):
+        if isinstance(i, slice):
             assert i.step in (None, 1),  'cannot handle slice with stride'
             return self.__class__(self.data[i.start:i.stop],
                                   items=self.items[i.start:i.stop],
@@ -1154,7 +1153,7 @@ class ViewList(object):
             return self.data[i]
 
     def __setitem__(self, i, item):
-        if isinstance(i, types.SliceType):
+        if isinstance(i, slice):
             assert i.step in (None, 1), 'cannot handle slice with stride'
             if not isinstance(item, ViewList):
                 raise TypeError('assigning non-ViewList to ViewList slice')
diff --git a/docutils/test/test_nodes.py b/docutils/test/test_nodes.py
index 84da41f2c..e0c7e0da8 100755
--- a/docutils/test/test_nodes.py
+++ b/docutils/test/test_nodes.py
@@ -11,7 +11,7 @@ Test module for nodes.py.
 
 import sys
 import unittest
-import types
+
 import DocutilsTestSupport              # must be imported before docutils
 from DocutilsTestSupport import nodes, utils
 
@@ -359,7 +359,7 @@ class MiscTests(unittest.TestCase):
         node_class_names = []
         for x in dir(nodes):
             c = getattr(nodes, x)
-            if isinstance(c, (type, types.ClassType)) and \
+            if isinstance(c, type) and \
                    issubclass(c, nodes.Node) and len(c.__bases__) > 1:
                 node_class_names.append(x)
         node_class_names.sort()
-- 
2.21.0