9 Oct
2016
9 Oct
'16
4:55 a.m.
On Fri, Oct 7, 2016 at 3:52 PM, Ivan Levkivskyi <levkivskyi@gmail.com> wrote:
I have a question about the registration of builtins. Currently, typing.py contains this line:
ByteString.register(type(memoryview(b'')))
But there are two test lines in test_collections.py
self.assertNotIsInstance(memoryview(b""), ByteString) self.assertFalse(issubclass(memoryview, ByteString))
This looks like a contradiction. Which one is right? Should these tests be removed or the registration in typing.py?
Looks like the registration is in error. The stubs (and hence mypy) don't consider memoryview consistent with ByteString. -- --Guido van Rossum (python.org/~guido)