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

Mac OS does not correctly support the math library long double constants #59

Closed
simoninns opened this issue Dec 20, 2018 · 1 comment · Fixed by #421
Closed

Mac OS does not correctly support the math library long double constants #59

simoninns opened this issue Dec 20, 2018 · 1 comment · Fixed by #421
Assignees
Labels
enhancement ld-decode-tools An issue only affecting the ld-decode-tools

Comments

@simoninns
Copy link
Collaborator

This causes and issue compiling the ld-decode-tools ld-comb-ntsc on mac:

comb.cpp:732:83: error: use of undeclared identifier 'M_PIl'; did you mean
'P_PID'?
...double>(y), x) * (180 / M_PIl));
^~~~~
P_PID

This can be fixed by defining the constant by hand:

#ifndef M_PIl
#define M_PIl 0xc.90fdaa22168c235p-2L
#endif

@simoninns simoninns added enhancement ld-decode-tools An issue only affecting the ld-decode-tools labels Dec 20, 2018
@simoninns simoninns self-assigned this Dec 20, 2018
@simoninns
Copy link
Collaborator Author

Closing as fixed (no Mac environment to test on) - reopen if the problem is still there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ld-decode-tools An issue only affecting the ld-decode-tools
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant