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

Inaccessible class member functions #86

Closed
tho1010 opened this issue Aug 14, 2020 · 5 comments
Closed

Inaccessible class member functions #86

tho1010 opened this issue Aug 14, 2020 · 5 comments

Comments

@tho1010
Copy link

tho1010 commented Aug 14, 2020

  • Arduino board: board independent software bug

  • Arduino IDE version (found in Arduino -> About Arduino menu): Windows 1.8.13

  • List the steps to reproduce the problem below (if possible attach a sketch or
    copy the sketch code in too):
    the two class member functions

  • float Adafruit_BME280::getTemperatureCompensation(vid){}

  • void Adafruit_BME280::setTemperatureCompensation(float adjustment){}

are not accessable from a Sketch and generate a compiler error:
'class Adafruit BME280' has no member named 'getTemperatureCompensation'
Despite the fact, that this member is explicitely specified in Adafruit_BME280.cpp and referenced in the public section of Adafruit_BME280.h

Maybe the compiled library code is not identical to the documented code.

@ladyada
Copy link
Member

ladyada commented Aug 14, 2020

@AgentZombie can help you with that

@jasonmf
Copy link
Contributor

jasonmf commented Aug 14, 2020

When I submitted the PR it didn't occur to me that the version might need to be bumped in library.properties. Before and after my changes are both 2.0.2. Testing it on a fresh project in platformio shows the issue described unless I specify the dependency as the repo instead of the name.

I'm not C++ savvy and I'm new with MCU dev. My best guess is that library.properties needs the version bumped to 2.0.3. Can someone more experienced weigh in?

@caternuson
Copy link
Contributor

Yah, looks like it just needs a version bump and a release.

@ladyada
Copy link
Member

ladyada commented Aug 14, 2020

we do a sweep once a week and bump old libraries - @siddacious will get to it shortly!

@siddacious
Copy link
Contributor

released!

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

No branches or pull requests

5 participants