Rev 3961: There is no need to create a progress bar that we don't even use. in lp:///~jameinel/bzr/1.12-progress-warnings

[John Arbash Meinel]

At lp:///~jameinel/bzr/1.12-progress-warnings

------------------------------------------------------------
revno: 3961
revision-id: john at arbash-meinel.com-20090126205756-wh552ozttirfo2m9
parent: pqm at pqm.ubuntu.com-20090126181248-yl5ctbxc3y6nu9m4
committer: John Arbash Meinel <john at arbash-meinel.com>
branch nick: 1.12-progress-warnings
timestamp: Mon 2009-01-26 14:57:56 -0600
message:
  There is no need to create a progress bar that we don't even use.
-------------- next part --------------
=== modified file 'bzrlib/fetch.py'
--- a/bzrlib/fetch.py	2008-12-02 05:11:07 +0000
+++ b/bzrlib/fetch.py	2009-01-26 20:57:56 +0000
@@ -232,23 +232,19 @@
     def _fetch_inventory_weave(self, revs, pb):
         pb.update("fetch inventory", 0, 2)
         to_weave = self.to_repository.inventories
-        child_pb = bzrlib.ui.ui_factory.nested_progress_bar()
-        try:
-            # just merge, this is optimisable and its means we don't
-            # copy unreferenced data such as not-needed inventories.
-            pb.update("fetch inventory", 1, 3)
-            from_weave = self.from_repository.inventories
-            pb.update("fetch inventory", 2, 3)
-            # we fetch only the referenced inventories because we do not
-            # know for unselected inventories whether all their required
-            # texts are present in the other repository - it could be
-            # corrupt.
-            to_weave.insert_record_stream(from_weave.get_record_stream(
-                [(rev_id,) for rev_id in revs],
-                self.to_repository._fetch_order,
-                not self.to_repository._fetch_uses_deltas))
-        finally:
-            child_pb.finished()
+        # just merge, this is optimisable and its means we don't
+        # copy unreferenced data such as not-needed inventories.
+        pb.update("fetch inventory", 1, 3)
+        from_weave = self.from_repository.inventories
+        pb.update("fetch inventory", 2, 3)
+        # we fetch only the referenced inventories because we do not
+        # know for unselected inventories whether all their required
+        # texts are present in the other repository - it could be
+        # corrupt.
+        to_weave.insert_record_stream(from_weave.get_record_stream(
+            [(rev_id,) for rev_id in revs],
+            self.to_repository._fetch_order,
+            not self.to_repository._fetch_uses_deltas))
 
     def _fetch_revision_texts(self, revs, pb):
         # fetch signatures first and then the revision texts