[MERGE] Nuke versioned file add/get delta support, allowing easy simplification of unannotated Content, reducing memory copies and friction during commit on unannotated texts.
Martin Pool
mbp at canonical.com
Thu Sep 6 02:18:22 BST 2007
Martin Pool has voted tweak.
Status is now: Conditionally approved
Comment:
+class ExistingContent(BzrError):
+
+ """Added in bzrlib 0.92, used by VersionedFile.add_lines."""
The first line of the docstring ought to tell what this *is*, and that
doesn't. Can you please add something before saying what this exception
means?
+class PlainKnitContent(KnitContent):
+ """Unannotated content."""
+
This has the special behaviour of, when annotated, claiming that
everything
came from a single version. That seems like quite a nice facility
but you should really say that's what happens.
+ # Technically this is a case of LBYL, but:
Please expand the acronym; you can capitalize the phrase if you want.
-class KnitContentTests(TestCase):
+class KnitContentTestsMixin(object):
I would kind of prefer that you change these to scenarios instead, but I
don't insist on it.
For details, see:
http://bundlebuggy.aaronbentley.com/request/%3C1189031297.27465.2.camel%40lifeless-64%3E
More information about the bazaar
mailing list