search

arturbac / fixed_math

FixedMath: A High-Performance C++23, C++17 Library for Fixed-Point 48.16 Arithmetic
https://arturbac.github.io/fixed_math/
Boost Software License 1.0
67 stars 4 forks source link

[BUG] include path is ambiguous #8

Open toge opened 2 months ago

toge commented 2 months ago

Describe the bug

In examples, the include path is fixedmath. https://github.com/arturbac/fixed_math/blob/v2.0.0/examples/brief-usage.cc#L5-L6

But the include path is fixed_math. https://github.com/arturbac/fixed_math/blob/master/fixed_lib/CMakeLists.txt#L25-L30

Which is valid?

I think fixedmath is valid because common.h includes <fixedmath/detail/type_traits.h>. https://github.com/arturbac/fixed_math/blob/v2.0.0/fixed_lib/include/fixedmath/detail/common.h#L8

Build info:

arturbac commented 2 months ago

Yes, looks like You are right. I use it by CPM dependency so I do not need to install and did not now about this. Will asap fix it.