facebook/sapling
1
1
Fork 0
mirror of https://github.com/facebook/sapling.git synced 2024-10-07 15:27:13 +03:00
Code Issues Projects Releases Wiki Activity

manifest: refer to bytestrings as bytes, not str

Browse Source 
Required on Python 3.
This commit is contained in:
Augie Fackler 2017-03-19 01:12:03 -04:00
parent 4840b79ea6
commit 7b7bdd3bf0
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
mercurial/manifest.py
View File

@ -284,17 +284,17 @@ class _lazymanifest(object):
self.data = self.data[:cur] + '\x00' + self.data[cur + 1:]
def __setitem__(self, key, value):
if not isinstance(key, str):
raise TypeError("setitem: manifest keys must be a string.")
if not isinstance(key, bytes):
raise TypeError("setitem: manifest keys must be a byte string.")
if not isinstance(value, tuple) or len(value) != 2:
raise TypeError("Manifest values must be a tuple of (node, flags).")
hashval = value[0]
if not isinstance(hashval, str) or not 20 <= len(hashval) <= 22:
raise TypeError("node must be a 20-byte string")
if not isinstance(hashval, bytes) or not 20 <= len(hashval) <= 22:
raise TypeError("node must be a 20-byte byte string")
flags = value[1]
if len(hashval) == 22:
hashval = hashval[:-1]
if not isinstance(flags, str) or len(flags) > 1:
if not isinstance(flags, bytes) or len(flags) > 1:
raise TypeError("flags must a 0 or 1 byte string, got %r", flags)
needle, found = self.bsearch2(key)
if found: