Should raise TypeError if unicode string is used as SHA index

[jonashaag]

repo[some_unicode_commit_sha] throws a KeyError when it should raise a TypeError (because unicode is now allowed).

    def __getitem__(self, name):
        ### Skipped because isinstance check does not hold
        if len(name) in (20, 40) and isinstance(name, str):
            try:
                return self.object_store[name]
            except (KeyError, ValueError):
                pass
        ### Fails because sha is no ref
        try:
            return self.object_store[self.refs[name]]
        except RefFormatError:
            raise KeyError(name)

[jelmer]

Agreed, Dulwich should raise TypeError in this case. Patches welcome :)