From 55f1cc029859d76f72b6fdfc5df3e1c211542f33 Mon Sep 17 00:00:00 2001
From: Bartek Kryza <bkryza@gmail.com>
Date: Tue, 3 Aug 2021 23:47:12 +0200
Subject: [PATCH] Updated clang-format to version 12

---
 Makefile                         |   2 +-
 src/uml/sequence_diagram_model.h |   2 +-
 tests/catch.h                    | 150 ++++++++++++++++---------------
 tests/t00002/t00002.cc           |   4 +-
 tests/t00003/t00003.cc           |   8 +-
 tests/t00004/t00004.cc           |   4 +-
 tests/t00011/t00011.cc           |   2 +-
 tests/t00013/t00013.cc           |   2 +-
 tests/t00015/t00015.cc           |   3 +-
 tests/t00021/t00021.cc           |  16 ++--
 tests/t00022/t00022.cc           |   8 +-
 tests/t00023/t00023.cc           |   6 +-
 tests/t00024/t00024.cc           |   8 +-
 tests/t00025/t00025.cc           |   8 +-
 tests/t00027/t00027.cc           |   8 +-
 tests/t20001/t20001.cc           |   4 +-
 16 files changed, 121 insertions(+), 114 deletions(-)

diff --git a/Makefile b/Makefile
index 002f4201..cc053baa 100644
--- a/Makefile
+++ b/Makefile
@@ -62,4 +62,4 @@ init_compile_commands: debug
 
 .PHONY: clang-format
 clang-format:
-	docker run --rm -v $(CURDIR):/root/sources bkryza/clang-format-check:1.2
+	docker run --rm -v $(CURDIR):/root/sources bkryza/clang-format-check:1.3
diff --git a/src/uml/sequence_diagram_model.h b/src/uml/sequence_diagram_model.h
index 504e3778..395a3714 100644
--- a/src/uml/sequence_diagram_model.h
+++ b/src/uml/sequence_diagram_model.h
@@ -55,7 +55,7 @@ struct diagram {
 
     std::map<std::string, activity> sequences;
 
-    void sort() {}
+    void sort() { }
 };
 }
 }
