Skip to content

Commit 94e243e

Browse files
jwellbelovejwellbelove
committed
Fix unaligned_type floating point checks in unit test
1 parent ac8566a commit 94e243e

File tree

6 files changed

+84
-727
lines changed

6 files changed

+84
-727
lines changed

‎test/CMakeLists.txt‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -290,7 +290,6 @@ add_executable(etl_tests
290290
test_type_select.cpp
291291
test_type_traits.cpp
292292
test_unaligned_type.cpp
293-
test_unaligned_type_constexpr.cpp
294293
test_unaligned_type_ext.cpp
295294
test_uncopyable.cpp
296295
test_unordered_map.cpp

‎test/test_unaligned_type.cpp‎

Lines changed: 36 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -355,19 +355,19 @@ namespace
355355
}
356356

357357
// float
358-
CHECK_FLOAT_SAME(etl::le_float_t(3.1415927f), etl::le_float_t(3.1415927f));
359-
CHECK_FLOAT_SAME(3.1415927f, etl::le_float_t(3.1415927f));
360-
CHECK_FLOAT_SAME(etl::le_float_t(3.1415927f), 3.1415927f);
358+
CHECK_CLOSE(etl::le_float_t(3.1415927f), etl::le_float_t(3.1415927f), 0.001);
359+
CHECK_CLOSE(3.1415927f, etl::le_float_t(3.1415927f), 0.001);
360+
CHECK_CLOSE(etl::le_float_t(3.1415927f), 3.1415927f, 0.001);
361361

362362
// double
363-
CHECK_FLOAT_SAME(etl::le_double_t(3.1415927), etl::le_double_t(3.1415927));
364-
CHECK_FLOAT_SAME(3.1415927, etl::le_double_t(3.1415927));
365-
CHECK_FLOAT_SAME(etl::le_double_t(3.1415927), 3.1415927);
363+
CHECK_CLOSE(etl::le_double_t(3.1415927), etl::le_double_t(3.1415927), 0.001);
364+
CHECK_CLOSE(3.1415927, etl::le_double_t(3.1415927), 0.001);
365+
CHECK_CLOSE(etl::le_double_t(3.1415927), 3.1415927, 0.001);
366366

367367
// long double
368-
CHECK_FLOAT_SAME(etl::le_long_double_t(3.1415927L), etl::le_long_double_t(3.1415927L));
369-
CHECK_FLOAT_SAME(3.1415927L, etl::le_long_double_t(3.1415927L));
370-
CHECK_FLOAT_SAME(etl::le_long_double_t(3.1415927L), 3.1415927L);
368+
CHECK_CLOSE(etl::le_long_double_t(3.1415927L), etl::le_long_double_t(3.1415927L), 0.001);
369+
CHECK_CLOSE(3.1415927L, etl::le_long_double_t(3.1415927L), 0.001);
370+
CHECK_CLOSE(etl::le_long_double_t(3.1415927L), 3.1415927L, 0.001);
371371
}
372372

373373
//*************************************************************************
@@ -503,19 +503,19 @@ namespace
503503
}
504504

505505
// float
506-
CHECK_FLOAT_SAME(etl::be_float_t(3.1415927f), etl::be_float_t(3.1415927f));
507-
CHECK_FLOAT_SAME(3.1415927f, etl::be_float_t(3.1415927f));
508-
CHECK_FLOAT_SAME(etl::be_float_t(3.1415927f), 3.1415927f);
506+
CHECK_CLOSE(etl::be_float_t(3.1415927f), etl::be_float_t(3.1415927f), 0.001);
507+
CHECK_CLOSE(3.1415927f, etl::be_float_t(3.1415927f), 0.001);
508+
CHECK_CLOSE(etl::be_float_t(3.1415927f), 3.1415927f, 0.001);
509509

510510
// double
511-
CHECK_FLOAT_SAME(etl::be_double_t(3.1415927), etl::be_double_t(3.1415927));
512-
CHECK_FLOAT_SAME(3.1415927, etl::be_double_t(3.1415927));
513-
CHECK_FLOAT_SAME(etl::be_double_t(3.1415927), 3.1415927);
511+
CHECK_CLOSE(etl::be_double_t(3.1415927), etl::be_double_t(3.1415927), 0.001);
512+
CHECK_CLOSE(3.1415927, etl::be_double_t(3.1415927), 0.001);
513+
CHECK_CLOSE(etl::be_double_t(3.1415927), 3.1415927, 0.001);
514514

515515
// long double
516-
CHECK_FLOAT_SAME(etl::be_long_double_t(3.1415927L), etl::be_long_double_t(3.1415927L));
517-
CHECK_FLOAT_SAME(3.1415927L, etl::be_long_double_t(3.1415927L));
518-
CHECK_FLOAT_SAME(etl::be_long_double_t(3.1415927L), 3.1415927L);
516+
CHECK_CLOSE(etl::be_long_double_t(3.1415927L), etl::be_long_double_t(3.1415927L), 0.001);
517+
CHECK_CLOSE(3.1415927L, etl::be_long_double_t(3.1415927L), 0.001);
518+
CHECK_CLOSE(etl::be_long_double_t(3.1415927L), 3.1415927L, 0.001);
519519
}
520520

521521
//*************************************************************************
@@ -648,17 +648,17 @@ namespace
648648
// float
649649
etl::le_float_t le_float;
650650
le_float = 3.1415927f;
651-
CHECK_FLOAT_SAME(3.1415927f, le_float);
651+
CHECK_CLOSE(3.1415927f, le_float, 0.001);
652652

653653
// double
654654
etl::le_double_t le_double;
655655
le_double = 3.1415927;
656-
CHECK_FLOAT_SAME(3.1415927, le_double);
656+
CHECK_CLOSE(3.1415927, le_double, 0.001);
657657

658658
// long double
659659
etl::le_long_double_t le_long_double;
660660
le_long_double = 3.1415927L;
661-
CHECK_FLOAT_SAME(3.1415927L, le_long_double);
661+
CHECK_CLOSE(3.1415927L, le_long_double, 0.001);
662662
}
663663

664664
//*************************************************************************
@@ -717,17 +717,17 @@ namespace
717717
// float
718718
etl::be_float_t be_float;
719719
be_float = 3.1415927f;
720-
CHECK_FLOAT_SAME(3.1415927f, be_float);
720+
CHECK_CLOSE(3.1415927f, be_float, 0.001);
721721

722722
// double
723723
etl::be_double_t be_double;
724724
be_double = 3.1415927;
725-
CHECK_FLOAT_SAME(3.1415927, be_double);
725+
CHECK_CLOSE(3.1415927, be_double, 0.001);
726726

727727
// long double
728728
etl::be_long_double_t be_long_double;
729729
be_long_double = 3.1415927L;
730-
CHECK_FLOAT_SAME(3.1415927L, be_long_double);
730+
CHECK_CLOSE(3.1415927L, be_long_double, 0.001);
731731
}
732732

733733
//*************************************************************************
@@ -766,10 +766,10 @@ namespace
766766
le_v3 = be_v1; // Assign le from be.
767767
be_v3 = le_v1; // Assign be from le.
768768

769-
CHECK_FLOAT_SAME(3.1415927f, le_v2);
770-
CHECK_FLOAT_SAME(3.1415927f, be_v2);
771-
CHECK_FLOAT_SAME(3.1415927f, le_v3);
772-
CHECK_FLOAT_SAME(3.1415927f, be_v3);
769+
CHECK_CLOSE(3.1415927f, le_v2, 0.001);
770+
CHECK_CLOSE(3.1415927f, be_v2, 0.001);
771+
CHECK_CLOSE(3.1415927f, le_v3, 0.001);
772+
CHECK_CLOSE(3.1415927f, be_v3, 0.001);
773773
}
774774

775775
//*************************************************************************
@@ -787,10 +787,10 @@ namespace
787787
le_v3 = be_v1; // Assign le from be.
788788
be_v3 = le_v1; // Assign be from le.
789789

790-
CHECK_FLOAT_SAME(3.1415927, le_v2);
791-
CHECK_FLOAT_SAME(3.1415927, be_v2);
792-
CHECK_FLOAT_SAME(3.1415927, le_v3);
793-
CHECK_FLOAT_SAME(3.1415927, be_v3);
790+
CHECK_CLOSE(3.1415927, le_v2, 0.001);
791+
CHECK_CLOSE(3.1415927, be_v2, 0.001);
792+
CHECK_CLOSE(3.1415927, le_v3, 0.001);
793+
CHECK_CLOSE(3.1415927, be_v3, 0.001);
794794
}
795795

796796
//*************************************************************************
@@ -808,10 +808,10 @@ namespace
808808
le_v3 = be_v1; // Assign le from be.
809809
be_v3 = le_v1; // Assign be from le.
810810

811-
CHECK_FLOAT_SAME(3.1415927L, le_v2);
812-
CHECK_FLOAT_SAME(3.1415927L, be_v2);
813-
CHECK_FLOAT_SAME(3.1415927L, le_v3);
814-
CHECK_FLOAT_SAME(3.1415927L, be_v3);
811+
CHECK_CLOSE(3.1415927L, le_v2, 0.001);
812+
CHECK_CLOSE(3.1415927L, be_v2, 0.001);
813+
CHECK_CLOSE(3.1415927L, le_v3, 0.001);
814+
CHECK_CLOSE(3.1415927L, be_v3, 0.001);
815815
}
816816

817817
//*************************************************************************

0 commit comments

Comments
 (0)