Bazaar

bzrlib.repository.InterDifferingSerializer lacks docstring

Bug #399061 reported by Samuel Bronson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Wishlist
Unassigned

Bug Description

bzrlib.repository.InterDifferingSerializer doesn't have a docstring or a comment or anything, and the name isn't terribly descriptive either. Need I say more?

Revision history for this message
Martin Pool (mbp) wrote :

I think Andrew plans to remove this.

Changed in bzr:
importance: Undecided → Wishlist
status: New → Confirmed
Revision history for this message
Martin Pool (mbp) wrote :

I think Andrew plans to remove this.

The point of it is to convert between repositories that use different serializations, ie different representations as strings of inventories, revisions, etc

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 399061] Re: bzrlib.repository.InterDifferingSerializer lacks docstring

On Mon, 2009-07-13 at 23:58 +0000, Martin Pool wrote:
> I think Andrew plans to remove this.

We're removing the *use* of it for client-server operations because of
its very choppy IO patterns. For local conversions IDS is still the
fastest path and at least in the short term it is likely to stay.

> The point of it is to convert between repositories that use different
> serializations, ie different representations as strings of inventories,
> revisions, etc

I think it would be good to add a docstring ;).

-Rob

Jelmer Vernooij (jelmer)
tags: added: check-for-breezy
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.