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

FMOscillator.cpp now also calls abs(float) #1826

Closed
mickski56 opened this issue May 8, 2020 · 3 comments · Fixed by #1828
Closed

FMOscillator.cpp now also calls abs(float) #1826

mickski56 opened this issue May 8, 2020 · 3 comments · Fixed by #1828
Labels
Linux Issues which only occur on Linux

Comments

@mickski56
Copy link
Contributor

Same build error as in #1814 call of overloaded 'abs(float&)' is ambiguous with older gcc versions on linux.
Fix is the same "#include <cmath>" , possibly in Oscillator.h as two files now need it.
@baconpaul
Copy link
Collaborator

oh thanks!

baconpaul added a commit to baconpaul/surge that referenced this issue May 8, 2020
@baconpaul
Missing Linux Header
da80e8b 
In FMOscillator

closes surge-synthesizer#1826
@mkruselj mkruselj added the Linux Issues which only occur on Linux label May 8, 2020
@baconpaul baconpaul mentioned this issue May 8, 2020
Merged
baconpaul added a commit that referenced this issue May 8, 2020
@baconpaul
Missing Linux Header (#1828)
9d19f89 
In FMOscillator

closes #1826
@baconpaul
Copy link
Collaborator

Find at da80e8b

@baconpaul
Copy link
Collaborator

(oh and we are happy to fix these as you report them; but also happy for you to just fire in a pull request with a fix if you want)

@bobobo199733 bobobo199733 mentioned this issue Mar 5, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Linux Issues which only occur on Linux
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Missing Linux Header baconpaul/surge
3 participants
@mkruselj @baconpaul @mickski56