Skip to content

Commit

Permalink
re-enable ldexp static assert test
Browse files Browse the repository at this point in the history
Signed-off-by: Ian <[email protected]>
  • Loading branch information
Rinzii committed Jan 13, 2025
1 parent 0d16ffe commit 10caa78
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions test/fmanip/ldexp_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,6 @@

// NOLINTBEGIN

static_assert(ccm::support::helpers::internal_ldexp(static_cast<float>(1.0), 2) == ccm::support::helpers::internal_ldexp(static_cast<float>(1.0), 2),
"ldexp failed static_assert test");

template <typename T>
class CcmathFmanipTests : public ::testing::Test
{
Expand All @@ -29,8 +26,7 @@ TYPED_TEST_SUITE(CcmathFmanipTests, TestTypes);

TYPED_TEST(CcmathFmanipTests, LdexpStaticAssert)
{
// TODO: IanP: Figure out why this static_assert fails
// EXPECT_EQ(ccm::ldexp(static_cast<TypeParam>(1.0), 0), 3.0);
static_assert(ccm::ldexp(static_cast<TypeParam>(1.0), 0) == ccm::ldexp(static_cast<TypeParam>(1.0), 0));
}

TYPED_TEST(CcmathFmanipTests, LdexpBasic)
Expand Down

0 comments on commit 10caa78

Please sign in to comment.