catch3/src/catch2/internal/catch_textflow.hpp, 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.
 Move-enable various parts of TextFlow 2023-12-27T20:43:49+00:00 Martin Hořeňovský 2023-12-27T20:43:40+00:00 b52d97855d794175e69abc6f0eceee9ec3c1bd0e This removes about 200 pointless copies from printing the help message (the original motivation for the change), and also nicely improves performance of the various reporters that depend on TextFlow. 
This removes about 200 pointless copies from printing the help
message (the original motivation for the change), and also nicely
improves performance of the various reporters that depend on
TextFlow.
Fix spelling 2023-04-29T10:55:51+00:00 Vertexwahn 2023-04-25T18:14:27+00:00 46539b6d9b1d429fcc2261d3e687482c878a9ddb 






Fix references to license file
2022-10-28T09:30:15+00:00

Lars Toenning

2022-10-28T09:22:53+00:00

f1084fb30990fe40c8f5b70034970c7eb4e1da72

The license file was renamed with 6a502cc2f5fedd59b3495b58708f0d6d987ed9e1





The license file was renamed with 6a502cc2f5fedd59b3495b58708f0d6d987ed9e1







Improve comments and names in TextFlow::Column
2021-11-01T18:14:37+00:00

Martin Hořeňovský

2021-11-01T18:14:37+00:00

70c4ec78fbbfadb19d520c72b83f33cd0915c078












Typedef Column::iterator as Column::const_iterator not vice versa
2021-10-31T12:01:41+00:00

Martin Hořeňovský

2021-10-31T12:01:41+00:00

455ae0c561b6254e05c5fe619b08968187d28a70












Return const_iterator from Column::begin/end const
2021-10-31T11:59:00+00:00

Martin Hořeňovský

2021-10-31T11:58:45+00:00

2520ad4b6e0dfacbbdf802f952495532361ccb02

This is what should normally happen, even if it does not change
anything given that `Column::const_iterator` is currently a typedef
for `Column::iterator`.





This is what should normally happen, even if it does not change
anything given that `Column::const_iterator` is currently a typedef
for `Column::iterator`.







Add standardized copyright notice + SPDX identifier to source files
2020-08-30T13:43:45+00:00

Martin Hořeňovský

2020-08-30T13:43:45+00:00

90aeffb97d6942d16f75a56b11e10fc0b010995a

This should also be done for test files, but that has lower priority.





This should also be done for test files, but that has lower priority.







Standardize include guard patterns to FILE_NAME_EXTENSION_INCLUDED
2020-08-30T12:09:27+00:00

Martin Hořeňovský

2020-08-29T18:48:32+00:00

a822cb9717b64b3b17047fe1ab4dc727c2ff39c4

This commit also strips the old copyright comment header in touched
files, as those will also be replaced with a more standardized and
machine-friendly version.





This commit also strips the old copyright comment header in touched
files, as those will also be replaced with a more standardized and
machine-friendly version.







Split TextFlow out from Clara
2020-08-03T21:40:14+00:00

Martin Hořeňovský

2020-08-01T18:32:26+00:00

e7aa432850c34557e857d8675bff9b3f03de41b8

Now that it has its own header, various reporter TUs that want to
format text do not have to also include Clara. Together with
outlining implementations from a header into a separate TU, this
has noticeably improved the compilation times of the testing impl.

As part of this split, I also implemented some improvements to the
TextFlow code in comparison to the upstream code. These are:

* Replaced the `Spacer` type with a free function that constructs
special `Column` that does the same thing.
* Generic performance improvements, such as eliminating needless
allocations, reserving space in needed allocations, and using smarter
algorithms in some places.
* Because `Column` only ever stored 1 string in its vector, it now
holds the string directly instead.





Now that it has its own header, various reporter TUs that want to
format text do not have to also include Clara. Together with
outlining implementations from a header into a separate TU, this
has noticeably improved the compilation times of the testing impl.

As part of this split, I also implemented some improvements to the
TextFlow code in comparison to the upstream code. These are:

* Replaced the `Spacer` type with a free function that constructs
special `Column` that does the same thing.
* Generic performance improvements, such as eliminating needless
allocations, reserving space in needed allocations, and using smarter
algorithms in some places.
* Because `Column` only ever stored 1 string in its vector, it now
holds the string directly instead.