support build using gcc

dsegerss commented 2 years ago

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 2 years ago

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 2 years ago

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 2 years ago

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 2 years ago

Thanks, yes! Now it seems to compile fine.

msokalski commented 2 years ago

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

msokalski / delabella

support build using gcc #10