[MERGE] bzrlib.osutils fixes

Robert Collins robertc at robertcollins.net
Fri Mar 9 14:55:55 GMT 2007


On Fri, 2007-03-09 at 08:50 -0600, John Arbash Meinel wrote:
> 
> > I believe the code processes only 5 tuples at all at each 'bzr
> selftest'
> > run so the fix just need to be as simple as possible. Moreover  the
> code
> > above is definitely slower than the my fix and I don't believe the
> > result code will be any faster.
> > 
> 
> I also think Aaron didn't realize this was just in the test suite.
> This
> wasn't actually part of day-to-day code.
> 
> Optimizing the test suite can be useful, but isn't a primary focus.
> 
> The primary focus is optimizing the internals of bzr. 

Its also worth noting:

profile, profile, profile.

All the notes I'm putting up are 'map, not territory' - there will
always be cases where objects are faster than tuples; its just a matter
of spotting them; or cases where the overhead of objects and lists is
outweighed by ease of use, or other alternatives can be found to achieve
performance.

Rob
-- 
GPG key available at: <http://www.robertcollins.net/keys.txt>.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070310/eb7fc079/attachment.pgp 


More information about the bazaar mailing list