Rename Dyn -> _Dyn (singleton; constructor should be private)

Technologicat · Technologicat · commit 5368f2727d7d · 2020-03-05T17:17:16.000+02:00
diff --git a/unpythonic/collections.py b/unpythonic/collections.py
@@ -22,7 +22,7 @@
 from .llist import cons
 from .misc import getattrrec
 from .env import env
-from .dynassign import Dyn
+from .dynassign import _Dyn
 
 def get_abcs(cls):
     """Return a set of the collections.abc superclasses of cls (virtuals too)."""
@@ -123,7 +123,7 @@ def doit(x):
             return {doit(elt) for elt in x}
         # env and dyn provide the Mapping API, but shouldn't get the general Mapping treatment here.
         # (This is important for the curry and lazify macros.)
-        elif isinstance(x, Mapping) and not isinstance(x, (env, Dyn)):
+        elif isinstance(x, Mapping) and not isinstance(x, (env, _Dyn)):
             ctor = type(x)
             return ctor({k: doit(v) for k, v in x.items()})
         elif isinstance(x, Sequence) and not isinstance(x, (str, bytes, range)):
diff --git a/unpythonic/dynassign.py b/unpythonic/dynassign.py
@@ -73,7 +73,7 @@ def __del__(self):
     def _refresh(self):
         self.maps = list(reversed(_getstack())) + [_global_dynvars]
 
-class Dyn(Singleton):
+class _Dyn(Singleton):
     """This module exports a singleton, ``dyn``, which provides dynamic assignment
     (like Racket's ``parameterize``; akin to Common Lisp's special variables.).
 
@@ -259,7 +259,7 @@ def __setitem__(self, k, v):
     def __repr__(self):
         bindings = ["{:s}={}".format(k, repr(self[k])) for k in self]
         return "<dyn object at 0x{:x}: {{{:s}}}>".format(id(self), ", ".join(bindings))
-dyn = Dyn()
+dyn = _Dyn()
 
 def make_dynvar(**bindings):
     """Create a dynamic variable and set its default value.
@@ -289,5 +289,5 @@ def make_dynvar(**bindings):
 
 # register virtual base classes
 for abscls in (Container, Sized, Iterable, Mapping):
-    abscls.register(Dyn)
+    abscls.register(_Dyn)
 del abscls
