catch3/src/catch2/catch_message.cpp, branch v3.2.0

Fix references to license file

2022-10-28T09:30:15+00:00

The license file was renamed with 6a502cc2f5fedd59b3495b58708f0d6d987ed9e1

Replace uses of std::move and std::forward with macros

2021-08-16T14:22:19+00:00

This improves the SelfTest build times by about 3% (measured
with Clang 10 on random Linux box I had lying around).

Inline MessageBuilder's constructor

2021-05-14T14:00:15+00:00

Add standardized copyright notice + SPDX identifier to source files

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

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

Cleanup some stuff found by MSVC's /analyze

2020-08-20T18:42:21+00:00

* Added some missing `noexcept`s on custom destructors.
* Fixed `std::move` being called on a const-reference.
* Initialized `ScopedMessage::m_moved` in class definition, instead
of doing so in constructors explicitly.
* Turned some `enum`s into `enum class`es.
* Initialized `StreamingReporterBase::currentTestCaseInfo` in class
definition.
* Some cleanups in SelfTest code.

Fix invalid isspace call detected by PREfast

2020-07-12T19:00:25+00:00

D:\vcpkg\toolsrc\include\catch2\catch.hpp(11285): warning C6330: 'char' passed as _Param_(1) when 'unsigned char' is required in call to 'isspace'.
D:\vcpkg\toolsrc\include\catch2\catch.hpp(11288): warning C6330: 'char' passed as _Param_(1) when 'unsigned char' is required in call to 'isspace'.

ISO/IEC 9899:2011:
"7.4 Character handling "/1
[...] In all cases the argument is an int, the value of which shall be
representable as an unsigned char or shall equal the value of the macro
EOF. If the argument has any other value, the behavior is undefined.

This means if isspace was passed a character like ñ it could corrupt
memory without the static_cast to treat it as a positive value after
integral promotion (and C libraries commonly use the int index supplied
as a key into a table which result in out of bounds access if the
resulting int is negative).

Split MessageInfo into its own header

2020-05-10T18:20:52+00:00

This is first step towards splitting apart reporter implementation
and `catch_string.hpp`.

Standardize header names and file locations

2020-04-24T16:58:44+00:00

This is both a really big and a really small commit. It is small in
that it only contains renaming, moving and modification of include
directives caused by this.

It is really big in the obvious way of touching something like 200
files.

The new rules for naming files is simple: headers use the `.hpp`
extension. The rules for physical file layout is still kinda in
progress, but the basics are also simple:
 * Significant parts of functionality get their own subfolder
   * Benchmarking is in `catch2/benchmark`
   * Matchers are in `catch2/matchers`
   * Generators are in `catch2/generators`
   * Reporters are in `catch2/reporters`
   * Baseline testing facilities are in `catch2/`
 * Various top level folders also contain `internal` subfolder,
   with files that users probably do not want to include directly,
   at least not until they have to write something like their own
   reporter.
    * The exact files in these subfolders is likely to change later
      on

Note that while some includes were cleaned up in this commit, it
is only the low hanging fruit and further cleanup using automatic
tooling will happen later.

Also note that various include guards, copyright notices and file
headers will also be standardized later, rather than in this commit.

Some refactorings:

2020-02-16T14:25:23+00:00

 - Overrides added
 - usages of push_back() replaced with emplace_back()
 - Loop variable made const-refernce
 - NULL replaced with nullptr
 - Names used in the declaration and definition unified
 - size() replaced with empty
 - Identical cases merged