search

VecPP / vecpp

A simple C++17 vector math library
Boost Software License 1.0
10 stars 4 forks source link

The angle constructors do not play well with constexpr #8

Closed FrancoisChabot closed 6 years ago

FrancoisChabot commented 6 years ago

I pushed a temporary fix, but the whole feature needs to be revisited with constexpr in mind.

Furthermore, this was only caught in the MSVC CI build. The vecpp::non_cste indirection did NOT catch this, which is worrisome.

FrancoisChabot commented 6 years ago

The good news is that the build passing was due to to the indirecction not being invoked properly.