Skip to content

readinto can also return None #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 15, 2025
Merged

readinto can also return None #6

merged 1 commit into from
Apr 15, 2025

Conversation

burtyb
Copy link
Contributor

@burtyb burtyb commented Apr 15, 2025

When timeout=0 "n = self.readinto(buf)" can return None which results in an error at "if n < ...".

TypeError: unsupported types for lt: 'NoneType', 'int'

When timeout=0 "n = self.readinto(buf)" can return None which results in an error at "if n < ...".

TypeError: unsupported types for __lt__: 'NoneType', 'int'
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@tannewt tannewt merged commit 66046da into adafruit:main Apr 15, 2025
1 check passed
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Apr 19, 2025
Updating https://github.com/adafruit/Adafruit_CircuitPython_TLV320 to 1.0.1 from 1.0.0:
  > make speaker louder by default
  > Update README.rst

Updating https://github.com/adafruit/Adafruit_CircuitPython_ImageLoad to 1.24.2 from 1.24.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_ImageLoad#100 from carlossless/binary-pbm-width-fix

Updating https://github.com/adafruit/Adafruit_CircuitPython_PIO_UART to 1.0.4 from 1.0.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_PIO_UART#6 from burtyb/patch-1

Updating https://github.com/adafruit/Adafruit_CircuitPython_USB_Host_Descriptors to 0.2.3 from 0.2.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_USB_Host_Descriptors#6 from FoamyGuy/update_two_mice_example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants