[MERGE] FileCollection

Martin Pool mbp at sourcefrog.net
Tue Jul 17 02:11:24 BST 2007


Adding such a utility could be good.

s/cand/can

FileCollection is not a very obvious name.  ListingFile?  FileListing?

It seems odd that it talks about sequence and stores them in sequence
but returns them as a set, in random order.

It seems like excessive policy to allocate the names as integers
within this file.  Why not let the caller supply them?

+    def iter_files_recursive(self):
+        Transport.iter_files_recursive(self)
+

I'd like a comment in those two methods explaining that the default
implementation raises an exception.

+    * New transport decorator 'unlistable+' which disables the list_dir
+      functionality for testing.
+

Take credit :)

-- 
Martin



More information about the bazaar mailing list