Skip to content
Snippets Groups Projects
Commit ea0c5bad authored by Stéphane Del Pino's avatar Stéphane Del Pino
Browse files

Remove useless test in determinant

Also add a LCOV_EXCL_LINE marker. lcov produced a false negative probably due to
a forced inline.
parent d48b9ee9
No related branches found
No related tags found
No related merge requests found
...@@ -20,8 +20,7 @@ class TinyMatrix ...@@ -20,8 +20,7 @@ class TinyMatrix
static_assert((N > 0), "TinyMatrix size must be strictly positive"); static_assert((N > 0), "TinyMatrix size must be strictly positive");
PUGS_FORCEINLINE PUGS_FORCEINLINE
constexpr size_t constexpr size_t _index(const size_t& i, const size_t& j) const noexcept // LCOV_EXCL_LINE (due to forced inline)
_index(const size_t& i, const size_t& j) const noexcept
{ {
return i * N + j; return i * N + j;
} }
...@@ -344,8 +343,6 @@ det(const TinyMatrix<N, T>& A) ...@@ -344,8 +343,6 @@ det(const TinyMatrix<N, T>& A)
} }
} }
const size_t I = index[i]; const size_t I = index[i];
if (M(I, i) == 0)
return 0;
const T inv_Mii = 1. / M(I, i); const T inv_Mii = 1. / M(I, i);
for (size_t k = i + 1; k < N; ++k) { for (size_t k = i + 1; k < N; ++k) {
const size_t K = index[k]; const size_t K = index[k];
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment