search

alicevision / geogram

Git mirror of the geogram library by INRIA
https://github.com/BrunoLevy/geogram
BSD 3-Clause "New" or "Revised" License
123 stars 54 forks source link

MS Visual Studio 2015 vc14 OpenMP build error #6

Closed adrelino closed 2 years ago

adrelino commented 6 years ago

I cannot compile geogram on Windows with vc14 C++ compiler and OpenMP enabled:

Error C3016 'i': index variable in OpenMP 'for' statement must have signed integral type \src\lib\geogram\basic\process.cpp#235

Looks like unsigned integral types in parallel for loop are only supported starting with OpenMP 3, not OpenMP 2.5 that comes with vc14: https://stackoverflow.com/questions/2820621/why-arent-unsigned-openmp-index-variables-allowed

fabiencastan commented 6 years ago

Hi, This is just a mirror of the original repository that we use to maintain a version for our usage in AliceVision. For any question or request for integration you should address them at the official mailing list geogram-users@lists.gforge.inria.fr. Best,

BrunoLevy commented 4 years ago

Fixed in 1.7.6 (upcoming release)