catch3/tests/SelfTest, branch constexpr_approx Progmasoft Catch3: an independently maintained C++17 test framework with a parallel runner, property checks, snapshots and structured results. Handle ANSI escape sequences when performing column wrapping (#2849) 2024-05-04T21:43:52+00:00 Jeremy Rifkin 2024-05-04T21:43:52+00:00 8ce2426e53b2596a9823b4a314f72f1c3b30be82 This PR adds functionality to skip around ANSI escape sequences in catch_textflow so they do not contribute to line length and line wrapping code does not split escape sequences in the middle. I've implemented this by creating a AnsiSkippingString abstraction that has a bidirectional iterator that can skip around escape sequences while iterating. Additionally I refactored Column::const_iterator to be iterator-based rather than index-based so this abstraction is a simple drop-in for std::string. Currently only color sequences are handled, other escape sequences are left unaffected. Motivation: Text with ANSI color sequences gets messed up when being output by Catch2 #2833. 
This PR adds functionality to skip around ANSI escape sequences in catch_textflow so they do not contribute to line length and line wrapping code does not split escape sequences in the middle. I've implemented this by creating a AnsiSkippingString abstraction that has a bidirectional iterator that can skip around escape sequences while iterating. Additionally I refactored Column::const_iterator to be iterator-based rather than index-based so this abstraction is a simple drop-in for std::string.

Currently only color sequences are handled, other escape sequences are left unaffected.

Motivation: Text with ANSI color sequences gets messed up when being output by Catch2 #2833.
 Don't include numerically unstable tests in approvals 2024-04-30T17:21:27+00:00 Martin Hořeňovský 2024-04-30T17:21:27+00:00 a654e4b0382f3c41bdb911739b05276a11cf644a 






Default StringMaker<FloatingPointType>::precision to max_digits10
2024-04-30T15:18:48+00:00

Martin Hořeňovský

2024-04-30T14:43:05+00:00

ef713582d268d9b1b15381325021d7e0f91d6ab9

`max_digits10` is the number of decimal digits that the type _can_
represent, in other words, the number of decimal digits needed to
disambiguate any two floating point numbers when serialized to
a string.





`max_digits10` is the number of decimal digits that the type _can_
represent, in other words, the number of decimal digits needed to
disambiguate any two floating point numbers when serialized to
a string.







Use Catch::StringMaker for output in WithinRelMatcher (#2846)
2024-04-15T11:35:39+00:00

Ian Bell

2024-04-15T11:35:39+00:00

53ddf37af4488cac7724761858ae3cca9d2d65e7

The WithinRelMatcher does not listen to the precision specification; it just naively pipes to stringstream. If you specify the precision, that has no impact on the outputted values when the match fails.

This fix makes the WithinRelMatcher listen to the precision (https://github.com/catchorg/Catch2/blob/devel/docs/tostring.md#floating-point-precision) specified by: Catch::StringMaker<float>::precision




The WithinRelMatcher does not listen to the precision specification; it just naively pipes to stringstream. If you specify the precision, that has no impact on the outputted values when the match fails.

This fix makes the WithinRelMatcher listen to the precision (https://github.com/catchorg/Catch2/blob/devel/docs/tostring.md#floating-point-precision) specified by: Catch::StringMaker<float>::precision






Fix & extend tests for comparing const instances of zero lit types
2024-04-08T11:15:35+00:00

Martin Hořeňovský

2024-04-08T11:15:31+00:00

05fb437cbbf60de794a7101c5e6ef8f650755ce4












Fix assertion for const instance of std::ordering
2024-04-08T09:33:43+00:00

Martin Hořeňovský

2024-04-08T09:02:22+00:00

71b11c4e3328090d2ec72268299e3fb9e0ac3f8e

The issue was that `capture_by_value` was meant to be specialized
by plain type, e.g. `capture_by_value<std::weak_ordering>`, but
the TMP in Decomposer did not properly throw away `const` (and
`volatile`) qualifiers, before taking the value of `capture_by_value`.





The issue was that `capture_by_value` was meant to be specialized
by plain type, e.g. `capture_by_value<std::weak_ordering>`, but
the TMP in Decomposer did not properly throw away `const` (and
`volatile`) qualifiers, before taking the value of `capture_by_value`.







Fix preprocessor check for enabling FP reproducibility tests
2024-04-06T18:27:15+00:00

Martin Hořeňovský

2024-04-06T18:19:47+00:00

0a6a2ce88764cc8c01411fee68834571a5b9a0a1

This solves warning about the `uniform_fp_test_params` helper
being unused on M1 mac (or any other platform using GCC and not
using SSE2).

Closes #2845





This solves warning about the `uniform_fp_test_params` helper
being unused on M1 mac (or any other platform using GCC and not
using SSE2).

Closes #2845







Optimize 64x64 extended multiplication implementation
2024-04-03T11:28:25+00:00

Martin Hořeňovský

2024-04-02T16:09:34+00:00

d99eb8bec8de53639540820ee2a6563b88c4ba8c

Now we use intrinsics when possible, and fallback to optimized
implementation in portable C++. The difference is about 4x when
we can use intrinsics and about 2x when we cannot.

This should speed up our Lemire's algorithm implementation nicely.





Now we use intrinsics when possible, and fallback to optimized
implementation in portable C++. The difference is about 4x when
we can use intrinsics and about 2x when we cannot.

This should speed up our Lemire's algorithm implementation nicely.







Use SizedUnsignedType_t to pick UnsignedType for uniform_integer_distribution
2024-04-03T11:27:10+00:00

Martin Hořeňovský

2024-04-03T10:29:51+00:00

f181de9df4c43f188ae932a412af32a0981c9067

The previously used `make_unsigned` approach combined with the overload
set of `extendedMult` caused compilation issues on MacOS platform. By
forcing the selection to be one of `std::uintX_t` types we don't need
to complicate the overload set further.





The previously used `make_unsigned` approach combined with the overload
set of `extendedMult` caused compilation issues on MacOS platform. By
forcing the selection to be one of `std::uintX_t` types we don't need
to complicate the overload set further.







Ignore C++98 related compiler warnings
2024-03-12T21:56:37+00:00

Chris Thrasher

2024-03-12T17:27:06+00:00

88f4ec3cc5620c45086e972bd372d3ba3df8c3d3

Catch2 has long since required a standard newer than C++98 so we
can safely ignore any warnings related to such old standards.





Catch2 has long since required a standard newer than C++98 so we
can safely ignore any warnings related to such old standards.