- symbols now depickle properly

- fixed some symbol __new__ abuse

1 files changed, 16 insertions, 14 deletions

diff --git a/lib/sqlalchemy/util.py b/lib/sqlalchemy/util.py
index 9adb3983d..af77d792e 100644
--- a/lib/sqlalchemy/util.py
+++ b/lib/sqlalchemy/util.py
@@ -981,6 +981,17 @@ class ScopedRegistry(object):
     def _get_key(self):
         return self.scopefunc()
 
+class _symbol(object):
+    def __init__(self, name):
+        """Construct a new named symbol."""
+        assert isinstance(name, str)
+        self.name = name
+    def __reduce__(self):
+        return symbol, (self.name,)
+    def __repr__(self):
+        return "<symbol '%s>" % self.name
+_symbol.__name__ = 'symbol'
+
 class symbol(object):
     """A constant symbol.
 
@@ -991,32 +1002,23 @@ class symbol(object):
 
     A slight refinement of the MAGICCOOKIE=object() pattern.  The primary
     advantage of symbol() is its repr().  They are also singletons.
-    """
 
+    Repeated calls of symbol('name') will all return the same instance.
+
+    """
     symbols = {}
     _lock = threading.Lock()
 
     def __new__(cls, name):
+        cls._lock.acquire()
         try:
-            symbol._lock.acquire()
             sym = cls.symbols.get(name)
             if sym is None:
-                cls.symbols[name] = sym = object.__new__(cls, name)
+                cls.symbols[name] = sym = _symbol(name)
             return sym
         finally:
             symbol._lock.release()
 
-    def __init__(self, name):
-        """Construct a new named symbol.
-
-        Repeated calls of symbol('name') will all return the same instance.
-        """
-
-        assert isinstance(name, str)
-        self.name = name
-    def __repr__(self):
-        return "<symbol '%s>" % self.name
-
 def warn(msg):
     if isinstance(msg, basestring):
         warnings.warn(msg, exceptions.SAWarning, stacklevel=3)