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

errors #30

Closed
VVE opened this issue Oct 12, 2016 · 3 comments
Closed

errors #30

VVE opened this issue Oct 12, 2016 · 3 comments

Comments

@VVE
Copy link

VVE commented Oct 12, 2016

Hi,
Compiling this project in Visual Studio 2015, I got an error message:
3>C:\Program Files (x86)\Windows Kits\10\Include\10.0.10240.0\ucrt\math.h(520): error C2059: syntax error: (
In math.h(520) I see:
Check_return _ACRTIMP double __cdecl rint(In double _X);
What should I fix? I try to use suitesparse-metis-for-windows for ceres-solver

@VVE VVE closed this as completed Oct 12, 2016
@VVE VVE reopened this Oct 12, 2016
@jlevallois
Copy link

Hi @VVE,

Check this issue: #23

A short fix is to add a check of the Visual Studio version in metis\GKlib\gk_arch.h:

#ifdef __MSC__
#if (_MSC_VER < 1800)
/* MSC does not have rint() function */
#define rint(x) ((int)((x)+0.5))
#endif
//....

rint() is available from Visual Studio 2013 (_MSC_VER 1800)

:octocat:

@Lokeshmsit
Copy link

Lokeshmsit commented Dec 29, 2016

Thanks, i rather commented the #define rint(x) ((int)((x)+0.5)) in maths.h.
In math.h #define for rint() is declared without any Visual Studio version check.

@zbeekman
Copy link

good lord, why is metis' build system so janky... I'm hoping that this is the only magic I need to get over this hurdle for a METIS build myself... Thanks for providing this fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants