Skip to content

Fix default end values. #45

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
Feb 19, 2020
Merged

Conversation

dhalbert
Copy link
Contributor

Fixes #44.

I2CDevice was passing None as a default value for end in read and write operations on busio.I2C objects. The busio.I2C documentation says that None will work, but it does not: len(buf) must be passed instead as the default.

If busio.I2C were fixed to allow None, this change could be reverted.

@dhalbert dhalbert requested review from caternuson and a team February 19, 2020 01:28
@caternuson
Copy link
Contributor

Tested and works.

Thanks for quick fix @dhalbert

@caternuson caternuson merged commit 6348879 into adafruit:master Feb 19, 2020
@dhalbert dhalbert deleted the fix-end-values branch February 19, 2020 01:54
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Feb 19, 2020
Updating https://github.com/adafruit/Adafruit_CircuitPython_BusDevice to 4.2.1 from 4.2.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_BusDevice#45 from dhalbert/fix-end-values

Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Text to 2.4.0 from 2.3.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Text#35 from FoamyGuy/master

Updating https://github.com/adafruit/Adafruit_CircuitPython_PyBadger to 1.1.1 from 1.1.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_PyBadger#20 from nnja/fix_business_card_display
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.

TypeError: can't convert NoneType to int
2 participants