Mercurial > public > mercurial-scm > hg-stable
diff mercurial/filelog.py @ 2579:0875cda033fd
use __contains__, index or split instead of str.find
str.find return -1 when the substring is not found, -1 evaluate
to True and is a valid index, which can lead to bugs.
Using alternatives when possible makes the code clearer and less
prone to bugs. (and __contains__ is faster in microbenchmarks)
| author | Benoit Boissinot <benoit.boissinot@ens-lyon.org> |
|---|---|
| date | Sun, 09 Jul 2006 01:30:30 +0200 |
| parents | fe1689273f84 |
| children | 345bac2bc4ec |
line wrap: on
line diff
--- a/mercurial/filelog.py Sat Jul 08 16:55:49 2006 +0200 +++ b/mercurial/filelog.py Sun Jul 09 01:30:30 2006 +0200 @@ -34,14 +34,14 @@ t = self.revision(node) if not t.startswith('\1\n'): return t - s = t.find('\1\n', 2) + s = t.index('\1\n', 2) return t[s+2:] def readmeta(self, node): t = self.revision(node) if not t.startswith('\1\n'): return {} - s = t.find('\1\n', 2) + s = t.index('\1\n', 2) mt = t[2:s] m = {} for l in mt.splitlines():