Added XXX comments about stray output produced by

testExceptionInTpcAbort().  The cause is clear now, but a solution isn't;
an exception in tpc_abort is nasty.

src/ZODB/tests/testTransaction.py

@@ -36,7 +36,7 @@ TODO
     add in tests for objects which are modified multiple times,
     for example an object that gets modified in multiple sub txns.
 
-$Id: testTransaction.py,v 1.21 2004/04/02 19:31:32 tim_one Exp $
+$Id: testTransaction.py,v 1.22 2004/04/02 19:48:22 tim_one Exp $
 """
 
 import unittest
@@ -385,6 +385,9 @@ class TransactionTests(unittest.TestCase):
         self.sub1.modify(nojar=1)
 
         try:
+            # XXX Transaction._cleanup() prints a stray "error tpc_abort"
+            # XXX (the string value of the TestTxnException instance raised)
+            # XXX to stdout.
             get_transaction().commit()
         except TestTxnException:
             pass

src/transaction/_transaction.py

@@ -324,11 +324,13 @@ class Transaction(object):
                 try:
                     rm.abort_sub(self)
                 except Exception, err:
+                    # XXX Just printing the error doesn't seem good enough.
                     print err
             else:
                 try:
                     rm.tpc_abort(self)
                 except Exception, err:
+                    # XXX Just printing the error doesn't seem good enough.
                     print err
 
     def _getResourceManagers(self, subtransaction):