Skip to content
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

Fix possible UR, clear compile warnings, add printPalErrorString() wrapper #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

felix-kolbe
Copy link

I improved the library regarding following points:

  • Fix a possible undefined read error in VL53L0X_CheckAndLoadInterruptSettings().
  • Clean library from compiler warnings by removing unused variables.
  • Add Adafruit_VL53L0X::printPalErrorString() to human-readably print the status code returned e.g. by Adafruit_VL53L0X::getSingleRangingMeasurement().

Thanks for maintaining and merging :-)

@caternuson
Copy link
Contributor

@felix-kolbe Hey, sorry this never got looked at. If you're still interested in submitting this PR, please update to resolve the merge conflicts.

… in human-readable format (wrapper to internal VL53L0X_GetPalErrorString)
@felix-kolbe
Copy link
Author

@caternuson Happy New Year ;-) I found the mail for your comment again and gave it a try.
As you can imagine I don't have the hardware setup anymore (not even back in 2022), so this is just a rebase with a style updates, basically. Nice to see this Arduino Library CI, though!

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