-
Notifications
You must be signed in to change notification settings - Fork 29
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
Changes for Linux and MOD Devices #1
base: master
Are you sure you want to change the base?
Conversation
If it's only Windows that needs it, leave it for Windows only. Overloading those is very nasty.
Using math macros is ugly, breaks std::sin and similar
All those math defines mess up with the regular math includes, and may lead to link errors due to function prototypes being different.
This reverts commit 0e68ba3.
Clear that Cycling74 doesn't care about running their code on Linux. |
@dromer sadly. bit of a waste in all honesty |
@lewlepton considering this was actually supposed to be some collaboration between the 2 companies .. |
you are over-reacting. A similar, DPF-based plugin version is available at https://github.com/moddevices/max-gen-skeleton |
bla, ok didn't realize that. but why keep this open for so long? this is likely to never get merged, then. |
Not that the dpf version worked for me: moddevices/max-gen-skeleton#2 But I gave up on using max/msp for creating cross-platform audio plugins anyway :# |
Split into small commits for easier check of the changes made.
Please review and comment.