Bump required clang-format version to 16

Bump the required clang-format version from 14 to 16. For reference,
this is the version the Babeltrace project uses [1], and it is checked
by the LTTng CI [2], so I think it makes sense to go to that version.
More recent versions might not be readily available on the CI machines
(mostly running Debian 12 at the moment).

FWIW, I think that the code changes generated by this version change are
improvements.

[1] https://github.com/efficios/babeltrace/blob/d16ccfd174984c3d18f4f4427e4e438b1a64730e/tools/format-cpp.sh#L7
[2] https://ci.lttng.org/view/Babeltrace/job/babeltrace_master_lint/

Change-Id: I188f04928357ac7562b1a2dce3238a622189b04a
Signed-off-by: Simon Marchi <simon.marchi@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

diff --git a/format-cpp b/format-cpp
index a3943ae578d07552e0ce9219910a63511cbde7dd..7bc26d1f35647e72e62fe4c4b504a4779af9f9cd 100755 (executable)
--- a/format-cpp
+++ b/format-cpp
@@ -4,7 +4,7 @@
 #
 # Copyright (C) 2020-2022 Philippe Proulx <pproulx@efficios.com>
 
-expected_formatter_major_version=14
+expected_formatter_major_version=16
 
 SCRIPT_DIR=$(dirname "$0")/
 

diff --git a/src/common/scope-exit.hpp b/src/common/scope-exit.hpp
index 3f8c90f80941fbe4384c498c559a6c4d6454be0c..f8225754a6ff7c1ff62146fe9fbc18ad228bd9f5 100644 (file)
--- a/src/common/scope-exit.hpp
+++ b/src/common/scope-exit.hpp
@@ -16,8 +16,7 @@ namespace details {
 /* Is operator() of InvocableType is marked as noexcept? */
 template <typename InvocableType>
 struct is_invocation_noexcept
-       : std::integral_constant<bool, noexcept((std::declval<InvocableType>())())> {
-};
+       : std::integral_constant<bool, noexcept((std::declval<InvocableType>())())> {};
 } /* namespace details. */
 
 /*

diff --git a/src/common/type-traits.hpp b/src/common/type-traits.hpp
index 3801bf13fe00f825b633bd8970b76927452407f1..1783a5302719fe83465c5a9f7369093750d5cbef 100644 (file)
--- a/src/common/type-traits.hpp
+++ b/src/common/type-traits.hpp
@@ -22,16 +22,13 @@ namespace traits {
  *     `false`
  */
 template <typename T, typename... Ts>
-struct is_one_of : std::false_type {
-};
+struct is_one_of : std::false_type {};
 
 template <typename T, typename... Ts>
-struct is_one_of<T, T, Ts...> : std::true_type {
-};
+struct is_one_of<T, T, Ts...> : std::true_type {};
 
 template <typename T, typename U, typename... Ts>
-struct is_one_of<T, U, Ts...> : is_one_of<T, Ts...> {
-};
+struct is_one_of<T, U, Ts...> : is_one_of<T, Ts...> {};
 } /* namespace traits */
 } /* namespace lttng */