admin/omath
1
0
Fork 0
mirror of https://github.com/orange-cpp/omath.git synced 2026-02-13 07:03:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

Adds nodiscard attribute to simplex functions

Browse Source 
Applies the `[[nodiscard]]` attribute to several functions within the `omath::collision` namespace to improve code safety and signal potential misuse when return values are ignored. This encourages developers to handle the results of these functions appropriately.
This commit is contained in:
Orange
2025-11-11 23:36:19 +03:00
parent 05bc7577b5
commit 28ef194586
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
include/omath/collision/simplex.hpp
View File

@@ -47,10 +47,13 @@ namespace omath::collision
++m_size; ++m_size;
} }
[[nodiscard]]
constexpr const VectorType& operator[](std::size_t i) const noexcept constexpr const VectorType& operator[](std::size_t i) const noexcept
{ {
return m_points[i]; return m_points[i];
} }
[[nodiscard]]
constexpr VectorType& operator[](std::size_t i) noexcept constexpr VectorType& operator[](std::size_t i) noexcept
{ {
return m_points[i]; return m_points[i];
@@ -126,12 +129,14 @@ namespace omath::collision
} }
template<class V> template<class V>
[[nodiscard]]
static constexpr bool near_zero(const V& v, const float eps = 1e-7f) static constexpr bool near_zero(const V& v, const float eps = 1e-7f)
{ {
return v.dot(v) <= eps * eps; return v.dot(v) <= eps * eps;
} }
template<class V> template<class V>
[[nodiscard]]
static constexpr V any_perp(const V& v) static constexpr V any_perp(const V& v)
{ {
for (const auto& dir : {V{1, 0, 0}, {0, 1, 0}, {0, 0, 1}}) for (const auto& dir : {V{1, 0, 0}, {0, 1, 0}, {0, 0, 1}})
@@ -140,6 +145,7 @@ namespace omath::collision
std::unreachable(); std::unreachable();
} }
[[nodiscard]]
constexpr bool handle_line(VectorType& direction) constexpr bool handle_line(VectorType& direction)
{ {
const auto& a = m_points[0]; const auto& a = m_points[0];