[MERGE] Add debugging flag for merges

Vincent Ladeuil v.ladeuil+lp at free.fr
Thu Dec 6 19:46:17 GMT 2007


Vincent Ladeuil has voted comment.
Status is now: Semi-approved
Comment:
+        if 'merge' in debug.debug_flags:
+            plan = list(plan)
+            trans_id = self.tt.trans_id_file_id(file_id)
+            name = self.tt.final_name(trans_id) + '.plan'
+            contents = ('%10s|%s' % l for l in plan)
+            self.tt.new_file(name, self.tt.final_parent(trans_id), 
contents)

why not simply :
             contents = ('%10s|%s' % l for l in list(plan))

the previous 'plan = list(plan)' is a bit scary as it introduced a 
side-effect only when debugging



For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C47583B8C.2020909%40utoronto.ca%3E



More information about the bazaar mailing list