diff --git a/tests/catch.h b/tests/catch.h
index 9a6a49f9..f531df86 100644
--- a/tests/catch.h
+++ b/tests/catch.h
@@ -853,8 +853,9 @@ constexpr auto operator"" _catch_sr(
     INTERNAL_CATCH_MAKE_TYPE_LIST2(INTERNAL_CATCH_REMOVE_PARENS(__VA_ARGS__))
 #else
 #define INTERNAL_CATCH_MAKE_TYPE_LIST2(...)                                    \
-    INTERNAL_CATCH_EXPAND_VARGS(decltype(                                      \
-        get_wrapper<INTERNAL_CATCH_REMOVE_PARENS_GEN(__VA_ARGS__)>()))
+    INTERNAL_CATCH_EXPAND_VARGS(                                               \
+        decltype(get_wrapper<INTERNAL_CATCH_REMOVE_PARENS_GEN(                 \
+                __VA_ARGS__)>()))
 #define INTERNAL_CATCH_MAKE_TYPE_LIST(...)                                     \
     INTERNAL_CATCH_EXPAND_VARGS(INTERNAL_CATCH_MAKE_TYPE_LIST2(                \
         INTERNAL_CATCH_REMOVE_PARENS(__VA_ARGS__)))
@@ -1277,8 +1278,8 @@ template <typename> struct true_given : std::true_type {
 };
 struct is_callable_tester {
     template <typename Fun, typename... Args>
-    true_given<decltype(
-        std::declval<Fun>()(std::declval<Args>()...))> static test(int);
+    true_given<decltype(std::declval<Fun>()(
+        std::declval<Args>()...))> static test(int);
     template <typename...> std::false_type static test(...);
 };
 
@@ -1634,8 +1635,8 @@ struct AutoReg : NonCopyable {
         };                                                                     \
         static int INTERNAL_CATCH_UNIQUE_NAME(globalRegistrar) = []() {        \
             using TestInit = typename create<TestName,                         \
-                decltype(                                                      \
-                    get_wrapper<INTERNAL_CATCH_REMOVE_PARENS(TmplTypes)>()),   \
+                decltype(get_wrapper<INTERNAL_CATCH_REMOVE_PARENS(             \
+                        TmplTypes)>()),                                        \
                 TypeList<INTERNAL_CATCH_MAKE_TYPE_LISTS_FROM_TYPES(            \
                     INTERNAL_CATCH_REMOVE_PARENS(TypesList))>>::type;          \
             TestInit t;                                                        \
@@ -1851,8 +1852,8 @@ struct AutoReg : NonCopyable {
         };                                                                     \
         static int INTERNAL_CATCH_UNIQUE_NAME(globalRegistrar) = []() {        \
             using TestInit = typename create<TestNameClass,                    \
-                decltype(                                                      \
-                    get_wrapper<INTERNAL_CATCH_REMOVE_PARENS(TmplTypes)>()),   \
+                decltype(get_wrapper<INTERNAL_CATCH_REMOVE_PARENS(             \
+                        TmplTypes)>()),                                        \
                 TypeList<INTERNAL_CATCH_MAKE_TYPE_LISTS_FROM_TYPES(            \
                     INTERNAL_CATCH_REMOVE_PARENS(TypesList))>>::type;          \
             TestInit t;                                                        \
@@ -2152,7 +2153,7 @@ inline id performOptionalSelector(id obj, SEL sel)
 #define CATCH_UNSAFE_UNRETAINED
 #define CATCH_ARC_STRONG
 #else
-inline void arcSafeRelease(NSObject *) {}
+inline void arcSafeRelease(NSObject *) { }
 inline id performOptionalSelector(id obj, SEL sel)
 {
 #ifdef __clang__
@@ -2194,8 +2195,9 @@ template <typename T> std::string rawMemoryToString(const T &object)
 
 template <typename T> class IsStreamInsertable {
     template <typename Stream, typename U>
-    static auto test(int) -> decltype(
-        std::declval<Stream &>() << std::declval<U>(), std::true_type());
+    static auto test(int)
+        -> decltype(std::declval<Stream &>() << std::declval<U>(),
+            std::true_type());
 
     template <typename, typename> static auto test(...) -> std::false_type;
 
@@ -2549,7 +2551,7 @@ struct TupleElementPrinter {
 
 template <typename Tuple, std::size_t N>
 struct TupleElementPrinter<Tuple, N, false> {
-    static void print(const Tuple &, std::ostream &) {}
+    static void print(const Tuple &, std::ostream &) { }
 };
 }
 
@@ -3318,7 +3320,7 @@ public:
     }
 
     template <typename T, typename... Ts>
-    void captureValues(size_t index, T const &value, Ts const &... values)
+    void captureValues(size_t index, T const &value, Ts const &...values)
     {
         captureValue(index, Catch::Detail::stringify(value));
         captureValues(index + 1, values...);
@@ -4698,13 +4700,15 @@ struct IGeneratorTracker {
 
 namespace Catch {
 #if !defined(CATCH_CONFIG_DISABLE_EXCEPTIONS)
-template <typename Ex>
-[[noreturn]] void throw_exception(Ex const &e) { throw e; }
+template <typename Ex> [[noreturn]] void throw_exception(Ex const &e)
+{
+    throw e;
+}
 #else // ^^ Exceptions are enabled //  Exceptions are disabled vv
 [[noreturn]] void throw_exception(std::exception const &e);
 #endif
 
-    [[noreturn]] void throw_logic_error(std::string const &msg);
+[[noreturn]] void throw_logic_error(std::string const &msg);
 [[noreturn]] void throw_domain_error(std::string const &msg);
 [[noreturn]] void throw_runtime_error(std::string const &msg);
 
@@ -4754,7 +4758,7 @@ namespace Generators {
 // !TBD move this into its own location?
 namespace pf {
 template <typename T, typename... Args>
-std::unique_ptr<T> make_unique(Args &&... args)
+std::unique_ptr<T> make_unique(Args &&...args)
 {
     return std::unique_ptr<T>(new T(std::forward<Args>(args)...));
 }
@@ -4847,14 +4851,14 @@ template <typename T> class Generators : public IGenerator<T> {
         populate(T(std::forward<U>(val)));
     }
     template <typename U, typename... Gs>
-    void populate(U &&valueOrGenerator, Gs &&... moreGenerators)
+    void populate(U &&valueOrGenerator, Gs &&...moreGenerators)
     {
         populate(std::forward<U>(valueOrGenerator));
         populate(std::forward<Gs>(moreGenerators)...);
     }
 
 public:
-    template <typename... Gs> Generators(Gs &&... moreGenerators)
+    template <typename... Gs> Generators(Gs &&...moreGenerators)
     {
         m_generators.reserve(sizeof...(Gs));
         populate(std::forward<Gs>(moreGenerators)...);
@@ -4888,7 +4892,7 @@ template <typename T> struct as {
 };
 
 template <typename T, typename... Gs>
-auto makeGenerators(GeneratorWrapper<T> &&generator, Gs &&... moreGenerators)
+auto makeGenerators(GeneratorWrapper<T> &&generator, Gs &&...moreGenerators)
     -> Generators<T>
 {
     return Generators<T>(
@@ -4900,13 +4904,13 @@ auto makeGenerators(GeneratorWrapper<T> &&generator) -> Generators<T>
     return Generators<T>(std::move(generator));
 }
 template <typename T, typename... Gs>
-auto makeGenerators(T &&val, Gs &&... moreGenerators) -> Generators<T>
+auto makeGenerators(T &&val, Gs &&...moreGenerators) -> Generators<T>
 {
     return makeGenerators(
         value(std::forward<T>(val)), std::forward<Gs>(moreGenerators)...);
 }
 template <typename T, typename U, typename... Gs>
-auto makeGenerators(as<T>, U &&val, Gs &&... moreGenerators) -> Generators<T>
+auto makeGenerators(as<T>, U &&val, Gs &&...moreGenerators) -> Generators<T>
 {
     return makeGenerators(
         value(T(std::forward<U>(val))), std::forward<Gs>(moreGenerators)...);
@@ -5749,7 +5753,7 @@ public:
     }
 
 private:
-    virtual ~OcMethod() {}
+    virtual ~OcMethod() { }
 
     Class m_cls;
     SEL m_sel;
@@ -6557,7 +6561,7 @@ struct IStreamingReporter {
 
     virtual void noMatchingTestCases(std::string const &spec) = 0;
 
-    virtual void reportInvalidArguments(std::string const &) {}
+    virtual void reportInvalidArguments(std::string const &) { }
 
     virtual void testRunStarting(TestRunInfo const &testRunInfo) = 0;
     virtual void testGroupStarting(GroupInfo const &groupInfo) = 0;
@@ -6566,10 +6570,10 @@ struct IStreamingReporter {
     virtual void sectionStarting(SectionInfo const &sectionInfo) = 0;
 
 #if defined(CATCH_CONFIG_ENABLE_BENCHMARKING)
-    virtual void benchmarkPreparing(std::string const &) {}
-    virtual void benchmarkStarting(BenchmarkInfo const &) {}
-    virtual void benchmarkEnded(BenchmarkStats<> const &) {}
-    virtual void benchmarkFailed(std::string const &) {}
+    virtual void benchmarkPreparing(std::string const &) { }
+    virtual void benchmarkStarting(BenchmarkInfo const &) { }
+    virtual void benchmarkEnded(BenchmarkStats<> const &) { }
+    virtual void benchmarkFailed(std::string const &) { }
 #endif // CATCH_CONFIG_ENABLE_BENCHMARKING
 
     virtual void assertionStarting(AssertionInfo const &assertionInfo) = 0;
@@ -6652,9 +6656,9 @@ template <typename DerivedT> struct StreamingReporterBase : IStreamingReporter {
 
     ~StreamingReporterBase() override = default;
 
-    void noMatchingTestCases(std::string const &) override {}
+    void noMatchingTestCases(std::string const &) override { }
 
-    void reportInvalidArguments(std::string const &) override {}
+    void reportInvalidArguments(std::string const &) override { }
 
     void testRunStarting(TestRunInfo const &_testRunInfo) override
     {
@@ -6718,7 +6722,7 @@ struct CumulativeReporterBase : IStreamingReporter {
             : value(_value)
         {
         }
-        virtual ~Node() {}
+        virtual ~Node() { }
 
         using ChildNodes = std::vector<std::shared_ptr<ChildNodeT>>;
         T value;
@@ -6794,10 +6798,10 @@ struct CumulativeReporterBase : IStreamingReporter {
         return {Verbosity::Normal};
     }
 
-    void testRunStarting(TestRunInfo const &) override {}
-    void testGroupStarting(GroupInfo const &) override {}
+    void testRunStarting(TestRunInfo const &) override { }
+    void testGroupStarting(GroupInfo const &) override { }
 
-    void testCaseStarting(TestCaseInfo const &) override {}
+    void testCaseStarting(TestCaseInfo const &) override { }
 
     void sectionStarting(SectionInfo const &sectionInfo) override
     {
@@ -6823,7 +6827,7 @@ struct CumulativeReporterBase : IStreamingReporter {
         m_deepestSection = std::move(node);
     }
 
-    void assertionStarting(AssertionInfo const &) override {}
+    void assertionStarting(AssertionInfo const &) override { }
 
     bool assertionEnded(AssertionStats const &assertionStats) override
     {
@@ -6873,7 +6877,7 @@ struct CumulativeReporterBase : IStreamingReporter {
     }
     virtual void testRunEndedCumulative() = 0;
 
-    void skipTest(TestCaseInfo const &) override {}
+    void skipTest(TestCaseInfo const &) override { }
 
     IConfigPtr m_config;
     std::ostream &stream;
@@ -7458,7 +7462,7 @@ inline void optimizer_barrier()
 template <typename T> inline void deoptimize_value(T &&x) { keep_memory(&x); }
 
 template <typename Fn, typename... Args>
-inline auto invoke_deoptimized(Fn &&fn, Args &&... args) ->
+inline auto invoke_deoptimized(Fn &&fn, Args &&...args) ->
     typename std::enable_if<
         !std::is_same<void, decltype(fn(args...))>::value>::type
 {
@@ -7466,7 +7470,7 @@ inline auto invoke_deoptimized(Fn &&fn, Args &&... args) ->
 }
 
 template <typename Fn, typename... Args>
-inline auto invoke_deoptimized(Fn &&fn, Args &&... args) ->
+inline auto invoke_deoptimized(Fn &&fn, Args &&...args) ->
     typename std::enable_if<
         std::is_same<void, decltype(fn(args...))>::value>::type
 {
@@ -7498,14 +7502,14 @@ template <typename T> using CompleteType_t = typename CompleteType<T>::type;
 
 template <typename Result> struct CompleteInvoker {
     template <typename Fun, typename... Args>
-    static Result invoke(Fun &&fun, Args &&... args)
+    static Result invoke(Fun &&fun, Args &&...args)
     {
         return std::forward<Fun>(fun)(std::forward<Args>(args)...);
     }
 };
 template <> struct CompleteInvoker<void> {
     template <typename Fun, typename... Args>
-    static CompleteType_t<void> invoke(Fun &&fun, Args &&... args)
+    static CompleteType_t<void> invoke(Fun &&fun, Args &&...args)
     {
         std::forward<Fun>(fun)(std::forward<Args>(args)...);
         return {};
@@ -7516,7 +7520,7 @@ template <typename Sig> using ResultOf_t = typename std::result_of<Sig>::type;
 // invoke and not return void :(
 template <typename Fun, typename... Args>
 CompleteType_t<ResultOf_t<Fun(Args...)>> complete_invoke(
-    Fun &&fun, Args &&... args)
+    Fun &&fun, Args &&...args)
 {
     return CompleteInvoker<ResultOf_t<Fun(Args...)>>::invoke(
         std::forward<Fun>(fun), std::forward<Args>(args)...);
@@ -7683,7 +7687,7 @@ private:
     };
 
     struct do_nothing {
-        void operator()() const {}
+        void operator()() const { }
     };
 
     template <typename T>
@@ -7802,7 +7806,7 @@ namespace Catch {
 namespace Benchmark {
 namespace Detail {
 template <typename Clock, typename Fun, typename... Args>
-TimingOf<Clock, Fun(Args...)> measure(Fun &&fun, Args &&... args)
+TimingOf<Clock, Fun(Args...)> measure(Fun &&fun, Args &&...args)
 {
     auto start = Clock::now();
     auto &&r = Detail::complete_invoke(fun, std::forward<Args>(args)...);
@@ -8153,8 +8157,7 @@ EnvironmentEstimate<FloatDuration<Clock>> estimate_clock_cost(
                 volatile auto ignored = Clock::now();
                 (void)ignored;
             }
-        })
-            .elapsed;
+        }).elapsed;
     };
     time_clock(1);
     int iters = clock_cost_estimation_iterations;
@@ -8442,7 +8445,7 @@ template <typename T, bool Destruct> struct ObjectStorage {
 
     ~ObjectStorage() { destruct_on_exit<T>(); }
 
-    template <typename... Args> void construct(Args &&... args)
+    template <typename... Args> void construct(Args &&...args)
     {
         new (&data) T(std::forward<Args>(args)...);
     }
@@ -10082,7 +10085,7 @@ public:
         return m_tokenBuffer.front();
     }
 
-    auto operator-> () const -> Token const *
+    auto operator->() const -> Token const *
     {
         assert(!m_tokenBuffer.empty());
         return &m_tokenBuffer.front();
@@ -11380,7 +11383,7 @@ struct IColourImpl {
 };
 
 struct NoColourImpl : IColourImpl {
-    void use(Colour::Code) override {}
+    void use(Colour::Code) override { }
 
     static IColourImpl *instance()
     {
@@ -11868,7 +11871,8 @@ void formatReconstructedExpression(std::ostream &os, std::string const &lhs,
 namespace Catch {
 #if defined(CATCH_CONFIG_DISABLE_EXCEPTIONS) &&                                \
     !defined(CATCH_CONFIG_DISABLE_EXCEPTIONS_CUSTOM_HANDLER)
-[[noreturn]] void throw_exception(std::exception const &e) {
+[[noreturn]] void throw_exception(std::exception const &e)
+{
     Catch::cerr()
         << "Catch will terminate because it needed to throw an exception.\n"
         << "The message was: " << e.what() << '\n';
@@ -11881,8 +11885,10 @@ namespace Catch {
     throw_exception(std::logic_error(msg));
 }
 
-[[noreturn]] void throw_domain_error(
-    std::string const &msg) { throw_exception(std::domain_error(msg)); }
+[[noreturn]] void throw_domain_error(std::string const &msg)
+{
+    throw_exception(std::domain_error(msg));
+}
 
 [[noreturn]] void throw_runtime_error(std::string const &msg)
 {
@@ -11925,7 +11931,7 @@ std::vector<StringRef> parseEnums(StringRef enums);
 
 namespace Catch {
 
-IMutableEnumValuesRegistry::~IMutableEnumValuesRegistry() {}
+IMutableEnumValuesRegistry::~IMutableEnumValuesRegistry() { }
 
 namespace Detail {
 
@@ -11954,7 +11960,7 @@ std::vector<StringRef> parseEnums(StringRef enums)
     return parsed;
 }
 
-EnumInfo::~EnumInfo() {}
+EnumInfo::~EnumInfo() { }
 
 StringRef EnumInfo::lookup(int value) const
 {
@@ -12033,7 +12039,7 @@ private:
 
 namespace Catch {
 
-ExceptionTranslatorRegistry::~ExceptionTranslatorRegistry() {}
+ExceptionTranslatorRegistry::~ExceptionTranslatorRegistry() { }
 
 void ExceptionTranslatorRegistry::registerTranslator(
     const IExceptionTranslator *translator)
@@ -12278,7 +12284,7 @@ char FatalConditionHandler::altStackMem[sigStackSize] = {};
 #else
 
 namespace Catch {
-void FatalConditionHandler::reset() {}
+void FatalConditionHandler::reset() { }
 }
 
 #endif // signals/SEH handling
@@ -12294,13 +12300,13 @@ void FatalConditionHandler::reset() {}
 
 namespace Catch {
 
-IGeneratorTracker::~IGeneratorTracker() {}
+IGeneratorTracker::~IGeneratorTracker() { }
 
 const char *GeneratorException::what() const noexcept { return m_msg; }
 
 namespace Generators {
 
-GeneratorUntypedBase::~GeneratorUntypedBase() {}
+GeneratorUntypedBase::~GeneratorUntypedBase() { }
 
 auto acquireGeneratorTracker(SourceLineInfo const &lineInfo)
     -> IGeneratorTracker &
@@ -12496,7 +12502,7 @@ TestRunStats::TestRunStats(
 
 TestRunStats::~TestRunStats() = default;
 
-void IStreamingReporter::fatalErrorEncountered(StringRef) {}
+void IStreamingReporter::fatalErrorEncountered(StringRef) { }
 bool IStreamingReporter::isMulti() const { return false; }
 
 IReporterFactory::~IReporterFactory() = default;
@@ -12539,7 +12545,7 @@ LeakDetector::LeakDetector()
 
 #else
 
-Catch::LeakDetector::LeakDetector() {}
+Catch::LeakDetector::LeakDetector() { }
 
 #endif
 
@@ -14145,7 +14151,7 @@ struct GeneratorTracker : TestCaseTracking::TrackerBase, IGeneratorTracker {
         m_generator = std::move(generator);
     }
 };
-GeneratorTracker::~GeneratorTracker() {}
+GeneratorTracker::~GeneratorTracker() { }
 }
 
 RunContext::RunContext(
@@ -15056,8 +15062,8 @@ int Session::runInternal()
         // the return value to 255 prevents false negative when some multiple
         // of 256 tests has failed
         return (std::min)(MaxExitCode,
-            (std::max)(
-                totals.error, static_cast<int>(totals.assertions.failed)));
+            (std::max)(totals.error,
+                static_cast<int>(totals.assertions.failed)));
     }
 #if !defined(CATCH_CONFIG_DISABLE_EXCEPTIONS)
     catch (std::exception &ex)
@@ -15086,7 +15092,7 @@ static auto getSingletons() -> std::vector<ISingleton *> *&
 }
 }
 
-ISingleton::~ISingleton() {}
+ISingleton::~ISingleton() { }
 
 void addSingleton(ISingleton *singleton)
 {
@@ -15519,7 +15525,7 @@ RegistrarForTagAliases::RegistrarForTagAliases(
 
 namespace Catch {
 
-TagAliasRegistry::~TagAliasRegistry() {}
+TagAliasRegistry::~TagAliasRegistry() { }
 
 TagAlias const *TagAliasRegistry::find(std::string const &alias) const
 {
@@ -15562,7 +15568,7 @@ void TagAliasRegistry::add(std::string const &alias, std::string const &tag,
                               << "\tRedefined at: " << lineInfo);
 }
 
-ITagAliasRegistry::~ITagAliasRegistry() {}
+ITagAliasRegistry::~ITagAliasRegistry() { }
 
 ITagAliasRegistry const &ITagAliasRegistry::get()
 {
@@ -17428,7 +17434,7 @@ std::set<Verbosity> TestEventListenerBase::getSupportedVerbosities()
     return {Verbosity::Quiet, Verbosity::Normal, Verbosity::High};
 }
 
-void TestEventListenerBase::assertionStarting(AssertionInfo const &) {}
+void TestEventListenerBase::assertionStarting(AssertionInfo const &) { }
 
 bool TestEventListenerBase::assertionEnded(AssertionStats const &)
 {
@@ -17702,7 +17708,7 @@ void CompactReporter::noMatchingTestCases(std::string const &spec)
     stream << "No test cases matched '" << spec << '\'' << std::endl;
 }
 
-void CompactReporter::assertionStarting(AssertionInfo const &) {}
+void CompactReporter::assertionStarting(AssertionInfo const &) { }
 
 bool CompactReporter::assertionEnded(AssertionStats const &_assertionStats)
 {
@@ -17739,7 +17745,7 @@ void CompactReporter::testRunEnded(TestRunStats const &_testRunStats)
     StreamingReporterBase::testRunEnded(_testRunStats);
 }
 
-CompactReporter::~CompactReporter() {}
+CompactReporter::~CompactReporter() { }
 
 CATCH_REGISTER_REPORTER("compact", CompactReporter)
 
@@ -18156,7 +18162,7 @@ void ConsoleReporter::reportInvalidArguments(std::string const &arg)
     stream << "Invalid Filter: " << arg << std::endl;
 }
 
-void ConsoleReporter::assertionStarting(AssertionInfo const &) {}
+void ConsoleReporter::assertionStarting(AssertionInfo const &) { }
 
 bool ConsoleReporter::assertionEnded(AssertionStats const &_assertionStats)
 {
@@ -18571,7 +18577,7 @@ JunitReporter::JunitReporter(ReporterConfig const &_config)
     m_reporterPrefs.shouldReportAllAssertions = true;
 }
 
-JunitReporter::~JunitReporter() {}
+JunitReporter::~JunitReporter() { }
 
 std::string JunitReporter::getDescription()
 {
@@ -18579,7 +18585,7 @@ std::string JunitReporter::getDescription()
            "junitreport target";
 }
 
-void JunitReporter::noMatchingTestCases(std::string const & /*spec*/) {}
+void JunitReporter::noMatchingTestCases(std::string const & /*spec*/) { }
 
 void JunitReporter::testRunStarting(TestRunInfo const &runInfo)
 {
@@ -19063,7 +19069,7 @@ void XmlReporter::sectionStarting(SectionInfo const &sectionInfo)
     }
 }
 
-void XmlReporter::assertionStarting(AssertionInfo const &) {}
+void XmlReporter::assertionStarting(AssertionInfo const &) { }
 
 bool XmlReporter::assertionEnded(AssertionStats const &assertionStats)
 {
diff --git a/tests/t00002/t00002.cc b/tests/t00002/t00002.cc
index 6a8b92dd..6eb8a5de 100644
--- a/tests/t00002/t00002.cc
+++ b/tests/t00002/t00002.cc
@@ -11,12 +11,12 @@ public:
 
 class B : public A {
 public:
-    virtual void foo_a() override {}
+    virtual void foo_a() override { }
 };
 
 class C : public A {
 public:
-    virtual void foo_c() override {}
+    virtual void foo_c() override { }
 };
 
 class D : public B, public C {
diff --git a/tests/t00003/t00003.cc b/tests/t00003/t00003.cc
index b63fd7bb..1820dc1c 100644
--- a/tests/t00003/t00003.cc
+++ b/tests/t00003/t00003.cc
@@ -15,9 +15,9 @@ public:
     A(const A &) = default;
     virtual ~A() = default;
 
-    void basic_method() {}
+    void basic_method() { }
     static int static_method() { return 0; }
-    void const_method() const {}
+    void const_method() const { }
     auto auto_method() { return 1; }
 
     auto double_int(const int i) { return 2 * i; }
@@ -37,7 +37,7 @@ public:
     static const auto auto_member{10UL};
 
 protected:
-    void protected_method() {}
+    void protected_method() { }
 
     int protected_member;
 
@@ -46,7 +46,7 @@ protected:
     };
 
 private:
-    void private_method() {}
+    void private_method() { }
 
     int private_member;
     int a, b, c;
diff --git a/tests/t00004/t00004.cc b/tests/t00004/t00004.cc
index 22be0004..3216ba7a 100644
--- a/tests/t00004/t00004.cc
+++ b/tests/t00004/t00004.cc
@@ -3,7 +3,7 @@ namespace t00004 {
 
 class A {
 public:
-    void foo() const {}
+    void foo() const { }
 
     class AA {
     public:
@@ -13,7 +13,7 @@ public:
         };
     };
 
-    void foo2() const {}
+    void foo2() const { }
 };
 }
 }
diff --git a/tests/t00011/t00011.cc b/tests/t00011/t00011.cc
index 20f9de5a..ea9bcb2d 100644
--- a/tests/t00011/t00011.cc
+++ b/tests/t00011/t00011.cc
@@ -14,7 +14,7 @@ template <typename T> class D {
 
 class A {
 private:
-    void foo() {}
+    void foo() { }
     friend class B;
     friend class external::C;
     // TODO
diff --git a/tests/t00013/t00013.cc b/tests/t00013/t00013.cc
index dc8ca465..d96d0e84 100644
--- a/tests/t00013/t00013.cc
+++ b/tests/t00013/t00013.cc
@@ -27,7 +27,7 @@ class R;
 
 struct D {
     int d;
-    void print(R *r) {}
+    void print(R *r) { }
 };
 
 template <typename T> struct E {
diff --git a/tests/t00015/t00015.cc b/tests/t00015/t00015.cc
index a7762cac..736eb9ff 100644
--- a/tests/t00015/t00015.cc
+++ b/tests/t00015/t00015.cc
@@ -8,7 +8,8 @@ class A {
 }
 
 namespace ns2_v0_9_0 {
-class [[deprecated]] A {};
+class [[deprecated]] A {
+};
 }
 
 namespace {
diff --git a/tests/t00021/t00021.cc b/tests/t00021/t00021.cc
index c3aaa1c4..0a338a12 100644
--- a/tests/t00021/t00021.cc
+++ b/tests/t00021/t00021.cc
@@ -15,20 +15,20 @@ public:
 
 class Visitor1 : public Visitor {
 public:
-    void visit_A(const A &item) const override {}
-    void visit_B(const B &item) const override {}
+    void visit_A(const A &item) const override { }
+    void visit_B(const B &item) const override { }
 };
 
 class Visitor2 : public Visitor {
 public:
-    void visit_A(const A &item) const override {}
-    void visit_B(const B &item) const override {}
+    void visit_A(const A &item) const override { }
+    void visit_B(const B &item) const override { }
 };
 
 class Visitor3 : public Visitor {
 public:
-    void visit_A(const A &item) const override {}
-    void visit_B(const B &item) const override {}
+    void visit_A(const A &item) const override { }
+    void visit_B(const B &item) const override { }
 };
 
 class Item {
@@ -39,12 +39,12 @@ public:
 
 class A : public Item {
 public:
-    void accept(const Visitor &visitor) const override {}
+    void accept(const Visitor &visitor) const override { }
 };
 
 class B : public Item {
 public:
-    void accept(const Visitor &visitor) const override {}
+    void accept(const Visitor &visitor) const override { }
 };
 }
 }
diff --git a/tests/t00022/t00022.cc b/tests/t00022/t00022.cc
index 2e26ace2..d226c4db 100644
--- a/tests/t00022/t00022.cc
+++ b/tests/t00022/t00022.cc
@@ -18,14 +18,14 @@ protected:
 
 class A1 : public A {
 protected:
-    void method1() override {}
-    void method2() override {}
+    void method1() override { }
+    void method2() override { }
 };
 
 class A2 : public A {
 protected:
-    void method1() override {}
-    void method2() override {}
+    void method1() override { }
+    void method2() override { }
 };
 }
 }
diff --git a/tests/t00023/t00023.cc b/tests/t00023/t00023.cc
index ac990ba5..3d3cdd94 100644
--- a/tests/t00023/t00023.cc
+++ b/tests/t00023/t00023.cc
@@ -11,17 +11,17 @@ public:
 
 class StrategyA : public Strategy {
 public:
-    void algorithm() override {}
+    void algorithm() override { }
 };
 
 class StrategyB : public Strategy {
 public:
-    void algorithm() override {}
+    void algorithm() override { }
 };
 
 class StrategyC : public Strategy {
 public:
-    void algorithm() override {}
+    void algorithm() override { }
 };
 
 class Context {
diff --git a/tests/t00024/t00024.cc b/tests/t00024/t00024.cc
index b22bb5c9..d082aee8 100644
--- a/tests/t00024/t00024.cc
+++ b/tests/t00024/t00024.cc
@@ -13,14 +13,14 @@ public:
 
 class Target1 : public Target {
 public:
-    void m1() override {}
-    void m2() override {}
+    void m1() override { }
+    void m2() override { }
 };
 
 class Target2 : public Target {
 public:
-    void m1() override {}
-    void m2() override {}
+    void m1() override { }
+    void m2() override { }
 };
 
 class Proxy : public Target {
diff --git a/tests/t00025/t00025.cc b/tests/t00025/t00025.cc
index c31ee5a8..091b8185 100644
--- a/tests/t00025/t00025.cc
+++ b/tests/t00025/t00025.cc
@@ -5,14 +5,14 @@ namespace t00025 {
 
 class Target1 {
 public:
-    void m1() {}
-    void m2() {}
+    void m1() { }
+    void m2() { }
 };
 
 class Target2 {
 public:
-    void m1() {}
-    void m2() {}
+    void m1() { }
+    void m2() { }
 };
 
 template <typename T> class Proxy {
diff --git a/tests/t00027/t00027.cc b/tests/t00027/t00027.cc
index 379a100d..7f0ea89c 100644
--- a/tests/t00027/t00027.cc
+++ b/tests/t00027/t00027.cc
@@ -16,7 +16,7 @@ class Line : public Shape, public T<Line<>>... {
 public:
     void display() override
     {
-        std::apply([](auto &&... x) { (x.display(), ...); },
+        std::apply([](auto &&...x) { (x.display(), ...); },
             std::forward_as_tuple(T<Line<>>()...));
     }
 };
@@ -26,7 +26,7 @@ class Text : public Shape, public T<Text<>>... {
 public:
     void display() override
     {
-        std::apply([](auto &&... x) { (x.display(), ...); },
+        std::apply([](auto &&...x) { (x.display(), ...); },
             std::forward_as_tuple(T<Text<>>()...));
     }
 };
@@ -37,12 +37,12 @@ struct ShapeDecorator {
 
 template <typename T> class Color : public ShapeDecorator {
 public:
-    void display() override {}
+    void display() override { }
 };
 
 template <typename T> class Weight : public ShapeDecorator {
 public:
-    void display() override {}
+    void display() override { }
 };
 
 struct Window {
diff --git a/tests/t20001/t20001.cc b/tests/t20001/t20001.cc
index 2523199b..18878c11 100644
--- a/tests/t20001/t20001.cc
+++ b/tests/t20001/t20001.cc
@@ -13,7 +13,7 @@ struct C {
 
 class A {
 public:
-    A() {}
+    A() { }
 
     int add(int x, int y) { return m_c.add(x, y); }
 
@@ -28,7 +28,7 @@ public:
         return res;
     }
 
-    void log_result(int r) {}
+    void log_result(int r) { }
 
 private:
     detail::C m_c{};