msokalski/delabella

support build using gcc

Closed this issue · 5 comments

dsegerss commented

Thanks for an interesting library :-)
Currently it does not seem to compile usign gcc, e.g. due to intrin.h missing. Support for gcc would be highly appreciated.

msokalski commented

Hi David,
I think I left over unused intrin.h header in some previous commit. Please update (pull) master branch.
If you still have an issue with it please let me know.

dsegerss commented

Thanks for the quick reply. However, there seems to be a number of remaining compatibility issues. I'm testing with gcc 8.5 (Red Hat...) and I'm getting:

In file included from delabella.cpp:24:
delabella.h:33:3: error: ‘uint8_t’ does not name a type; did you mean ‘u_int8_t’?
   uint8_t flags;
delabella.h: In member function ‘bool IDelaBella2<T, I>::Simplex::IsDelaunay() const’:
delabella.h:37:13: error: ‘flags’ was not declared in this scope
    return !(flags & 0b10000000);
delabella.cpp:2109:15: error: ‘struct CDelaBella2<long double, signed char>::Face’ has no member named ‘flags’
     if (dela->flags & (0b1000 << it.around))
delabella-sdl2.cpp:37:10: fatal error: SDL2/SDL.h: No such file or directory
msokalski commented

Ok somehow I have forgotten to add

#include <stdint.h>

at the begining of delabella.h

I'll fix it later.

Regarding errors with SDL2 headers you should install SDL2 package for your linux distribution.

Does it solve your issues?

dsegerss commented

Thanks, yes! Now it seems to compile fine.

  • 👍1
msokalski commented

Great. If you have any other problems please open a new issue.