-
Notifications
You must be signed in to change notification settings - Fork 12
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
Code no longer compiles with MSVC #20
Comments
EDIT: I have seen the link you have sent I will get on this right away |
MSVC seems to implement all the POSIX complex functions and the |
Thanks. It's not necessary to add zenith back in, I can remove that SDRangel. It's the complex in the hilbert_transform that's the main problem. |
This should be fixed, can you check? |
It compiles, thanks. Will try integrating it a little later on. |
Seems to work fine on Linux and Windows. Thanks. |
With the introduction of the complex data type, aptdec no longer compiles with MSVC, which is a shame, as the latest code can't be included in SDRangel
It's possible to write a little abstraction layer, so complex types can be used both with gcc/clang/MSVC, see here as an example:
https://github.com/srcejon/codec2/blob/windows-freedv/src/compiler.h
Would you be open to a patch that used something along these lines?
The text was updated successfully, but these errors were encountered: