From ee663d08f648c83e3478d80fb6231d0cebf34fb8 Mon Sep 17 00:00:00 2001 From: conrad Date: Mon, 7 Jul 2025 21:37:13 +1000 Subject: [PATCH] remove outdated comments --- tests2/bounds.cpp | 2 +- tests2/fn_accu.cpp | 2 +- tests2/mat_plus.cpp | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tests2/bounds.cpp b/tests2/bounds.cpp index b7ecaef5..736f97f5 100644 --- a/tests2/bounds.cpp +++ b/tests2/bounds.cpp @@ -31,7 +31,7 @@ TEST_CASE("bounds_1", "[bounds]") REQUIRE_NOTHROW( A(n_rows-1,n_cols-1) = 0 ); - // out of bounds access will throw unless ARMA_NO_DEBUG is defined + // out of bounds access will throw REQUIRE_THROWS( A(n_rows,n_cols) = 0 ); } diff --git a/tests2/fn_accu.cpp b/tests2/fn_accu.cpp index bba6e061..e23ec356 100644 --- a/tests2/fn_accu.cpp +++ b/tests2/fn_accu.cpp @@ -114,7 +114,7 @@ TEST_CASE("fn_accu_4", "[accu]") REQUIRE( accu(A % B) == Approx(double((2*4)*(A.n_rows*A.n_cols))) ); REQUIRE( accu(A(span::all,span::all) % B(span::all,span::all)) == Approx(double((2*4)*(A.n_rows*A.n_cols))) ); - // A and C matrices are non-conformat, so accu() will throw unless ARMA_NO_DEBUG is defined + // A and C matrices are non-conformant REQUIRE_THROWS( accu(A % C) ); REQUIRE_THROWS( accu(A(span::all,span::all) % C(span::all,span::all)) ); } diff --git a/tests2/mat_plus.cpp b/tests2/mat_plus.cpp index a67be0f4..b8bd81b3 100644 --- a/tests2/mat_plus.cpp +++ b/tests2/mat_plus.cpp @@ -110,7 +110,7 @@ TEST_CASE("mat_plus_2", "[plus]") mat X(6,5); mat Y; - REQUIRE_THROWS( Y = A+X ); // adding non-conformant matrices will throw unless ARMA_NO_DEBUG is defined + REQUIRE_THROWS( Y = A+X ); // adding non-conformant matrices will throw }