Code review comment for lp:~lifeless/bzr/bug-395556

Revision history for this message
Robert Collins (lifeless) wrote :

> Anyway, we might want a comment about performance if we see a regression from
> this. ISTR that Inventory.copy() time can easily became a dominant factor in a
> lot of operations. But we can certainly start here.

Yah, copy() is bad ;). That said, we're copying tuples lists and strings not complex objects, so there is some reason to believe it will be lighter weight.

« Back to merge proposal