Skip to content

sqlite3.Blob indexing is not conformant with existing norms #92019

Closed

Description

@erlend-aasland

Yesterday, @JelleZijlstra enlightened me that sqlite3.Blob indexing operate with bytes objects of length 1. This is a deviation from how bytes, bytearray, and memoryview work:

>>> bytes(b"0")[0]
48
>>> bytearray(b"0")[0]
48
>>> memoryview(b"0")[0]
48
>>> b = bytearray(b"0")
>>> b[0] = 1
>>> b[0] = b"1"
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'bytes' object cannot be interpreted as an integer
>>> m = memoryview(b)
>>> m[0] = 1
>>> m[0] = b"1"
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: memoryview: invalid type for format 'B'
>>> 

Suggesting to align sqlite3.Blob indexing behaviour so it expects ints and returns ints.

Activity

  1. added a commit that references this issue on Apr 28, 2022
  2. JelleZijlstra commented on Apr 28, 2022

    @JelleZijlstra
    Member

    Credit to @Akuli who brought this up in python/typeshed#7684 (comment)

  3. serhiy-storchaka commented on Apr 29, 2022

    @serhiy-storchaka
    Member

    There is also a similar deviation in mmap.

  4. JelleZijlstra commented on Apr 29, 2022

    @JelleZijlstra
    Member

    We probably can't fix that though, since mmap has already been in the stdlib for a long time.

  5. added a commit that references this issue on Apr 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions