Merge pull request #201 from elbeno/fix-clang-format

👽 Fix clang-format changes from upstream

Author: elbeno

include/stdx/atomic_bitset.hpp

@@ -1,7 +1,5 @@
 #pragma once
 
-#include <conc/atomic.hpp>
-
 #include <stdx/bit.hpp>
 #include <stdx/bitset.hpp>
 #include <stdx/compiler.hpp>
@@ -11,6 +9,8 @@
 #include <stdx/type_traits.hpp>
 #include <stdx/udls.hpp>
 
+#include <conc/atomic.hpp>
+
 #include <algorithm>
 #include <atomic>
 #include <cstddef>

include/stdx/rollover.hpp

@@ -75,10 +75,10 @@ template <typename T> struct rollover_t {
         return not(lhs == rhs);
     }
 
-    constexpr friend auto operator<(rollover_t, rollover_t) -> bool = delete;
-    constexpr friend auto operator<=(rollover_t, rollover_t) -> bool = delete;
-    constexpr friend auto operator>(rollover_t, rollover_t) -> bool = delete;
-    constexpr friend auto operator>=(rollover_t, rollover_t) -> bool = delete;
+    friend constexpr auto operator<(rollover_t, rollover_t) -> bool = delete;
+    friend constexpr auto operator<=(rollover_t, rollover_t) -> bool = delete;
+    friend constexpr auto operator>(rollover_t, rollover_t) -> bool = delete;
+    friend constexpr auto operator>=(rollover_t, rollover_t) -> bool = delete;
 
     [[nodiscard]] constexpr friend auto cmp_less(rollover_t lhs, rollover_t rhs)
         -> bool {

include/stdx/tuple.hpp

@@ -499,7 +499,7 @@ template <typename... Ts>
 class one_of : public detail::tuple_impl<std::index_sequence_for<Ts...>,
                                          detail::index_function_list<>, Ts...> {
     template <typename T>
-    constexpr friend auto operator==(one_of const &lhs, T const &rhs) -> bool {
+    friend constexpr auto operator==(one_of const &lhs, T const &rhs) -> bool {
         return lhs.apply(
             [&](auto &&...args) { return ((args == rhs) || ...); });
     }

test/fail/tuple_equality_with_element.cpp

@@ -3,7 +3,7 @@
 // EXPECT: deleted
 
 struct S {
-    constexpr friend auto operator==(S, S) -> bool = default;
+    friend constexpr auto operator==(S, S) -> bool = default;
 };
 
 auto main() -> int {

test/optional.cpp

@@ -14,10 +14,10 @@ enum struct E : std::uint8_t { VALUE, ALT_VALUE };
 
 struct S {
     constexpr explicit S(int i) : value{i} {}
-    constexpr friend auto operator==(S const &x, S const &y) -> bool {
+    friend constexpr auto operator==(S const &x, S const &y) -> bool {
         return x.value == y.value;
     }
-    constexpr friend auto operator<(S const &x, S const &y) -> bool {
+    friend constexpr auto operator<(S const &x, S const &y) -> bool {
         return x.value < y.value;
     }
     int value{};
@@ -332,7 +332,7 @@ struct move_only {
     constexpr move_only(int i) : value{i} {}
     constexpr move_only(move_only &&) = default;
     int value{};
-    constexpr friend auto operator==(move_only const &x, move_only const &y)
+    friend constexpr auto operator==(move_only const &x, move_only const &y)
         -> bool {
         return x.value == y.value;
     }
@@ -342,7 +342,7 @@ struct non_movable {
     constexpr non_movable(int i) : value{i} {}
     constexpr non_movable(non_movable &&) = delete;
     int value{};
-    constexpr friend auto operator==(non_movable const &x, non_movable const &y)
+    friend constexpr auto operator==(non_movable const &x, non_movable const &y)
         -> bool {
         return x.value == y.value;
     }

test/tuple_algorithms.cpp

@@ -337,7 +337,7 @@ TEST_CASE("fold_left (heterogeneous types in fold)", "[tuple_algorithms]") {
 }
 
 template <auto N> struct addend {
-    constexpr friend auto operator==(addend, addend) -> bool { return true; }
+    friend constexpr auto operator==(addend, addend) -> bool { return true; }
 };
 template <auto X, auto Y> constexpr auto operator+(addend<X>, addend<Y>) {
     return addend<X + Y>{};