nushoin / RTree

N-dimensional RTree implementation in C++
Other
376 stars 109 forks source link

Avoid Min/Max makros, if possible #30

Closed Neonit closed 7 months ago

Neonit commented 8 months ago

RTree.h unfortunately always defines the makros Min and Max, which may lead to issues with code using one of these names as identifier, e.g. class name (example: FastNoise2 in Blends.h). I suggest introducing a new makro with a name complying to usual standards (all uppercase), which takes over the value of Min/Max for projects/platforms using these makros.

Also, while compiling the tests, my compiler complained about std::numeric_limits not existing so I added #include <limits> to the tests in a separate commit.