You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ju...@apache.org on 2011/08/09 13:28:11 UTC

svn commit: r1155313 - /subversion/trunk/subversion/bindings/ctypes-python/csvn/repos.py

Author: julianfoad
Date: Tue Aug  9 11:28:11 2011
New Revision: 1155313

URL: http://svn.apache.org/viewvc?rev=1155313&view=rev
Log:
Remove unused broken code in the ctypes-python bindings.

* subversion/bindings/ctypes-python/csvn/repos.py
  (_fs): Remove the txn() method, which was unused and referred to
    '_fs_root' which doesn't exist. Thus avoid storing a reference to the
    'repos' object during __init__(), which was a circular reference because
    this '_fs' object is itself stored within the 'repos' object.

Modified:
    subversion/trunk/subversion/bindings/ctypes-python/csvn/repos.py

Modified: subversion/trunk/subversion/bindings/ctypes-python/csvn/repos.py
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/ctypes-python/csvn/repos.py?rev=1155313&r1=1155312&r2=1155313&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/ctypes-python/csvn/repos.py (original)
+++ subversion/trunk/subversion/bindings/ctypes-python/csvn/repos.py Tue Aug  9 11:28:11 2011
@@ -541,7 +541,6 @@ class _fs(object):
        This class represents an svn_fs_t object"""
 
     def __init__(self, repos):
-        self.repos = repos
         self.iterpool = Pool()
         self._as_parameter_ = svn_repos_fs(repos)
 
@@ -572,13 +571,6 @@ class _fs(object):
            temporary allocations."""
         return _fs_root(self, rev, txn, pool, iterpool)
 
-    def txn(self, message, base_rev=None):
-        """Open a new transaction for commit to the specified
-           repository, assuming that our data is up to date as
-           of base_rev. Setup the author and commit message
-           revprops."""
-        return _fs_txn(self.repos, message, base_rev)
-
 class _fs_root(object):
     """NOTE: This is a private class. Don't use it outside of
        this module. Use the Repos.txn() method instead.