[MERGE] remove spurious call in revision store add_revision.

John Arbash Meinel john at arbash-meinel.com
Wed Aug 22 18:29:48 BST 2007


John Arbash Meinel has voted tweak.
Status is now: Conditionally approved
Comment:
I think this is better expressed as:
--- bzrlib/store/revision/knit.py       2007-07-10 21:18:54 +0000
+++ bzrlib/store/revision/knit.py       2007-08-22 17:29:04 +0000
@@ -66,7 +66,7 @@
          """Template method helper to store revision in this store."""
          # FIXME: make this ghost aware at the knit level
          rf = self.get_revision_file(transaction)
-        self.get_revision_file(transaction).add_lines_with_ghosts(
+        rf.add_lines_with_ghosts(
              revision.revision_id,
              revision.parent_ids,
              osutils.split_lines(revision_as_file.read()))

(yes we don't want to grab it 2 times, but we should also avoid calling 
methods on the return value of functions rather than using a variable)


For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C1187769900.17572.106.camel%40localhost.localdomain%3E



More information about the bazaar mailing list