[MERGE] rand_chars() optimization
Aaron Bentley
aaron.bentley at utoronto.ca
Tue Mar 13 14:33:22 GMT 2007
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
John Arbash Meinel wrote:
> John Arbash Meinel has voted +1 (conditional).
> Status is now: Semi-approved
> Comment:
> I have to wonder why you are doing 20 characters in one pass, and 16 in
> the other. Also, I'd like to see it more like 50,000 rather than
> 100,000. It just seems that you have a fairly fast system, and it is
> still above the 1s target time.
This patch, like most optimizations, reduces legibility. So far, I
haven't seen any evidence that it makes any Bazaar operation observably
faster. So to my mind, there's no justification for including it.
Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFF9rYy0F+nu1YWqI0RAgw1AJ9W33BHpdxeg031zDvtvrkG9zxCFgCeOnqX
eydIwMefZFF5Yo2WX6Aeqi4=
=maq3
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list