diff --git a/AK/Atomic.h b/AK/Atomic.h index 7d7901bf95f..365d6d31d9e 100644 --- a/AK/Atomic.h +++ b/AK/Atomic.h @@ -85,7 +85,6 @@ template::Type> return __atomic_compare_exchange_n(const_cast(var), &expected, nullptr, false, order, order); } - template static inline T atomic_fetch_add(volatile T* var, T val, MemoryOrder order = memory_order_seq_cst) noexcept { diff --git a/AK/Badge.h b/AK/Badge.h index 850b597370a..a8e146f6704 100644 --- a/AK/Badge.h +++ b/AK/Badge.h @@ -31,7 +31,7 @@ namespace AK { template class Badge { friend T; - Badge() {} + Badge() { } Badge(const Badge&) = delete; Badge& operator=(const Badge&) = delete; @@ -43,4 +43,3 @@ class Badge { } using AK::Badge; - diff --git a/AK/Function.h b/AK/Function.h index bea6708ca5d..61759acfb9f 100644 --- a/AK/Function.h +++ b/AK/Function.h @@ -38,7 +38,7 @@ template class Function { public: Function() = default; - Function(std::nullptr_t) {} + Function(std::nullptr_t) { } template::value && IsFunction::Type>::value) && IsRvalueReference::value>::Type> Function(CallableType&& callable) @@ -83,7 +83,7 @@ public: private: class CallableWrapperBase { public: - virtual ~CallableWrapperBase() {} + virtual ~CallableWrapperBase() { } virtual Out call(In...) const = 0; }; diff --git a/AK/GenericLexer.cpp b/AK/GenericLexer.cpp index d288809ab2e..c5818f714b3 100644 --- a/AK/GenericLexer.cpp +++ b/AK/GenericLexer.cpp @@ -276,7 +276,6 @@ void GenericLexer::ignore_until(Condition condition) m_index++; } - bool is_control(char c) { return (c >= 0 && c <= 31) || c == 127; diff --git a/AK/HashFunctions.h b/AK/HashFunctions.h index ffa21a0190d..757c43653e6 100644 --- a/AK/HashFunctions.h +++ b/AK/HashFunctions.h @@ -53,7 +53,7 @@ inline unsigned u64_hash(u64 key) inline unsigned ptr_hash(FlatPtr ptr) { - if constexpr(sizeof(ptr) == 8) + if constexpr (sizeof(ptr) == 8) return u64_hash((u64)ptr); else return int_hash((u32)ptr); diff --git a/AK/HashMap.h b/AK/HashMap.h index dedd1856554..33e982cb332 100644 --- a/AK/HashMap.h +++ b/AK/HashMap.h @@ -47,7 +47,7 @@ private: }; public: - HashMap() {} + HashMap() { } bool is_empty() const { return m_table.is_empty(); } size_t size() const { return m_table.size(); } diff --git a/AK/IDAllocator.h b/AK/IDAllocator.h index 04e967b4653..e55a31cb0f9 100644 --- a/AK/IDAllocator.h +++ b/AK/IDAllocator.h @@ -26,16 +26,16 @@ #pragma once -#include #include +#include namespace AK { class IDAllocator { public: - IDAllocator() {} - ~IDAllocator() {} + IDAllocator() { } + ~IDAllocator() { } int allocate() { diff --git a/AK/InlineLinkedList.h b/AK/InlineLinkedList.h index 94c685737cb..d40634e8382 100644 --- a/AK/InlineLinkedList.h +++ b/AK/InlineLinkedList.h @@ -104,7 +104,7 @@ inline T* InlineLinkedListNode::next() const template class InlineLinkedList { public: - InlineLinkedList() {} + InlineLinkedList() { } bool is_empty() const { return !m_head; } size_t size_slow() const; diff --git a/AK/JsonArray.h b/AK/JsonArray.h index af8654f6311..63ac74a4e0c 100644 --- a/AK/JsonArray.h +++ b/AK/JsonArray.h @@ -34,8 +34,8 @@ namespace AK { class JsonArray { public: - JsonArray() {} - ~JsonArray() {} + JsonArray() { } + ~JsonArray() { } JsonArray(const JsonArray& other) : m_values(other.m_values) diff --git a/AK/JsonObject.h b/AK/JsonObject.h index 0e970cf560b..2fe396c026d 100644 --- a/AK/JsonObject.h +++ b/AK/JsonObject.h @@ -37,8 +37,8 @@ namespace AK { class JsonObject { public: - JsonObject() {} - ~JsonObject() {} + JsonObject() { } + ~JsonObject() { } JsonObject(const JsonObject& other) : m_order(other.m_order) diff --git a/AK/LexicalPath.cpp b/AK/LexicalPath.cpp index 6a7d2ff46e0..a699b5aa8c8 100644 --- a/AK/LexicalPath.cpp +++ b/AK/LexicalPath.cpp @@ -92,10 +92,10 @@ void LexicalPath::canonicalize() Optional last_dot = StringView(m_basename).find_last_of('.'); if (last_dot.has_value()) { - m_title = m_basename.substring(0, last_dot.value()); - m_extension = m_basename.substring(last_dot.value() + 1, m_basename.length() - last_dot.value() - 1); + m_title = m_basename.substring(0, last_dot.value()); + m_extension = m_basename.substring(last_dot.value() + 1, m_basename.length() - last_dot.value() - 1); } else { - m_title = m_basename; + m_title = m_basename; } StringBuilder builder(approximate_canonical_length); diff --git a/AK/MACAddress.h b/AK/MACAddress.h index 4de31bfcc86..e511da23f19 100644 --- a/AK/MACAddress.h +++ b/AK/MACAddress.h @@ -33,7 +33,7 @@ class [[gnu::packed]] MACAddress { public: - MACAddress() {} + MACAddress() { } MACAddress(const u8 data[6]) { __builtin_memcpy(m_data, data, 6); @@ -47,7 +47,7 @@ public: m_data[4] = e; m_data[5] = f; } - ~MACAddress() {} + ~MACAddress() { } u8 operator[](int i) const { @@ -78,7 +78,7 @@ static_assert(sizeof(MACAddress) == 6); namespace AK { -template <> +template<> struct Traits : public GenericTraits { static unsigned hash(const MACAddress& address) { return string_hash((const char*)&address, sizeof(address)); } }; diff --git a/AK/NonnullOwnPtrVector.h b/AK/NonnullOwnPtrVector.h index b5e440a4d57..ae4aca29542 100644 --- a/AK/NonnullOwnPtrVector.h +++ b/AK/NonnullOwnPtrVector.h @@ -26,14 +26,13 @@ #pragma once -#include #include +#include namespace AK { template -class NonnullOwnPtrVector : public NonnullPtrVector, inline_capacity> -{ +class NonnullOwnPtrVector : public NonnullPtrVector, inline_capacity> { }; } diff --git a/AK/NonnullRefPtrVector.h b/AK/NonnullRefPtrVector.h index ca6898e1c0b..9becc301658 100644 --- a/AK/NonnullRefPtrVector.h +++ b/AK/NonnullRefPtrVector.h @@ -32,8 +32,7 @@ namespace AK { template -class NonnullRefPtrVector : public NonnullPtrVector, inline_capacity> -{ +class NonnullRefPtrVector : public NonnullPtrVector, inline_capacity> { }; } diff --git a/AK/Queue.h b/AK/Queue.h index f686fab6e07..320c31e9c77 100644 --- a/AK/Queue.h +++ b/AK/Queue.h @@ -88,4 +88,3 @@ private: } using AK::Queue; - diff --git a/AK/RefPtr.h b/AK/RefPtr.h index 394d786919b..98ba2ff3521 100644 --- a/AK/RefPtr.h +++ b/AK/RefPtr.h @@ -44,7 +44,7 @@ public: Adopt }; - RefPtr() {} + RefPtr() { } RefPtr(const T* ptr) : m_ptr(const_cast(ptr)) { @@ -108,7 +108,7 @@ public: m_ptr = (T*)(0xe0e0e0e0); #endif } - RefPtr(std::nullptr_t) {} + RefPtr(std::nullptr_t) { } template RefPtr(const OwnPtr&) = delete; diff --git a/AK/Result.h b/AK/Result.h index fd0c3982b15..6c72163b197 100644 --- a/AK/Result.h +++ b/AK/Result.h @@ -32,7 +32,8 @@ namespace AK { template -class [[nodiscard]] Result { +class [[nodiscard]] Result +{ public: Result(const ValueType& res) : m_result(res) @@ -50,13 +51,19 @@ public: { } + // FIXME: clang-format gets confused about Result. Why? + // clang-format off Result(Result&& other) + // clang-format on : m_result(move(other.m_result)) , m_error(move(other.m_error)) { } + // FIXME: clang-format gets confused about Result. Why? + // clang-format off Result(Result& other) + // clang-format on : m_result(other.m_result) , m_error(other.m_error) { diff --git a/AK/ScopeGuard.h b/AK/ScopeGuard.h index b6b18922201..02d28219571 100644 --- a/AK/ScopeGuard.h +++ b/AK/ScopeGuard.h @@ -70,5 +70,5 @@ private: } -using AK::ScopeGuard; using AK::ArmedScopeGuard; +using AK::ScopeGuard; diff --git a/AK/Singleton.h b/AK/Singleton.h index 0abfa2b2771..ae686a64d2b 100644 --- a/AK/Singleton.h +++ b/AK/Singleton.h @@ -30,7 +30,7 @@ #include #include #ifdef KERNEL -#include +# include #endif #ifndef __serenity__ @@ -51,6 +51,7 @@ template::create> class Singleton { AK_MAKE_NONCOPYABLE(Singleton); AK_MAKE_NONMOVABLE(Singleton); + public: Singleton() = default; diff --git a/AK/SinglyLinkedListWithCount.h b/AK/SinglyLinkedListWithCount.h index 16087b5c6f6..f46daa7a82d 100644 --- a/AK/SinglyLinkedListWithCount.h +++ b/AK/SinglyLinkedListWithCount.h @@ -34,8 +34,8 @@ template class SinglyLinkedListWithCount : private SinglyLinkedList { public: - SinglyLinkedListWithCount() {} - ~SinglyLinkedListWithCount() {} + SinglyLinkedListWithCount() { } + ~SinglyLinkedListWithCount() { } using List = SinglyLinkedList; diff --git a/AK/Tests/TestAtomic.cpp b/AK/Tests/TestAtomic.cpp index 62fcab7b04a..8a49baf875e 100644 --- a/AK/Tests/TestAtomic.cpp +++ b/AK/Tests/TestAtomic.cpp @@ -30,335 +30,335 @@ TEST_CASE(construct_empty) { - EXPECT(Atomic().load() == false); - EXPECT(Atomic().load() == 0); - EXPECT(Atomic().load() == 0); - EXPECT(Atomic().load() == 0); - - EXPECT(Atomic().load() == nullptr); + EXPECT(Atomic().load() == false); + EXPECT(Atomic().load() == 0); + EXPECT(Atomic().load() == 0); + EXPECT(Atomic().load() == 0); + + EXPECT(Atomic().load() == nullptr); } TEST_CASE(construct_with_value) { - EXPECT(Atomic(false).load() == false); - EXPECT(Atomic(true).load() == true); - EXPECT(Atomic(2).load() == 2); - EXPECT(Atomic(3).load() == 3); - EXPECT(Atomic(4).load() == 4); - - u16 v_u16 = 0; - EXPECT(Atomic(&v_u16).load() == &v_u16); + EXPECT(Atomic(false).load() == false); + EXPECT(Atomic(true).load() == true); + EXPECT(Atomic(2).load() == 2); + EXPECT(Atomic(3).load() == 3); + EXPECT(Atomic(4).load() == 4); + + u16 v_u16 = 0; + EXPECT(Atomic(&v_u16).load() == &v_u16); } TEST_CASE(do_exchange) { - Atomic a_bool(false); - EXPECT(a_bool.exchange(true) == false); - EXPECT(a_bool.load() == true && static_cast(a_bool) == true); + Atomic a_bool(false); + EXPECT(a_bool.exchange(true) == false); + EXPECT(a_bool.load() == true && static_cast(a_bool) == true); - Atomic a_u32(2); - EXPECT(a_u32.exchange(22) == 2); - EXPECT(a_u32.load() == 22 && static_cast(a_u32) == 22); + Atomic a_u32(2); + EXPECT(a_u32.exchange(22) == 2); + EXPECT(a_u32.load() == 22 && static_cast(a_u32) == 22); - Atomic a_u16(3); - EXPECT(a_u16.exchange(33) == 3); - EXPECT(a_u16.load() == 33 && static_cast(a_u16) == 33); + Atomic a_u16(3); + EXPECT(a_u16.exchange(33) == 3); + EXPECT(a_u16.load() == 33 && static_cast(a_u16) == 33); - Atomic a_u8(4); - EXPECT(a_u8.exchange(44) == 4); - EXPECT(a_u8.load() == 44 && static_cast(a_u8) == 44); - - u16 v_u16[6]; - Atomic a_pu16(&v_u16[2]); - EXPECT(a_pu16.load() == &v_u16[2] && static_cast(a_pu16) == &v_u16[2]); + Atomic a_u8(4); + EXPECT(a_u8.exchange(44) == 4); + EXPECT(a_u8.load() == 44 && static_cast(a_u8) == 44); + + u16 v_u16[6]; + Atomic a_pu16(&v_u16[2]); + EXPECT(a_pu16.load() == &v_u16[2] && static_cast(a_pu16) == &v_u16[2]); } TEST_CASE(do_compare_exchange) { - Atomic a_bool(false); - bool e_bool = true; - EXPECT(a_bool.compare_exchange_strong(e_bool, true) == false); - EXPECT(e_bool == false); - EXPECT(a_bool.load() == false && static_cast(a_bool) == false); - e_bool = false; - EXPECT(a_bool.compare_exchange_strong(e_bool, true) == true); - EXPECT(a_bool.load() == true && static_cast(a_bool) == true); + Atomic a_bool(false); + bool e_bool = true; + EXPECT(a_bool.compare_exchange_strong(e_bool, true) == false); + EXPECT(e_bool == false); + EXPECT(a_bool.load() == false && static_cast(a_bool) == false); + e_bool = false; + EXPECT(a_bool.compare_exchange_strong(e_bool, true) == true); + EXPECT(a_bool.load() == true && static_cast(a_bool) == true); - Atomic a_u32(2); - u32 e_u32 = 99; - EXPECT(a_u32.compare_exchange_strong(e_u32, 22) == false); - EXPECT(e_u32 == 2); - EXPECT(a_u32.load() == 2 && static_cast(a_u32) == 2); - e_u32 = 2; - EXPECT(a_u32.compare_exchange_strong(e_u32, 22) == true); - EXPECT(a_u32.load() == 22 && static_cast(a_u32) == 22); + Atomic a_u32(2); + u32 e_u32 = 99; + EXPECT(a_u32.compare_exchange_strong(e_u32, 22) == false); + EXPECT(e_u32 == 2); + EXPECT(a_u32.load() == 2 && static_cast(a_u32) == 2); + e_u32 = 2; + EXPECT(a_u32.compare_exchange_strong(e_u32, 22) == true); + EXPECT(a_u32.load() == 22 && static_cast(a_u32) == 22); - Atomic a_u16(3); - u16 e_u16 = 99; - EXPECT(a_u16.compare_exchange_strong(e_u16, 33) == false); - EXPECT(e_u16 == 3); - EXPECT(a_u16.load() == 3 && static_cast(a_u16) == 3); - e_u16 = 3; - EXPECT(a_u16.compare_exchange_strong(e_u16, 33) == true); - EXPECT(a_u16.load() == 33 && static_cast(a_u16) == 33); + Atomic a_u16(3); + u16 e_u16 = 99; + EXPECT(a_u16.compare_exchange_strong(e_u16, 33) == false); + EXPECT(e_u16 == 3); + EXPECT(a_u16.load() == 3 && static_cast(a_u16) == 3); + e_u16 = 3; + EXPECT(a_u16.compare_exchange_strong(e_u16, 33) == true); + EXPECT(a_u16.load() == 33 && static_cast(a_u16) == 33); - Atomic a_u8(4); - u8 e_u8 = 99; - EXPECT(a_u8.compare_exchange_strong(e_u8, 44) == false); - EXPECT(e_u8 == 4); - EXPECT(a_u8.load() == 4 && static_cast(a_u8) == 4); - e_u8 = 4; - EXPECT(a_u8.compare_exchange_strong(e_u8, 44) == true); - EXPECT(a_u8.load() == 44 && static_cast(a_u8) == 44); + Atomic a_u8(4); + u8 e_u8 = 99; + EXPECT(a_u8.compare_exchange_strong(e_u8, 44) == false); + EXPECT(e_u8 == 4); + EXPECT(a_u8.load() == 4 && static_cast(a_u8) == 4); + e_u8 = 4; + EXPECT(a_u8.compare_exchange_strong(e_u8, 44) == true); + EXPECT(a_u8.load() == 44 && static_cast(a_u8) == 44); } TEST_CASE(fetch_add) { - Atomic a_u32(5); - EXPECT(a_u32.fetch_add(2) == 5); - EXPECT(a_u32.load() == 7 && static_cast(a_u32) == 7); + Atomic a_u32(5); + EXPECT(a_u32.fetch_add(2) == 5); + EXPECT(a_u32.load() == 7 && static_cast(a_u32) == 7); - Atomic a_u16(5); - EXPECT(a_u16.fetch_add(2) == 5); - EXPECT(a_u16.load() == 7 && static_cast(a_u16) == 7); + Atomic a_u16(5); + EXPECT(a_u16.fetch_add(2) == 5); + EXPECT(a_u16.load() == 7 && static_cast(a_u16) == 7); - Atomic a_u8(5); - EXPECT(a_u8.fetch_add(2) == 5); - EXPECT(a_u8.load() == 7 && static_cast(a_u8) == 7); + Atomic a_u8(5); + EXPECT(a_u8.fetch_add(2) == 5); + EXPECT(a_u8.load() == 7 && static_cast(a_u8) == 7); - u32 v_u32[6]; - Atomic a_pu32(&v_u32[2]); - EXPECT(a_pu32.load() == &v_u32[2] && static_cast(a_pu32) == &v_u32[2]); - EXPECT(a_pu32.fetch_add(2) == &v_u32[2]); - EXPECT(a_pu32.load() == &v_u32[4] && static_cast(a_pu32) == &v_u32[4]); - EXPECT(a_pu32.fetch_add(-3) == &v_u32[4]); - EXPECT(a_pu32.load() == &v_u32[1] && static_cast(a_pu32) == &v_u32[1]); + u32 v_u32[6]; + Atomic a_pu32(&v_u32[2]); + EXPECT(a_pu32.load() == &v_u32[2] && static_cast(a_pu32) == &v_u32[2]); + EXPECT(a_pu32.fetch_add(2) == &v_u32[2]); + EXPECT(a_pu32.load() == &v_u32[4] && static_cast(a_pu32) == &v_u32[4]); + EXPECT(a_pu32.fetch_add(-3) == &v_u32[4]); + EXPECT(a_pu32.load() == &v_u32[1] && static_cast(a_pu32) == &v_u32[1]); - u16 v_u16[6]; - Atomic a_pu16(&v_u16[2]); - EXPECT(a_pu16.load() == &v_u16[2] && static_cast(a_pu16) == &v_u16[2]); - EXPECT(a_pu16.fetch_add(2) == &v_u16[2]); - EXPECT(a_pu16.load() == &v_u16[4] && static_cast(a_pu16) == &v_u16[4]); - EXPECT(a_pu16.fetch_add(-3) == &v_u16[4]); - EXPECT(a_pu16.load() == &v_u16[1] && static_cast(a_pu16) == &v_u16[1]); + u16 v_u16[6]; + Atomic a_pu16(&v_u16[2]); + EXPECT(a_pu16.load() == &v_u16[2] && static_cast(a_pu16) == &v_u16[2]); + EXPECT(a_pu16.fetch_add(2) == &v_u16[2]); + EXPECT(a_pu16.load() == &v_u16[4] && static_cast(a_pu16) == &v_u16[4]); + EXPECT(a_pu16.fetch_add(-3) == &v_u16[4]); + EXPECT(a_pu16.load() == &v_u16[1] && static_cast(a_pu16) == &v_u16[1]); - u8 v_u8[6]; - Atomic a_pu8(&v_u8[2]); - EXPECT(a_pu8.load() == &v_u8[2] && static_cast(a_pu8) == &v_u8[2]); - EXPECT(a_pu8.fetch_add(2) == &v_u8[2]); - EXPECT(a_pu8.load() == &v_u8[4] && static_cast(a_pu8) == &v_u8[4]); - EXPECT(a_pu8.fetch_add(-3) == &v_u8[4]); - EXPECT(a_pu8.load() == &v_u8[1] && static_cast(a_pu8) == &v_u8[1]); + u8 v_u8[6]; + Atomic a_pu8(&v_u8[2]); + EXPECT(a_pu8.load() == &v_u8[2] && static_cast(a_pu8) == &v_u8[2]); + EXPECT(a_pu8.fetch_add(2) == &v_u8[2]); + EXPECT(a_pu8.load() == &v_u8[4] && static_cast(a_pu8) == &v_u8[4]); + EXPECT(a_pu8.fetch_add(-3) == &v_u8[4]); + EXPECT(a_pu8.load() == &v_u8[1] && static_cast(a_pu8) == &v_u8[1]); } TEST_CASE(fetch_sub) { - Atomic a_u32(5); - EXPECT(a_u32.fetch_sub(2) == 5); - EXPECT(a_u32.load() == 3 && static_cast(a_u32) == 3); + Atomic a_u32(5); + EXPECT(a_u32.fetch_sub(2) == 5); + EXPECT(a_u32.load() == 3 && static_cast(a_u32) == 3); - Atomic a_u16(5); - EXPECT(a_u16.fetch_sub(2) == 5); - EXPECT(a_u16.load() == 3 && static_cast(a_u16) == 3); + Atomic a_u16(5); + EXPECT(a_u16.fetch_sub(2) == 5); + EXPECT(a_u16.load() == 3 && static_cast(a_u16) == 3); - Atomic a_u8(5); - EXPECT(a_u8.fetch_sub(2) == 5); - EXPECT(a_u8.load() == 3 && static_cast(a_u8) == 3); + Atomic a_u8(5); + EXPECT(a_u8.fetch_sub(2) == 5); + EXPECT(a_u8.load() == 3 && static_cast(a_u8) == 3); - u32 v_u32[6]; - Atomic a_pu32(&v_u32[2]); - EXPECT(a_pu32.load() == &v_u32[2] && static_cast(a_pu32) == &v_u32[2]); - EXPECT(a_pu32.fetch_sub(2) == &v_u32[2]); - EXPECT(a_pu32.load() == &v_u32[0] && static_cast(a_pu32) == &v_u32[0]); - EXPECT(a_pu32.fetch_sub(-3) == &v_u32[0]); - EXPECT(a_pu32.load() == &v_u32[3] && static_cast(a_pu32) == &v_u32[3]); + u32 v_u32[6]; + Atomic a_pu32(&v_u32[2]); + EXPECT(a_pu32.load() == &v_u32[2] && static_cast(a_pu32) == &v_u32[2]); + EXPECT(a_pu32.fetch_sub(2) == &v_u32[2]); + EXPECT(a_pu32.load() == &v_u32[0] && static_cast(a_pu32) == &v_u32[0]); + EXPECT(a_pu32.fetch_sub(-3) == &v_u32[0]); + EXPECT(a_pu32.load() == &v_u32[3] && static_cast(a_pu32) == &v_u32[3]); - u16 v_u16[6]; - Atomic a_pu16(&v_u16[2]); - EXPECT(a_pu16.load() == &v_u16[2] && static_cast(a_pu16) == &v_u16[2]); - EXPECT(a_pu16.fetch_sub(2) == &v_u16[2]); - EXPECT(a_pu16.load() == &v_u16[0] && static_cast(a_pu16) == &v_u16[0]); - EXPECT(a_pu16.fetch_sub(-3) == &v_u16[0]); - EXPECT(a_pu16.load() == &v_u16[3] && static_cast(a_pu16) == &v_u16[3]); + u16 v_u16[6]; + Atomic a_pu16(&v_u16[2]); + EXPECT(a_pu16.load() == &v_u16[2] && static_cast(a_pu16) == &v_u16[2]); + EXPECT(a_pu16.fetch_sub(2) == &v_u16[2]); + EXPECT(a_pu16.load() == &v_u16[0] && static_cast(a_pu16) == &v_u16[0]); + EXPECT(a_pu16.fetch_sub(-3) == &v_u16[0]); + EXPECT(a_pu16.load() == &v_u16[3] && static_cast(a_pu16) == &v_u16[3]); - u8 v_u8[6]; - Atomic a_pu8(&v_u8[2]); - EXPECT(a_pu8.load() == &v_u8[2] && static_cast(a_pu8) == &v_u8[2]); - EXPECT(a_pu8.fetch_sub(2) == &v_u8[2]); - EXPECT(a_pu8.load() == &v_u8[0] && static_cast(a_pu8) == &v_u8[0]); - EXPECT(a_pu8.fetch_sub(-3) == &v_u8[0]); - EXPECT(a_pu8.load() == &v_u8[3] && static_cast(a_pu8) == &v_u8[3]); + u8 v_u8[6]; + Atomic a_pu8(&v_u8[2]); + EXPECT(a_pu8.load() == &v_u8[2] && static_cast(a_pu8) == &v_u8[2]); + EXPECT(a_pu8.fetch_sub(2) == &v_u8[2]); + EXPECT(a_pu8.load() == &v_u8[0] && static_cast(a_pu8) == &v_u8[0]); + EXPECT(a_pu8.fetch_sub(-3) == &v_u8[0]); + EXPECT(a_pu8.load() == &v_u8[3] && static_cast(a_pu8) == &v_u8[3]); } TEST_CASE(fetch_inc) { - Atomic a_u32(5); - EXPECT(a_u32++ == 5); - EXPECT(a_u32.load() == 6 && a_u32 == 6); - EXPECT(++a_u32 == 7); - EXPECT(a_u32.load() == 7 && a_u32 == 7); - EXPECT((a_u32 += 2) == 9); - EXPECT(a_u32.load() == 9 && a_u32 == 9); + Atomic a_u32(5); + EXPECT(a_u32++ == 5); + EXPECT(a_u32.load() == 6 && a_u32 == 6); + EXPECT(++a_u32 == 7); + EXPECT(a_u32.load() == 7 && a_u32 == 7); + EXPECT((a_u32 += 2) == 9); + EXPECT(a_u32.load() == 9 && a_u32 == 9); - Atomic a_u16(5); - EXPECT(a_u16++ == 5); - EXPECT(a_u16.load() == 6 && a_u16 == 6); - EXPECT(++a_u16 == 7); - EXPECT(a_u16.load() == 7 && a_u16 == 7); - EXPECT((a_u16 += 2) == 9); - EXPECT(a_u16.load() == 9 && a_u16 == 9); + Atomic a_u16(5); + EXPECT(a_u16++ == 5); + EXPECT(a_u16.load() == 6 && a_u16 == 6); + EXPECT(++a_u16 == 7); + EXPECT(a_u16.load() == 7 && a_u16 == 7); + EXPECT((a_u16 += 2) == 9); + EXPECT(a_u16.load() == 9 && a_u16 == 9); - Atomic a_u8(5); - EXPECT(a_u8++ == 5); - EXPECT(a_u8.load() == 6 && a_u8 == 6); - EXPECT(++a_u8 == 7); - EXPECT(a_u8.load() == 7 && a_u8 == 7); - EXPECT((a_u8 += 2) == 9); - EXPECT(a_u8.load() == 9 && a_u8 == 9); + Atomic a_u8(5); + EXPECT(a_u8++ == 5); + EXPECT(a_u8.load() == 6 && a_u8 == 6); + EXPECT(++a_u8 == 7); + EXPECT(a_u8.load() == 7 && a_u8 == 7); + EXPECT((a_u8 += 2) == 9); + EXPECT(a_u8.load() == 9 && a_u8 == 9); - u32 v_u32[8]; - Atomic a_pu32(&v_u32[2]); - EXPECT(a_pu32++ == &v_u32[2]); - EXPECT(a_pu32.load() == &v_u32[3] && a_pu32 == &v_u32[3]); - EXPECT(++a_pu32 == &v_u32[4]); - EXPECT(a_pu32.load() == &v_u32[4] && a_pu32 == &v_u32[4]); - EXPECT((a_pu32 += 2) == &v_u32[6]); - EXPECT(a_pu32.load() == &v_u32[6] && a_pu32 == &v_u32[6]); + u32 v_u32[8]; + Atomic a_pu32(&v_u32[2]); + EXPECT(a_pu32++ == &v_u32[2]); + EXPECT(a_pu32.load() == &v_u32[3] && a_pu32 == &v_u32[3]); + EXPECT(++a_pu32 == &v_u32[4]); + EXPECT(a_pu32.load() == &v_u32[4] && a_pu32 == &v_u32[4]); + EXPECT((a_pu32 += 2) == &v_u32[6]); + EXPECT(a_pu32.load() == &v_u32[6] && a_pu32 == &v_u32[6]); - u16 v_u16[8]; - Atomic a_pu16(&v_u16[2]); - EXPECT(a_pu16++ == &v_u16[2]); - EXPECT(a_pu16.load() == &v_u16[3] && a_pu16 == &v_u16[3]); - EXPECT(++a_pu16 == &v_u16[4]); - EXPECT(a_pu16.load() == &v_u16[4] && a_pu16 == &v_u16[4]); - EXPECT((a_pu16 += 2) == &v_u16[6]); - EXPECT(a_pu16.load() == &v_u16[6] && a_pu16 == &v_u16[6]); + u16 v_u16[8]; + Atomic a_pu16(&v_u16[2]); + EXPECT(a_pu16++ == &v_u16[2]); + EXPECT(a_pu16.load() == &v_u16[3] && a_pu16 == &v_u16[3]); + EXPECT(++a_pu16 == &v_u16[4]); + EXPECT(a_pu16.load() == &v_u16[4] && a_pu16 == &v_u16[4]); + EXPECT((a_pu16 += 2) == &v_u16[6]); + EXPECT(a_pu16.load() == &v_u16[6] && a_pu16 == &v_u16[6]); - u8 v_u8[8]; - Atomic a_pu8(&v_u8[2]); - EXPECT(a_pu8++ == &v_u8[2]); - EXPECT(a_pu8.load() == &v_u8[3] && a_pu8 == &v_u8[3]); - EXPECT(++a_pu8 == &v_u8[4]); - EXPECT(a_pu8.load() == &v_u8[4] && a_pu8 == &v_u8[4]); - EXPECT((a_pu8 += 2) == &v_u8[6]); - EXPECT(a_pu8.load() == &v_u8[6] && a_pu8 == &v_u8[6]); + u8 v_u8[8]; + Atomic a_pu8(&v_u8[2]); + EXPECT(a_pu8++ == &v_u8[2]); + EXPECT(a_pu8.load() == &v_u8[3] && a_pu8 == &v_u8[3]); + EXPECT(++a_pu8 == &v_u8[4]); + EXPECT(a_pu8.load() == &v_u8[4] && a_pu8 == &v_u8[4]); + EXPECT((a_pu8 += 2) == &v_u8[6]); + EXPECT(a_pu8.load() == &v_u8[6] && a_pu8 == &v_u8[6]); } TEST_CASE(fetch_dec) { - Atomic a_u32(5); - EXPECT(a_u32-- == 5); - EXPECT(a_u32.load() == 4 && a_u32 == 4); - EXPECT(--a_u32 == 3); - EXPECT(a_u32.load() == 3 && a_u32 == 3); - EXPECT((a_u32 -= 2) == 1); - EXPECT(a_u32.load() == 1 && a_u32 == 1); + Atomic a_u32(5); + EXPECT(a_u32-- == 5); + EXPECT(a_u32.load() == 4 && a_u32 == 4); + EXPECT(--a_u32 == 3); + EXPECT(a_u32.load() == 3 && a_u32 == 3); + EXPECT((a_u32 -= 2) == 1); + EXPECT(a_u32.load() == 1 && a_u32 == 1); - Atomic a_u16(5); - EXPECT(a_u16-- == 5); - EXPECT(a_u16.load() == 4 && a_u16 == 4); - EXPECT(--a_u16 == 3); - EXPECT(a_u16.load() == 3 && a_u16 == 3); - EXPECT((a_u16 -= 2) == 1); - EXPECT(a_u16.load() == 1 && a_u16 == 1); + Atomic a_u16(5); + EXPECT(a_u16-- == 5); + EXPECT(a_u16.load() == 4 && a_u16 == 4); + EXPECT(--a_u16 == 3); + EXPECT(a_u16.load() == 3 && a_u16 == 3); + EXPECT((a_u16 -= 2) == 1); + EXPECT(a_u16.load() == 1 && a_u16 == 1); - Atomic a_u8(5); - EXPECT(a_u8-- == 5); - EXPECT(a_u8.load() == 4 && a_u8 == 4); - EXPECT(--a_u8 == 3); - EXPECT(a_u8.load() == 3 && a_u8 == 3); - EXPECT((a_u8 -= 2) == 1); - EXPECT(a_u8.load() == 1 && a_u8 == 1); + Atomic a_u8(5); + EXPECT(a_u8-- == 5); + EXPECT(a_u8.load() == 4 && a_u8 == 4); + EXPECT(--a_u8 == 3); + EXPECT(a_u8.load() == 3 && a_u8 == 3); + EXPECT((a_u8 -= 2) == 1); + EXPECT(a_u8.load() == 1 && a_u8 == 1); - u32 v_u32[8]; - Atomic a_pu32(&v_u32[7]); - EXPECT(a_pu32-- == &v_u32[7]); - EXPECT(a_pu32.load() == &v_u32[6] && a_pu32 == &v_u32[6]); - EXPECT(--a_pu32 == &v_u32[5]); - EXPECT(a_pu32.load() == &v_u32[5] && a_pu32 == &v_u32[5]); - EXPECT((a_pu32 -= 2) == &v_u32[3]); - EXPECT(a_pu32.load() == &v_u32[3] && a_pu32 == &v_u32[3]); + u32 v_u32[8]; + Atomic a_pu32(&v_u32[7]); + EXPECT(a_pu32-- == &v_u32[7]); + EXPECT(a_pu32.load() == &v_u32[6] && a_pu32 == &v_u32[6]); + EXPECT(--a_pu32 == &v_u32[5]); + EXPECT(a_pu32.load() == &v_u32[5] && a_pu32 == &v_u32[5]); + EXPECT((a_pu32 -= 2) == &v_u32[3]); + EXPECT(a_pu32.load() == &v_u32[3] && a_pu32 == &v_u32[3]); - u16 v_u16[8]; - Atomic a_pu16(&v_u16[7]); - EXPECT(a_pu16-- == &v_u16[7]); - EXPECT(a_pu16.load() == &v_u16[6] && a_pu16 == &v_u16[6]); - EXPECT(--a_pu16 == &v_u16[5]); - EXPECT(a_pu16.load() == &v_u16[5] && a_pu16 == &v_u16[5]); - EXPECT((a_pu16 -= 2) == &v_u16[3]); - EXPECT(a_pu16.load() == &v_u16[3] && a_pu16 == &v_u16[3]); + u16 v_u16[8]; + Atomic a_pu16(&v_u16[7]); + EXPECT(a_pu16-- == &v_u16[7]); + EXPECT(a_pu16.load() == &v_u16[6] && a_pu16 == &v_u16[6]); + EXPECT(--a_pu16 == &v_u16[5]); + EXPECT(a_pu16.load() == &v_u16[5] && a_pu16 == &v_u16[5]); + EXPECT((a_pu16 -= 2) == &v_u16[3]); + EXPECT(a_pu16.load() == &v_u16[3] && a_pu16 == &v_u16[3]); - u8 v_u8[8]; - Atomic a_pu8(&v_u8[7]); - EXPECT(a_pu8-- == &v_u8[7]); - EXPECT(a_pu8.load() == &v_u8[6] && a_pu8 == &v_u8[6]); - EXPECT(--a_pu8 == &v_u8[5]); - EXPECT(a_pu8.load() == &v_u8[5] && a_pu8 == &v_u8[5]); - EXPECT((a_pu8 -= 2) == &v_u8[3]); - EXPECT(a_pu8.load() == &v_u8[3] && a_pu8 == &v_u8[3]); + u8 v_u8[8]; + Atomic a_pu8(&v_u8[7]); + EXPECT(a_pu8-- == &v_u8[7]); + EXPECT(a_pu8.load() == &v_u8[6] && a_pu8 == &v_u8[6]); + EXPECT(--a_pu8 == &v_u8[5]); + EXPECT(a_pu8.load() == &v_u8[5] && a_pu8 == &v_u8[5]); + EXPECT((a_pu8 -= 2) == &v_u8[3]); + EXPECT(a_pu8.load() == &v_u8[3] && a_pu8 == &v_u8[3]); } TEST_CASE(fetch_and) { - Atomic a_u32(0xdeadbeef); - EXPECT(a_u32.fetch_and(0x8badf00d) == 0xdeadbeef); - EXPECT(a_u32.load() == 0x8aadb00d && static_cast(a_u32) == 0x8aadb00d); - a_u32 = 0xdeadbeef; - EXPECT((a_u32 &= 0x8badf00d) == 0x8aadb00d); + Atomic a_u32(0xdeadbeef); + EXPECT(a_u32.fetch_and(0x8badf00d) == 0xdeadbeef); + EXPECT(a_u32.load() == 0x8aadb00d && static_cast(a_u32) == 0x8aadb00d); + a_u32 = 0xdeadbeef; + EXPECT((a_u32 &= 0x8badf00d) == 0x8aadb00d); - Atomic a_u16(0xbeef); - EXPECT(a_u16.fetch_and(0xf00d) == 0xbeef); - EXPECT(a_u16.load() == 0xb00d && static_cast(a_u16) == 0xb00d); - a_u16 = 0xbeef; - EXPECT((a_u16 &= 0xf00d) == 0xb00d); + Atomic a_u16(0xbeef); + EXPECT(a_u16.fetch_and(0xf00d) == 0xbeef); + EXPECT(a_u16.load() == 0xb00d && static_cast(a_u16) == 0xb00d); + a_u16 = 0xbeef; + EXPECT((a_u16 &= 0xf00d) == 0xb00d); - Atomic a_u8(0xef); - EXPECT(a_u8.fetch_and(0x0d) == 0xef); - EXPECT(a_u8.load() == 0x0d && static_cast(a_u8) == 0x0d); - a_u8 = 0xef; - EXPECT((a_u8 &= 0x0d) == 0x0d); + Atomic a_u8(0xef); + EXPECT(a_u8.fetch_and(0x0d) == 0xef); + EXPECT(a_u8.load() == 0x0d && static_cast(a_u8) == 0x0d); + a_u8 = 0xef; + EXPECT((a_u8 &= 0x0d) == 0x0d); } TEST_CASE(fetch_or) { - Atomic a_u32(0xaadb00d); - EXPECT(a_u32.fetch_or(0xdeadbeef) == 0xaadb00d); - EXPECT(a_u32.load() == 0xdeadbeef && static_cast(a_u32) == 0xdeadbeef); - a_u32 = 0xaadb00d; - EXPECT((a_u32 |= 0xdeadbeef) == 0xdeadbeef); + Atomic a_u32(0xaadb00d); + EXPECT(a_u32.fetch_or(0xdeadbeef) == 0xaadb00d); + EXPECT(a_u32.load() == 0xdeadbeef && static_cast(a_u32) == 0xdeadbeef); + a_u32 = 0xaadb00d; + EXPECT((a_u32 |= 0xdeadbeef) == 0xdeadbeef); - Atomic a_u16(0xb00d); - EXPECT(a_u16.fetch_or(0xbeef) == 0xb00d); - EXPECT(a_u16.load() == 0xbeef && static_cast(a_u16) == 0xbeef); - a_u16 = 0xb00d; - EXPECT((a_u16 |= 0xbeef) == 0xbeef); + Atomic a_u16(0xb00d); + EXPECT(a_u16.fetch_or(0xbeef) == 0xb00d); + EXPECT(a_u16.load() == 0xbeef && static_cast(a_u16) == 0xbeef); + a_u16 = 0xb00d; + EXPECT((a_u16 |= 0xbeef) == 0xbeef); - Atomic a_u8(0x0d); - EXPECT(a_u8.fetch_or(0xef) == 0x0d); - EXPECT(a_u8.load() == 0xef && static_cast(a_u8) == 0xef); - a_u8 = 0x0d; - EXPECT((a_u8 |= 0xef) == 0xef); + Atomic a_u8(0x0d); + EXPECT(a_u8.fetch_or(0xef) == 0x0d); + EXPECT(a_u8.load() == 0xef && static_cast(a_u8) == 0xef); + a_u8 = 0x0d; + EXPECT((a_u8 |= 0xef) == 0xef); } TEST_CASE(fetch_xor) { - Atomic a_u32(0x55004ee2); - EXPECT(a_u32.fetch_xor(0xdeadbeef) == 0x55004ee2); - EXPECT(a_u32.load() == 0x8badf00d && static_cast(a_u32) == 0x8badf00d); - a_u32 = 0x55004ee2; - EXPECT((a_u32 ^= 0xdeadbeef) == 0x8badf00d); + Atomic a_u32(0x55004ee2); + EXPECT(a_u32.fetch_xor(0xdeadbeef) == 0x55004ee2); + EXPECT(a_u32.load() == 0x8badf00d && static_cast(a_u32) == 0x8badf00d); + a_u32 = 0x55004ee2; + EXPECT((a_u32 ^= 0xdeadbeef) == 0x8badf00d); - Atomic a_u16(0x4ee2); - EXPECT(a_u16.fetch_xor(0xbeef) == 0x4ee2); - EXPECT(a_u16.load() == 0xf00d && static_cast(a_u16) == 0xf00d); - a_u16 = 0x4ee2; - EXPECT((a_u16 ^= 0xbeef) == 0xf00d); + Atomic a_u16(0x4ee2); + EXPECT(a_u16.fetch_xor(0xbeef) == 0x4ee2); + EXPECT(a_u16.load() == 0xf00d && static_cast(a_u16) == 0xf00d); + a_u16 = 0x4ee2; + EXPECT((a_u16 ^= 0xbeef) == 0xf00d); - Atomic a_u8(0xe2); - EXPECT(a_u8.fetch_xor(0xef) == 0xe2); - EXPECT(a_u8.load() == 0x0d && static_cast(a_u8) == 0x0d); - a_u8 = 0xe2; - EXPECT((a_u8 ^= 0xef) == 0x0d); + Atomic a_u8(0xe2); + EXPECT(a_u8.fetch_xor(0xef) == 0xe2); + EXPECT(a_u8.load() == 0x0d && static_cast(a_u8) == 0x0d); + a_u8 = 0xe2; + EXPECT((a_u8 ^= 0xef) == 0x0d); } TEST_MAIN(Atomic) diff --git a/AK/Tests/TestCircularDeque.cpp b/AK/Tests/TestCircularDeque.cpp index 7a9d7070b19..7263d12b894 100644 --- a/AK/Tests/TestCircularDeque.cpp +++ b/AK/Tests/TestCircularDeque.cpp @@ -59,7 +59,7 @@ TEST_CASE(enqueue_begin_being_moved_from) { CircularDeque strings; - String str{"test"}; + String str { "test" }; strings.enqueue_begin(move(str)); EXPECT(str.is_null()); } diff --git a/AK/Tests/TestOptional.cpp b/AK/Tests/TestOptional.cpp index 8128249dd11..76b3bf26761 100644 --- a/AK/Tests/TestOptional.cpp +++ b/AK/Tests/TestOptional.cpp @@ -26,8 +26,8 @@ #include -#include #include +#include TEST_CASE(basic_optional) { diff --git a/AK/Tests/TestQueue.cpp b/AK/Tests/TestQueue.cpp index 4ddaba9511c..0142324ec28 100644 --- a/AK/Tests/TestQueue.cpp +++ b/AK/Tests/TestQueue.cpp @@ -26,8 +26,8 @@ #include -#include #include +#include TEST_CASE(construct) { diff --git a/AK/Tests/TestRefPtr.cpp b/AK/Tests/TestRefPtr.cpp index 5208dc03adf..93261ce2e00 100644 --- a/AK/Tests/TestRefPtr.cpp +++ b/AK/Tests/TestRefPtr.cpp @@ -80,14 +80,14 @@ TEST_CASE(assign_copy_self) RefPtr object = adopt(*new Object); EXPECT_EQ(object->ref_count(), 1u); - #ifdef __clang__ - #pragma clang diagnostic push - #pragma clang diagnostic ignored "-Wself-assign-overloaded" - #endif - object = object; - #ifdef __clang__ - #pragma clang diagnostic pop - #endif +#ifdef __clang__ +# pragma clang diagnostic push +# pragma clang diagnostic ignored "-Wself-assign-overloaded" +#endif + object = object; +#ifdef __clang__ +# pragma clang diagnostic pop +#endif EXPECT_EQ(object->ref_count(), 1u); } diff --git a/AK/Tests/TestStringUtils.cpp b/AK/Tests/TestStringUtils.cpp index d2a18d48df3..a34c91c253f 100644 --- a/AK/Tests/TestStringUtils.cpp +++ b/AK/Tests/TestStringUtils.cpp @@ -157,22 +157,22 @@ TEST_CASE(ends_with) { String test_string = "ABCDEF"; EXPECT(AK::StringUtils::ends_with(test_string, "DEF", CaseSensitivity::CaseSensitive)); - EXPECT(AK::StringUtils::ends_with(test_string, "ABCDEF", CaseSensitivity::CaseSensitive)); + EXPECT(AK::StringUtils::ends_with(test_string, "ABCDEF", CaseSensitivity::CaseSensitive)); EXPECT(!AK::StringUtils::ends_with(test_string, "ABCDE", CaseSensitivity::CaseSensitive)); EXPECT(!AK::StringUtils::ends_with(test_string, "ABCDEFG", CaseSensitivity::CaseSensitive)); - EXPECT(AK::StringUtils::ends_with(test_string, "def", CaseSensitivity::CaseInsensitive)); - EXPECT(!AK::StringUtils::ends_with(test_string, "def", CaseSensitivity::CaseSensitive)); + EXPECT(AK::StringUtils::ends_with(test_string, "def", CaseSensitivity::CaseInsensitive)); + EXPECT(!AK::StringUtils::ends_with(test_string, "def", CaseSensitivity::CaseSensitive)); } TEST_CASE(starts_with) { String test_string = "ABCDEF"; EXPECT(AK::StringUtils::starts_with(test_string, "ABC", CaseSensitivity::CaseSensitive)); - EXPECT(AK::StringUtils::starts_with(test_string, "ABCDEF", CaseSensitivity::CaseSensitive)); + EXPECT(AK::StringUtils::starts_with(test_string, "ABCDEF", CaseSensitivity::CaseSensitive)); EXPECT(!AK::StringUtils::starts_with(test_string, "BCDEF", CaseSensitivity::CaseSensitive)); EXPECT(!AK::StringUtils::starts_with(test_string, "ABCDEFG", CaseSensitivity::CaseSensitive)); - EXPECT(AK::StringUtils::starts_with(test_string, "abc", CaseSensitivity::CaseInsensitive)); - EXPECT(!AK::StringUtils::starts_with(test_string, "abc", CaseSensitivity::CaseSensitive)); + EXPECT(AK::StringUtils::starts_with(test_string, "abc", CaseSensitivity::CaseInsensitive)); + EXPECT(!AK::StringUtils::starts_with(test_string, "abc", CaseSensitivity::CaseSensitive)); } TEST_MAIN(StringUtils) diff --git a/AK/Time.h b/AK/Time.h index e1475033d47..a919f36f5e2 100644 --- a/AK/Time.h +++ b/AK/Time.h @@ -133,8 +133,8 @@ inline void timespec_add_timeval(const TimespecType& a, const TimevalType& b, Ti template inline void timeval_to_timespec(const TimevalType& tv, TimespecType& ts) { - ts.tv_sec = tv.tv_sec; - ts.tv_nsec = tv.tv_usec * 1000; + ts.tv_sec = tv.tv_sec; + ts.tv_nsec = tv.tv_usec * 1000; } template diff --git a/AK/WeakPtr.h b/AK/WeakPtr.h index bec76e7d96a..e5f50e77887 100644 --- a/AK/WeakPtr.h +++ b/AK/WeakPtr.h @@ -39,8 +39,8 @@ class WeakPtr { friend class Weakable; public: - WeakPtr() {} - WeakPtr(std::nullptr_t) {} + WeakPtr() { } + WeakPtr(std::nullptr_t) { } template WeakPtr(WeakPtr&& other)