Skip to content

Give _int_pin and _reset_pin sane default values #13

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
Nov 29, 2023

Conversation

rah2501
Copy link

@rah2501 rah2501 commented Nov 28, 2023

Assign sane default values to _int_pin and _reset_pin. Otherwise they get assigned zero and all the "if (_int_pin == -1)" checks fail and bad things happen.

Assign sane default values to _int_pin and _reset_pin.  Otherwise they
get assigned zero and all the "if (_int_pin == -1)" checks fail and
bad things happen.
@lewispg228
Copy link
Member

Good catch. All of our library examples were recently updated to use (and define) the INT and RST pins, so we hadn't come across this issue yet. Thanks for your PR!

@lewispg228 lewispg228 merged commit c90b84c into sparkfun:main Nov 29, 2023
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