In WeakSet, use the `valuerefs` method, added in Python 2.5 instead of...

In WeakSet, use the `valuerefs` method, added in Python 2.5 instead of accessing the internal `data` implementation detail.

This lets us support Jython.

.travis.yml

@@ -11,6 +11,27 @@ python:
 install:
     - pip install . --use-mirrors
 script:
-    - python setup.py test -q
+    - $PYTHON_EXE setup.py test -q
 notifications:
     email: false
+env:
+    - JYTHON=true
+    - JYTHON=false
+before_script: if [ "$JYTHON" == "true" ]; then export PYTHON_EXE=jython; jython -c "print ''"; else export PYTHON_EXE=python; fi
+before_install:
+  - export JYTHON_URL='http://search.maven.org/remotecontent?filepath=org/python/jython-installer/2.7-rc2/jython-installer-2.7-rc2.jar'
+  - if [ "$JYTHON" == "true" ]; then wget $JYTHON_URL -O jython_installer.jar; java -jar jython_installer.jar -s -d $HOME/jython; export PATH=$HOME/jython:$PATH; fi
+matrix:
+  exclude:
+    - python: 2.6
+      env: JYTHON=true
+    - python: 3.2
+      env: JYTHON=true
+    - python: 3.3
+      env: JYTHON=true
+    - python: 3.4
+      env: JYTHON=true
+    - python: pypy
+      env: JYTHON=true
+    - python: pypy3
+      env: JYTHON=true

CHANGES.rst

@@ -4,7 +4,7 @@ Changes
 1.4.4 (unreleased)
 ------------------
 
-- Add support for PyPy3.
+- Add support for PyPy3 and Jython 2.7.
 
 - Require 100% branch coverage (in addition to 100% statement coverage).
 

setup.py

@@ -48,6 +48,7 @@ setup(name='transaction',
         "Programming Language :: Python :: 3.4",
         "Programming Language :: Python :: Implementation :: CPython",
         "Programming Language :: Python :: Implementation :: PyPy",
+        "Programming Language :: Python :: Implementation :: Jython",
         "Framework :: ZODB",
         ],
       author="Zope Corporation",

tox.ini

 [tox]
-envlist = 
-# Jython support pending 2.7 support, due 2012-07-15 or so.  See:
-# http://fwierzbicki.blogspot.com/2012/03/adconion-to-fund-jython-27.html
-#    py26,py27,py32,pypy,jython,coverage
+# Jython 2.7rc2 does work, but unfortunately has an issue running
+# with Tox 1.9.2 (apparently its non-interactive console still prints
+# interactive prompts ):
+#   $ tox -e jython
+#   ...
+#   ERROR: InvocationError: Failed to get version_info for jython: could not decode ">>> ... ... ... >>> {'version_info': (2, 7, 0, 'candidate', 2)}\n>>> "
+envlist =
     py26,py27,pypy,py32,py33,py34,pypy3,coverage,docs
 
 [testenv]
-commands = 
+commands =
     python setup.py test -q
 deps = transaction
 
 [testenv:jython]
-commands = 
+commands =
    jython setup.py test -q
 
 [testenv:coverage]
 basepython =
     python2.6
-commands = 
+commands =
     nosetests --with-xunit --with-xcoverage
 deps =
     nose
@@ -27,7 +30,7 @@ deps =
 [testenv:docs]
 basepython =
     python2.6
-commands = 
+commands =
     sphinx-build -b html -d docs/_build/doctrees docs docs/_build/html
     sphinx-build -b doctest -d docs/_build/doctrees docs docs/_build/doctest
 deps =

transaction/_compat.py

@@ -2,6 +2,7 @@ import sys
 import types
 
 PY3 = sys.version_info[0] == 3
+JYTHON = sys.platform.startswith('java')
 
 if PY3: # pragma: no cover
     string_types = str,
@@ -79,12 +80,11 @@ if PY3: #pragma NO COVER
         from threading import _get_ident as get_thread_ident
 else:
     from thread import get_ident as get_thread_ident
-    
-    
+
+
 if PY3:
     def func_name(func): #pragma NO COVER
         return func.__name__
 else:
     def func_name(func): #pragma NO COVER
         return func.func_name
- 

transaction/tests/test_weakset.py

@@ -12,7 +12,7 @@
 #
 ##############################################################################
 import unittest
-
+from transaction._compat import JYTHON
 
 class WeakSetTests(unittest.TestCase):
     def test_contains(self):
@@ -35,7 +35,9 @@ class WeakSetTests(unittest.TestCase):
         self.assertEqual(len(w), 2)
         del d1
         gc.collect()
-        self.assertEqual(len(w), 1)
+        if not JYTHON:
+            # The Jython GC is non deterministic
+            self.assertEqual(len(w), 1)
 
     def test_remove(self):
         from transaction.weakset import WeakSet
@@ -100,17 +102,17 @@ class WeakSetTests(unittest.TestCase):
             gc.collect()
             return result
         w.as_weakref_list = _as_weakref_list
-            
+
         def poker(x):
             x.poked = 1
         w.map(poker)
         for thing in dummy, dummy2:
             self.assertEqual(thing.poked, 1)
-        
+
 
 class Dummy:
     pass
-        
+
 
 def test_suite():
     return unittest.makeSuite(WeakSetTests)

transaction/weakset.py

@@ -13,6 +13,7 @@
 ############################################################################
 
 import weakref
+from ._compat import PY3
 
 # A simple implementation of weak sets, supplying just enough of Python's
 # sets.Set interface for our needs.
@@ -62,7 +63,6 @@ class WeakSet(object):
     # underlying dict may change size during iteration, due to gc or
     # activity from other threads).  as_weakef_list() is safe.
     #
-    # Something like this should really be a method of Python's weak dicts.
     # If we invoke self.data.values() instead, we get back a list of live
     # objects instead of weakrefs.  If gc occurs while this list is alive,
     # all the objects move to an older generation (because they're strongly
@@ -76,8 +76,8 @@ class WeakSet(object):
     # we avoid that, although the decision to use weakrefs is now very
     # visible to our clients.
     def as_weakref_list(self):
-        # We're cheating by breaking into the internals of Python's
-        # WeakValueDictionary here (accessing its .data attribute).
-        # Python 3: be sure to freeze the list, to avoid RuntimeError:
+        # Python 3: be sure to freeze the iterator, to avoid RuntimeError:
         # dictionary changed size during iteration.
-        return list(self.data.data.values())
+        # On Python2 we already get a list, no need to copy
+        refs = self.data.valuerefs()
+        return list(refs) if PY3 else refs