davideberly / GeometricTools

A collection of source code for computing in the fields of mathematics, geometry, graphics, image analysis and physics.
Boost Software License 1.0
1.14k stars 214 forks source link

Fixed infinite recursion issue when compiling with clang #38

Closed tdeslandes closed 2 years ago

tdeslandes commented 2 years ago

When compiling with clang and -Winfinite-recursion this code returns an error. I assume the goal was to call the uint version of the method since there is a positive check just above, this changelist fixes the issue

davideberly commented 2 years ago

Yes, the goal was to call the uint version. I am currently focusing on GTL, and it does not have the int versions. Unit tests for that code all pass. Thank you for reporting this.