made bbrepr recursive using reprlib

glom/core.py

@@ -40,10 +40,12 @@
 if PY2:
     _AbstractIterableBase = object
     from .chainmap_backport import ChainMap
+    import repr as reprlib
 else:
     basestring = str
     _AbstractIterableBase = ABCMeta('_AbstractIterableBase', (object,), {})
     from collections import ChainMap
+    import reprlib
 
 GLOM_DEBUG = os.getenv('GLOM_DEBUG', '').strip().lower()
 GLOM_DEBUG = False if (GLOM_DEBUG in ('', '0', 'false')) else True
@@ -400,14 +402,25 @@ def get_message(self):
 _BUILTIN_ID_NAME_MAP = dict([(id(v), k)
                              for k, v in __builtins__.items()])
 
-def bbrepr(obj):
+
+class _BBRepr(reprlib.Repr):
     """A better repr for builtins, when the built-in repr isn't
     roundtrippable.
     """
-    ret = repr(obj)
-    if not ret.startswith('<'):
-        return ret
-    return _BUILTIN_ID_NAME_MAP.get(id(obj), ret)
+    def __init__(self):
+        super(_BBRepr, self).__init__()
+        # turn up all the length limits very high
+        for name in self.__dict__:
+            setattr(self, name, 1024)
+
+    def repr1(self, x, maxlevel):
+        ret = super(_BBRepr, self).repr1(x, maxlevel)
+        if not ret.startswith('<'):
+            return ret
+        return _BUILTIN_ID_NAME_MAP.get(id(x), ret)
+
+
+bbrepr = _BBRepr().repr
 
 
 class _BBReprFormatter(string.Formatter):
@@ -421,10 +434,7 @@ def convert_field(self, value, conversion):
         return super(_BBReprFormatter, self).convert_field(value, conversion)
 
 
-_BB_REPR_FORMATTER = _BBReprFormatter()
-
-
-bbformat = _BB_REPR_FORMATTER.format
+bbformat = _BBReprFormatter().format
 
 
 # TODO: push this back up to boltons with repr kwarg

glom/test/test_basic.py

@@ -8,7 +8,7 @@
 from glom import Auto, Fill, Iter
 
 import glom.core as glom_core
-from glom.core import UP, ROOT, Let, bbformat
+from glom.core import UP, ROOT, Let, bbformat, bbrepr
 
 from glom import OMIT  # backwards compat
 
@@ -463,3 +463,7 @@ def test_api_repr():
 
 def test_bbformat():
     assert bbformat("{0.__name__}", int) == "int"
+
+
+def test_bbrepr():
+    assert bbrepr({str: int}) == "{str: int}"