From 8eb1f077c2be03f72ef31ddd2cfe805ffbfbd524 Mon Sep 17 00:00:00 2001 From: Serhiy Storchaka Date: Sat, 16 May 2015 21:38:05 +0300 Subject: Issue #18682: Optimized pprint functions for builtin scalar types. --- Lib/pprint.py | 22 ++++------------------ Misc/NEWS | 2 ++ 2 files changed, 6 insertions(+), 18 deletions(-) diff --git a/Lib/pprint.py b/Lib/pprint.py index e084dc6..87649b4 100644 --- a/Lib/pprint.py +++ b/Lib/pprint.py @@ -489,24 +489,8 @@ class PrettyPrinter: def _safe_repr(object, context, maxlevels, level): typ = type(object) - if typ is str: - if 'locale' not in _sys.modules: - return repr(object), True, False - if "'" in object and '"' not in object: - closure = '"' - quotes = {'"': '\\"'} - else: - closure = "'" - quotes = {"'": "\\'"} - qget = quotes.get - sio = _StringIO() - write = sio.write - for char in object: - if char.isalpha(): - write(char) - else: - write(qget(char, repr(char)[1:-1])) - return ("%s%s%s" % (closure, sio.getvalue(), closure)), True, False + if typ in _builtin_scalars: + return repr(object), True, False r = getattr(typ, "__repr__", None) if issubclass(typ, dict) and r is dict.__repr__: @@ -571,6 +555,8 @@ def _safe_repr(object, context, maxlevels, level): rep = repr(object) return rep, (rep and not rep.startswith('<')), False +_builtin_scalars = frozenset({str, bytes, bytearray, int, float, complex, + bool, type(None)}) def _recursion(object): return ("" diff --git a/Misc/NEWS b/Misc/NEWS index a00d967..1b23a98 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -47,6 +47,8 @@ Core and Builtins Library ------- +- Issue #18682: Optimized pprint functions for builtin scalar types. + - Issue #22027: smtplib now supports RFC 6531 (SMTPUTF8). - Issue #23488: Random generator objects now consume 2x less memory on 64-bit. -- cgit v0.12