[MERGE] Inventory.iter_just_entries()

Ian Clatworthy ian.clatworthy at internode.on.net
Tue Mar 17 23:34:32 GMT 2009


John Arbash Meinel wrote:
> Ian Clatworthy wrote:

>> I strongly agree that iter_just_entries() shouldn't be used without
>> careful consideration in each case. That's part of the reason why
>> I didn't make changes throughout the codebase to begin using it
>> in this patch. I do feel however that generating data that isn't
>> needed is generally wasteful so we need some way of avoiding that in
>> the inventory API.

> 
> I'm fine with it as a stop-gap, but I think the code that is using it
> should actually be written differently. As there are limits to time and
> energy, we can start with this until we get to the point of changing things.

Is this bb:comment, tweak or approve? Perhaps I should make the API
_iter_just_entries() for now to discourage usage outside of bzrlib?

Ian C.



More information about the bazaar mailing list