[MERGE][1.3.1][#208418] Return the correct knit serialisation method in _StreamAccess

Robert Collins robertc at robertcollins.net
Mon Apr 7 06:00:52 BST 2008


Robert Collins has voted tweak.
Status is now: Conditionally approved
Comment:
+            orig_options = 
list(self.backing_index.get_options(version_id))
+            if 'fulltext' in orig_options:
+                pass
+            elif 'line-delta' in orig_options:
+                # Texts from the backing knit are always returned from 
the stream
+                # as full texts
+                orig_options.remove('line-delta')
+                orig_options.append('fulltext')
+            else:
+                raise errors.KnitIndexUnknownMethod(self, options)
+            return orig_options


That looks like a list-aliasing bug waiting to happen. Better to create 
a new options tuple I think.

For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C20080404070508.GA20605%40steerpike.home.puzzling.org%3E



More information about the bazaar mailing list