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.

Sign up for GitHub

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

Jump to bottom

Adafruit_Sensor.h: drop unused private member #51

Merged
merged 1 commit into from Jul 19, 2023
Merged

Adafruit_Sensor.h: drop unused private member #51

merged 1 commit into from Jul 19, 2023

Conversation

maribu
Copy link
Contributor

@maribu maribu commented Jul 17, 2023

clang++ is very unhappy about unused private members and, unless told with additional CFLAGS, refuses to compile the code. This makes clang++ happy again.

@maribu maribu mentioned this pull request Jul 17, 2023
Merged
4 tasks
brentru
brentru approved these changes Jul 19, 2023
View reviewed changes
Copy link
Member

@brentru brentru left a comment

Choose a reason for hiding this comment

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

Approved. I don't see any usage of _autoRange across this library, possibly it is an artifact from an older release.

@brentru brentru merged commit e341e1d into adafruit:master Jul 19, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brentru brentru brentru approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@maribu @brentru