catch3/src/catch2/internal/catch_clara.cpp, branch constexpr_approx Progmasoft Catch3: an independently maintained C++17 test framework with a parallel runner, property checks, snapshots and structured results. 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.
Avoid copying TokenStreams while parsing 2023-12-27T20:14:12+00:00 Martin Hořeňovský 2023-12-27T20:11:34+00:00 eaafd076743407e930a11297425f33c1f78220b0 The code is now even worse mess than before, due to the ad-hoc implementation of Result-ish type based on virtual functions in Clara, but it has dropped the allocations for empty binary down to 151. 
The code is now even worse mess than before, due to the ad-hoc
implementation of Result-ish type based on virtual functions in
Clara, but it has dropped the allocations for empty binary down
to 151.
Args and TokenStream use StringRefs instead of std::strings 2023-12-27T15:24:33+00:00 Martin Hořeňovský 2023-12-27T15:19:49+00:00 5d637d4c6b72dc21e2e198894ba0527729fba535 Because TokenStream is copied around a lot, moving it to use `StringRef` removes _a lot_ of allocations per `Opt` in the parser. Args are not copied around much, but changing them as well makes it obvious that they do not participate in the ownership. The changes add up to removing ~180 allocations for "empty" invocation of the test binary. (`./tests/ExtraTests/NoTests --allow-running-no-tests -o /dev/null` is down to 317 allocs) 
Because TokenStream is copied around a lot, moving it to use
`StringRef` removes _a lot_ of allocations per `Opt` in the parser.
Args are not copied around much, but changing them as well makes it
obvious that they do not participate in the ownership.

The changes add up to removing ~180 allocations for "empty"
invocation of the test binary.
(`./tests/ExtraTests/NoTests --allow-running-no-tests -o /dev/null`
is down to 317 allocs)
Use StringRef for Opt's optNames 2023-12-26T15:03:22+00:00 Martin Hořeňovský 2023-12-26T13:01:15+00:00 cd3c7ebe873c3da91ce57c6b2221ba95ee7f93e6 Removes another ~70 allocations. 
Removes another ~70 allocations.
Cleanup help string construction in Clara 2023-12-26T13:57:29+00:00 Martin Hořeňovský 2023-12-23T15:06:30+00:00 822c44a20395c6194d527110d59f7513a55b885c * Clara::Opt::getHelpColumns returns single item It could never return multiple items, but for some reason it was wrapping that single item in a vector. * Use ReusableStringStream in Clara * Reserve HelpColumns ahead of time * Use StringRef for descriptions in HelpColumn type The combination of these changes ends up removing about 7% (~200) of allocations when Catch2 has to prepare output for `-h`. 
* Clara::Opt::getHelpColumns returns single item

  It could never return multiple items, but for some reason it
  was wrapping that single item in a vector.

* Use ReusableStringStream in Clara
* Reserve HelpColumns ahead of time
* Use StringRef for descriptions in HelpColumn type

The combination of these changes ends up removing about 7% (~200)
of allocations when Catch2 has to prepare output for `-h`.
Store Opt/Arg hint and description as StringRef 2023-12-26T13:40:45+00:00 Martin Hořeňovský 2023-12-23T15:02:37+00:00 2295d2c8ccc0dba4b35adb95ef8946a05996df87 There is no good reason for these to be std::strings, as these are just (optional) constants for nice user output. This ends up reducing the allocations significantly. When measuring allocations when running no tests, the changes are `tests/SelfTest` 9213 -> 7705 `tests/ExtraTests/NoTests` 3723 -> 2215 
There is no good reason for these to be std::strings, as these
are just (optional) constants for nice user output. This ends up
reducing the allocations significantly.

When measuring allocations when running no tests, the changes are
`tests/SelfTest` 9213 -> 7705
`tests/ExtraTests/NoTests` 3723 -> 2215
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
Tiny cleanup in catch_clara.cpp 2022-04-24T11:31:25+00:00 Martin Hořeňovský 2022-04-24T10:33:16+00:00 880285b433e90fc8d757cb65217016914b139196 






Some IWYU include cleanups
2022-04-14T15:16:01+00:00

Martin Hořeňovský

2022-04-14T15:16:01+00:00

c742ea9ad97a34c8f652dd7f67012d5b3f818c2d












Demote some single character strings into plain chars
2021-10-03T08:37:30+00:00

Martin Hořeňovský

2021-10-03T08:09:10+00:00

928e198ef289c2c259ad0d6ee2b3741c3a2b185a