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

Fix for OS other than Windows (Linux tested) & cmake #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix for OS other than Windows (Linux tested) & cmake #4

wants to merge 2 commits into from

Commits on Jan 18, 2024

  1. Fix code for other OS than Windows 

    The code had to be fixed do compile on other OS than Windows. Some small
changes also had to be applied to have correct results on other OS.

- the '\' had to be replaced by '/' in #include, the latter being valid on
both Windows and Linux (and Mac, and ...) OS
- the "_copysign" function is "copysign" for non Windows OS
- the use of "abs" is code led to bad results (it would convert double
to integers instead of taking absolute values), it's been replace by
"fabs"
    @NicolasNOE
    NicolasNOE committed Jan 18, 2024
    Configuration menu
    Copy the full SHA
    6ffa50a View commit details
    Browse the repository at this point in the history

  2. In order to use CMake for project generation 

    Tested on both Windows (Visual Studio projects) and Linux (makefiles for
gcc).

With the code fixes, the results are identifical on both OS.
    @NicolasNOE
    NicolasNOE committed Jan 18, 2024
    Configuration menu
    Copy the full SHA
    1cab6d7 View commit details
    Browse the repository at this point in the history