| Alphare | |
| pulkit |
| hg-reviewers |
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
| def __setitem__(self, key, value): | def __setitem__(self, key, value): | ||||
| if not isinstance(key, bytes): | if not isinstance(key, bytes): | ||||
| raise TypeError(b"setitem: manifest keys must be a byte string.") | raise TypeError(b"setitem: manifest keys must be a byte string.") | ||||
| if not isinstance(value, tuple) or len(value) != 2: | if not isinstance(value, tuple) or len(value) != 2: | ||||
| raise TypeError( | raise TypeError( | ||||
| b"Manifest values must be a tuple of (node, flags)." | b"Manifest values must be a tuple of (node, flags)." | ||||
| ) | ) | ||||
| hashval = value[0] | hashval = value[0] | ||||
| if not isinstance(hashval, bytes) or not 20 <= len(hashval) <= 22: | # hashes are either 20 or 32 bytes (sha1 or its replacement), | ||||
| raise TypeError(b"node must be a 20-byte byte string") | # and allow one extra byte taht won't be persisted to disk but | ||||
| # is sometimes used in memory. | |||||
| if not isinstance(hashval, bytes) or not ( | |||||
| 20 <= len(hashval) <= 22 or 32 <= len(hashval) <= 34 | |||||
| ): | |||||
| raise TypeError(b"node must be a 20-byte or 32-byte byte string") | |||||
| flags = value[1] | flags = value[1] | ||||
| if len(hashval) == 22: | if len(hashval) == 22: | ||||
| hashval = hashval[:-1] | hashval = hashval[:-1] | ||||
| if not isinstance(flags, bytes) or len(flags) > 1: | if not isinstance(flags, bytes) or len(flags) > 1: | ||||
| raise TypeError(b"flags must a 0 or 1 byte string, got %r", flags) | raise TypeError(b"flags must a 0 or 1 byte string, got %r", flags) | ||||
| needle, found = self.bsearch2(key) | needle, found = self.bsearch2(key) | ||||
| if found: | if found: | ||||
| # put the item | # put the item | ||||