diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 84042b8d..7406111d 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -3,6 +3,6 @@ # repos: - repo: https://github.com/pre-commit/mirrors-clang-format - rev: 'v18.1.8' + rev: 'v19.1.3' hooks: - id: clang-format diff --git a/CHANGES b/CHANGES index b8128c8e..3ec2e8fb 100644 --- a/CHANGES +++ b/CHANGES @@ -1,3 +1,7 @@ +2.8.0-dev.137 | 2024-11-07 19:06:35 +0100 + + * Update `clang-format` on CI (Dominik Charousset) + 2.8.0-dev.135 | 2024-11-07 16:48:29 +0100 * Bump dockerfiles for Fedora 39 and 40 (Dominik Charousset) diff --git a/VERSION b/VERSION index 06ec37ca..89a1bf28 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -2.8.0-dev.135 +2.8.0-dev.137 diff --git a/bindings/python/_broker.cpp b/bindings/python/_broker.cpp index aa8400e4..89226f71 100644 --- a/bindings/python/_broker.cpp +++ b/bindings/python/_broker.cpp @@ -272,8 +272,8 @@ PYBIND11_MODULE(_broker, m) { "StatusSubscriber"); status_subscriber .def("get", - (broker::status_subscriber::value_type(broker::status_subscriber::*)()) - & broker::status_subscriber::get) + (broker::status_subscriber::value_type( + broker::status_subscriber::*)()) &broker::status_subscriber::get) .def("get", [](broker::status_subscriber& ep, double secs) -> std::optional { diff --git a/libbroker/broker/alm/multipath.hh b/libbroker/broker/alm/multipath.hh index c781dcb7..9a62e125 100644 --- a/libbroker/broker/alm/multipath.hh +++ b/libbroker/broker/alm/multipath.hh @@ -68,14 +68,14 @@ private: }; template -auto operator==(node_iterator x, - node_iterator y) -> decltype(x.get() == y.get()) { +auto operator==(node_iterator x, node_iterator y) + -> decltype(x.get() == y.get()) { return x.get() == y.get(); } template -auto operator!=(node_iterator x, - node_iterator y) -> decltype(x.get() != y.get()) { +auto operator!=(node_iterator x, node_iterator y) + -> decltype(x.get() != y.get()) { return x.get() != y.get(); } diff --git a/libbroker/broker/detail/type_traits.hh b/libbroker/broker/detail/type_traits.hh index 0426bc61..b1d2057b 100644 --- a/libbroker/broker/detail/type_traits.hh +++ b/libbroker/broker/detail/type_traits.hh @@ -139,8 +139,8 @@ inline constexpr bool are_same_v = are_same::value; template struct has_convert { template - static auto test(const T* x) -> decltype(convert(*x, std::declval()), - std::true_type()); + static auto test(const T* x) + -> decltype(convert(*x, std::declval()), std::true_type()); template static auto test(...) -> std::false_type; @@ -264,9 +264,9 @@ inline constexpr bool is_tuple = is_tuple_oracle::value; class has_encode_overload { \ private: \ template \ - static auto \ - sfinae(U& y) -> decltype(::ns_name::encode(y, std::declval()), \ - std::true_type{}); \ + static auto sfinae(U& y) \ + -> decltype(::ns_name::encode(y, std::declval()), \ + std::true_type{}); \ static std::false_type sfinae(...); \ using result_type = decltype(sfinae(std::declval())); \ \ @@ -282,9 +282,9 @@ inline constexpr bool is_tuple = is_tuple_oracle::value; class has_decode_overload { \ private: \ template \ - static auto \ - sfinae(U& y) -> decltype(::ns_name::decode(y, std::declval()), \ - std::true_type{}); \ + static auto sfinae(U& y) \ + -> decltype(::ns_name::decode(y, std::declval()), \ + std::true_type{}); \ static std::false_type sfinae(...); \ using result_type = decltype(sfinae(std::declval())); \ \ diff --git a/libbroker/broker/expected.hh b/libbroker/broker/expected.hh index 731f601d..b8295147 100644 --- a/libbroker/broker/expected.hh +++ b/libbroker/broker/expected.hh @@ -246,8 +246,8 @@ private: /// @relates expected template -auto operator==(const expected& x, - const expected& y) -> decltype(*x == *y) { +auto operator==(const expected& x, const expected& y) + -> decltype(*x == *y) { return x && y ? *x == *y : (!x && !y ? x.error() == y.error() : false); } @@ -290,8 +290,8 @@ bool operator==(ec x, const expected& y) { /// @relates expected template -auto operator!=(const expected& x, - const expected& y) -> decltype(*x == *y) { +auto operator!=(const expected& x, const expected& y) + -> decltype(*x == *y) { return !(x == y); } diff --git a/libbroker/broker/intrusive_ptr.hh b/libbroker/broker/intrusive_ptr.hh index bad08387..f264c33e 100644 --- a/libbroker/broker/intrusive_ptr.hh +++ b/libbroker/broker/intrusive_ptr.hh @@ -224,15 +224,15 @@ bool operator!=(const T* x, const intrusive_ptr& y) { /// @relates intrusive_ptr template -auto operator==(const intrusive_ptr& x, - const intrusive_ptr& y) -> decltype(x.get() == y.get()) { +auto operator==(const intrusive_ptr& x, const intrusive_ptr& y) + -> decltype(x.get() == y.get()) { return x.get() == y.get(); } /// @relates intrusive_ptr template -auto operator!=(const intrusive_ptr& x, - const intrusive_ptr& y) -> decltype(x.get() != y.get()) { +auto operator!=(const intrusive_ptr& x, const intrusive_ptr& y) + -> decltype(x.get() != y.get()) { return x.get() != y.get(); } diff --git a/tests/benchmarks/routing-table/routing-table.cc b/tests/benchmarks/routing-table/routing-table.cc index eae6c7e3..89e9d761 100644 --- a/tests/benchmarks/routing-table/routing-table.cc +++ b/tests/benchmarks/routing-table/routing-table.cc @@ -208,9 +208,9 @@ class routing_table : public benchmark::Fixture { }; for (size_t index = 0; index < 10; ++index) fill_tbl(topologies[index], g, {}, 0, index, on_new_node); - // for (size_t index = 0; index < 10; ++index) - // fill_tbl(v2_topologies[index], g, {}, 0, index, [](auto&&...) {}); - // Sanity checking + // for (size_t index = 0; index < 10; ++index) + // fill_tbl(v2_topologies[index], g, {}, 0, index, [](auto&&...) {}); + // Sanity checking #ifndef NDEBUG for (size_t index = 0; index < 10; ++index) assert(ids[index].size() == (1 << (index + 1)));