catch3/src/catch2/catch_session.cpp, branch v3.0.0-preview3 Progmasoft Catch3: an independently maintained C++17 test framework with a parallel runner, property checks, snapshots and structured results. 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.
Remove the rest of old-style copyright headers 2020-08-30T12:45:05+00:00 Martin Hořeňovský 2020-08-30T12:41:39+00:00 c2453c2bf8278100386f7b97029fe5cc47a77755 






Split out ReporterFactory out of catch_interfaces_reporter.hpp
2020-08-23T20:35:01+00:00

Martin Hořeňovský

2020-08-23T19:19:26+00:00

6a08d401aabac2ead19c3eff17647a2b482f96b0












Split IReporterRegistry into its own header file
2020-08-23T19:01:04+00:00

Martin Hořeňovský

2020-08-23T16:16:10+00:00

9677df6d8b4a8b8e7a1b677068975af2caca5cc9

Doing this removes `<map>` from the include set of the base reporter
interface, and thus from bunch more TUs. This provides about 1.5%
improvements in the debug build of the static library, and 1% in
release build.





Doing this removes `<map>` from the include set of the base reporter
interface, and thus from bunch more TUs. This provides about 1.5%
improvements in the debug build of the static library, and 1% in
release build.







Don't include catch_random_number_generator.hpp in catch_session.cpp
2020-08-14T07:33:16+00:00

Martin Hořeňovský

2020-08-13T12:07:00+00:00

07fb96d42c3295b7b9d3f63396d290e0bbfbc024












Clara cleanups
2020-08-10T18:30:40+00:00

Martin Hořeňovský

2020-08-08T16:18:27+00:00

24b83edf8aca78c975c5589baf22a5de311498a7

* Clara is now split between a header and a cpp file.
* Removed the deprecated `+` and `+=` operators for composing
  a parser.
* Renamed `clara` and `detail` namespaces to be inline with the
rest of Catch2 (they are now `Clara` and `Detail` respectively).
* Taken most of user-exposed types out of the `Detail` namespace
completely (instead of using `using` directives to bring them into
the outer namespace).





* Clara is now split between a header and a cpp file.
* Removed the deprecated `+` and `+=` operators for composing
  a parser.
* Renamed `clara` and `detail` namespaces to be inline with the
rest of Catch2 (they are now `Clara` and `Detail` respectively).
* Taken most of user-exposed types out of the `Detail` namespace
completely (instead of using `using` directives to bring them into
the outer namespace).







Fixup missing transitive includes
2020-08-10T18:28:14+00:00

Martin Hořeňovský

2020-08-08T12:40:54+00:00

b824d06844ab3faf4ddffa036a0ea7ef0eec117b

Found after updating to newest MSVC...





Found after updating to newest MSVC...







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.







Apply some IWYU suggestions
2020-07-29T19:51:05+00:00

Martin Hořeňovský

2020-07-29T14:29:41+00:00

ed0ea3014942a952471ca32da466f5df213b5eb2

This is not nearly all of them, because IWYU does not support the
way Catch2 manages includes -- it expects that non-system includes
are done using `#include "foo/bar/baz.hpp"`, while Catch2 uses
`<foo/bar/baz.hpp>`. This causes trouble, because IWYU suggests
removing every single internal header, and then adding them again,
but using `""` in the include directive... the resulting suggestions
cannot be used without a lot of manual work, as they are largely
bogus.

For bonus points, IWYU also _loves_ to suggest kinda-random stdlib
headers for `size_t` and similar. Still, the resulting inclusion
graph is somewhat better than it was before.





This is not nearly all of them, because IWYU does not support the
way Catch2 manages includes -- it expects that non-system includes
are done using `#include "foo/bar/baz.hpp"`, while Catch2 uses
`<foo/bar/baz.hpp>`. This causes trouble, because IWYU suggests
removing every single internal header, and then adding them again,
but using `""` in the include directive... the resulting suggestions
cannot be used without a lot of manual work, as they are largely
bogus.

For bonus points, IWYU also _loves_ to suggest kinda-random stdlib
headers for `size_t` and similar. Still, the resulting inclusion
graph is somewhat better than it was before.







Replace all uses of std::unique_ptr with Catch::Detail::unique_ptr
2020-05-31T13:20:24+00:00

Martin Hořeňovský

2020-05-25T07:45:24+00:00

1d1ccf8f3c0c9e121b16afd4fafb850d14ef9fb0

Doing some benchmarking with ClangBuildAnalyzer suggests that
compiling Catch2's `SelfTest` spends 10% of the time instantiating
`std::unique_ptr` for some interface types required for registering
and running tests.

The lesser compilation overhead of `Catch::Detail::unique_ptr` should
significantly reduce that time.

The compiled implementation was also changed to use the custom impl,
to avoid having to convert between using `std::unique_ptr` and
`Catch::Detail::unique_ptr`. This will likely also improve the compile
times of the implementation, but that is less important than improving
compilation times of the user's TUs with tests.





Doing some benchmarking with ClangBuildAnalyzer suggests that
compiling Catch2's `SelfTest` spends 10% of the time instantiating
`std::unique_ptr` for some interface types required for registering
and running tests.

The lesser compilation overhead of `Catch::Detail::unique_ptr` should
significantly reduce that time.

The compiled implementation was also changed to use the custom impl,
to avoid having to convert between using `std::unique_ptr` and
`Catch::Detail::unique_ptr`. This will likely also improve the compile
times of the implementation, but that is less important than improving
compilation times of the user's TUs with tests.