diff options
Diffstat (limited to 'Lib/lib2to3/fixes/fix_operator.py')
| -rw-r--r-- | Lib/lib2to3/fixes/fix_operator.py | 32 |
1 files changed, 17 insertions, 15 deletions
diff --git a/Lib/lib2to3/fixes/fix_operator.py b/Lib/lib2to3/fixes/fix_operator.py index 7bf2c0dd22c..1aa17bae58d 100644 --- a/Lib/lib2to3/fixes/fix_operator.py +++ b/Lib/lib2to3/fixes/fix_operator.py @@ -9,6 +9,8 @@ operator.repeat(obj, n) -> operator.mul(obj, n) operator.irepeat(obj, n) -> operator.imul(obj, n) """ +import collections + # Local imports from lib2to3 import fixer_base from lib2to3.fixer_util import Call, Name, String, touch_import @@ -45,33 +47,33 @@ class FixOperator(fixer_base.BaseFix): @invocation("operator.contains(%s)") def _sequenceIncludes(self, node, results): - return self._handle_rename(node, results, u"contains") + return self._handle_rename(node, results, "contains") @invocation("hasattr(%s, '__call__')") def _isCallable(self, node, results): obj = results["obj"] - args = [obj.clone(), String(u", "), String(u"'__call__'")] - return Call(Name(u"hasattr"), args, prefix=node.prefix) + args = [obj.clone(), String(", "), String("'__call__'")] + return Call(Name("hasattr"), args, prefix=node.prefix) @invocation("operator.mul(%s)") def _repeat(self, node, results): - return self._handle_rename(node, results, u"mul") + return self._handle_rename(node, results, "mul") @invocation("operator.imul(%s)") def _irepeat(self, node, results): - return self._handle_rename(node, results, u"imul") + return self._handle_rename(node, results, "imul") @invocation("isinstance(%s, collections.Sequence)") def _isSequenceType(self, node, results): - return self._handle_type2abc(node, results, u"collections", u"Sequence") + return self._handle_type2abc(node, results, "collections", "Sequence") @invocation("isinstance(%s, collections.Mapping)") def _isMappingType(self, node, results): - return self._handle_type2abc(node, results, u"collections", u"Mapping") + return self._handle_type2abc(node, results, "collections", "Mapping") @invocation("isinstance(%s, numbers.Number)") def _isNumberType(self, node, results): - return self._handle_type2abc(node, results, u"numbers", u"Number") + return self._handle_type2abc(node, results, "numbers", "Number") def _handle_rename(self, node, results, name): method = results["method"][0] @@ -81,16 +83,16 @@ class FixOperator(fixer_base.BaseFix): def _handle_type2abc(self, node, results, module, abc): touch_import(None, module, node) obj = results["obj"] - args = [obj.clone(), String(u", " + u".".join([module, abc]))] - return Call(Name(u"isinstance"), args, prefix=node.prefix) + args = [obj.clone(), String(", " + ".".join([module, abc]))] + return Call(Name("isinstance"), args, prefix=node.prefix) def _check_method(self, node, results): - method = getattr(self, "_" + results["method"][0].value.encode("ascii")) - if callable(method): + method = getattr(self, "_" + results["method"][0].value) + if isinstance(method, collections.Callable): if "module" in results: return method else: - sub = (unicode(results["obj"]),) - invocation_str = unicode(method.invocation) % sub - self.warning(node, u"You should use '%s' here." % invocation_str) + sub = (str(results["obj"]),) + invocation_str = method.invocation % sub + self.warning(node, "You should use '%s' here." % invocation_str) return None |