Added some more XXXes for dangerfan.

src/utils/transaction.py

@@ -93,6 +93,24 @@ class TransactionMixin(python.Object):
         
 
 class Transaction(TransactionMixin):
+    # XXX Transaction needs to be made threadsafe.
+    
+    # XXX There needs to be a way, given a transaction, to get a
+    #     "sub-transaction", which:
+    #
+    #     1. Doesn't try to grab the txnDir and move it, but instead is just
+    #        given the actual directory being used and uses that.
+    #     2. Acquires the lock of the original transaction, only releasing it
+    #        when its .commit method is called (assuming Transaction is
+    #        threadsafe).
+    #     3. Has a no-op .commit method (i.e., doesn't commit).
+    #
+    #     This is so that, for instance, an object with an active Transaction
+    #     can give other objects a Transaction-ish object without worrying that
+    #     the transaction will be committed, while still allowing those objects
+    #     to work properly with real transactions (i.e., they still call
+    #     as they would on a normal Transaction, it just has no effect with a
+    #     sub-transaction).
     def __init__(self, *args, **kwargs):
         """Transaction(root, txnDir) -> None