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

call of overloaded 'tone(int&, int&, int&)' is ambiguous #1487

Closed
s0170071 opened this issue Jun 8, 2018 · 7 comments
Closed

call of overloaded 'tone(int&, int&, int&)' is ambiguous #1487

s0170071 opened this issue Jun 8, 2018 · 7 comments
Labels
Category: Core related Related to the (external) core libraries Status: Fixed Commit has been made, ready for testing Type: Bug Considered a bug

Comments

@s0170071
Copy link
Contributor

s0170071 commented Jun 8, 2018 •
edited
Loading

... using latest ESPEasy and latest Core, Arduino IDE

edit: is core related:

Support multiple tone(), analogWrite(), and Servo (#4640) esp8266/Arduino#4640

we might want to adapt our code.
@Grovkillen Grovkillen added Type: Bug Considered a bug Category: Core related Related to the (external) core libraries labels Jun 8, 2018
@TD-er
Copy link
Member

TD-er commented Jun 8, 2018

What is the latest core version?

@s0170071
Copy link
Contributor Author

s0170071 commented Jun 8, 2018

Master on GitHub

@Grovkillen
Copy link
Member

Ok, so it's not yet released?

@s0170071
Copy link
Contributor Author

s0170071 commented Jun 8, 2018

No.

@s0170071
Copy link
Contributor Author

s0170071 commented Jun 8, 2018

Its probably no big deal either.

@v-a-d-e-r
Copy link

v-a-d-e-r commented Jun 10, 2018 •
edited
Loading

Same here. For the moment I made this:
File: Arduino.h
282 //void tone(uint8_t _pin, double frequency, unsigned long duration = 0);

@thomastech
Copy link
Contributor

ESP8266 2.4.2 was released last week. I've commented out tone(uint8_t _pin, unsigned int frequency, unsigned long duration) in Tone.cpp as temporary workaround when building ESPEasy.

  • Thomas

TD-er added a commit to TD-er/ESPEasy that referenced this issue Aug 14, 2018
@TD-er
[core 2.4.2] Update to PlatformIO [email protected]
35ba167 
Latest version would not build due to conflicting function definition for tone. See letscontrolit#1487
TD-er added a commit to TD-er/ESPEasy that referenced this issue Aug 14, 2018
@TD-er
[core 2.4.2] Update to PlatformIO [email protected]
8930e33 
Latest version would not build due to conflicting function definition for tone. See letscontrolit#1487
@TD-er TD-er mentioned this issue Aug 14, 2018
Merged
@TD-er TD-er added the Status: Fixed Commit has been made, ready for testing label Aug 14, 2018
@TD-er TD-er closed this as completed Aug 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Category: Core related Related to the (external) core libraries Status: Fixed Commit has been made, ready for testing Type: Bug Considered a bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@TD-er @thomastech @Grovkillen @s0170071 @v-a-d-e-r