diff --git a/AK/DistinctNumeric.h b/AK/DistinctNumeric.h index f3546c220b7..08689934194 100644 --- a/AK/DistinctNumeric.h +++ b/AK/DistinctNumeric.h @@ -46,13 +46,13 @@ namespace AK { */ namespace DistinctNumericFeature { -enum Arithmetic {}; -enum CastToBool {}; -enum CastToUnderlying {}; -enum Comparison {}; -enum Flags {}; -enum Increment {}; -enum Shift {}; +enum Arithmetic { }; +enum CastToBool { }; +enum CastToUnderlying { }; +enum Comparison { }; +enum Flags { }; +enum Increment { }; +enum Shift { }; }; template diff --git a/AK/IntrusiveList.h b/AK/IntrusiveList.h index 5a289f8c9c0..2617f55ccd7 100644 --- a/AK/IntrusiveList.h +++ b/AK/IntrusiveList.h @@ -20,9 +20,9 @@ class IntrusiveListNode; struct ExtractIntrusiveListTypes { template - static V value(IntrusiveListNode T::*x); + static V value(IntrusiveListNode T::* x); template - static Container container(IntrusiveListNode T::*x); + static Container container(IntrusiveListNode T::* x); }; template> @@ -33,14 +33,14 @@ class IntrusiveListStorage { private: friend class IntrusiveListNode; - template T_::*member> + template T_::* member> friend class IntrusiveList; SubstitutedIntrusiveListNode* m_first { nullptr }; SubstitutedIntrusiveListNode* m_last { nullptr }; }; -template T::*member> +template T::* member> class IntrusiveList { AK_MAKE_NONCOPYABLE(IntrusiveList); AK_MAKE_NONMOVABLE(IntrusiveList); @@ -162,7 +162,7 @@ public: // to be of equal types. so for now, just make the members public on clang. #if !defined(AK_COMPILER_CLANG) private: - template T_::*member> + template T_::* member> friend class ::AK::Detail::IntrusiveList; #endif @@ -172,7 +172,7 @@ private: [[no_unique_address]] SelfReferenceIfNeeded m_self; }; -template T::*member> +template T::* member> inline typename IntrusiveList::Iterator& IntrusiveList::Iterator::erase() { auto old = m_value; @@ -181,26 +181,26 @@ inline typename IntrusiveList::Iterator& IntrusiveList T::*member> +template T::* member> inline IntrusiveList::~IntrusiveList() { clear(); } -template T::*member> +template T::* member> inline void IntrusiveList::clear() { while (m_storage.m_first) m_storage.m_first->remove(); } -template T::*member> +template T::* member> inline bool IntrusiveList::is_empty() const { return m_storage.m_first == nullptr; } -template T::*member> +template T::* member> inline size_t IntrusiveList::size_slow() const { size_t size = 0; @@ -211,7 +211,7 @@ inline size_t IntrusiveList::size_slow() const return size; } -template T::*member> +template T::* member> inline void IntrusiveList::append(T& n) { remove(n); @@ -230,7 +230,7 @@ inline void IntrusiveList::append(T& n) m_storage.m_first = &nnode; } -template T::*member> +template T::* member> inline void IntrusiveList::prepend(T& n) { remove(n); @@ -249,7 +249,7 @@ inline void IntrusiveList::prepend(T& n) m_storage.m_last = &nnode; } -template T::*member> +template T::* member> inline void IntrusiveList::insert_before(T& bn, T& n) { remove(n); @@ -271,7 +271,7 @@ inline void IntrusiveList::insert_before(T& bn, T& n) new_node.m_self.reference = &n; } -template T::*member> +template T::* member> inline void IntrusiveList::remove(T& n) { auto& nnode = n.*member; @@ -279,20 +279,20 @@ inline void IntrusiveList::remove(T& n) nnode.remove(); } -template T::*member> +template T::* member> inline bool IntrusiveList::contains(T const& n) const { auto& nnode = n.*member; return nnode.m_storage == &m_storage; } -template T::*member> +template T::* member> inline Container IntrusiveList::first() const { return m_storage.m_first ? node_to_value(*m_storage.m_first) : nullptr; } -template T::*member> +template T::* member> inline Container IntrusiveList::take_first() { if (Container ptr = first()) { @@ -302,7 +302,7 @@ inline Container IntrusiveList::take_first() return nullptr; } -template T::*member> +template T::* member> inline Container IntrusiveList::take_last() { if (Container ptr = last()) { @@ -312,13 +312,13 @@ inline Container IntrusiveList::take_last() return nullptr; } -template T::*member> +template T::* member> inline Container IntrusiveList::last() const { return m_storage.m_last ? node_to_value(*m_storage.m_last) : nullptr; } -template T::*member> +template T::* member> inline T const* IntrusiveList::next(T const* current) { auto& nextnode = (current->*member).m_next; @@ -326,7 +326,7 @@ inline T const* IntrusiveList::next(T const* current) return nextstruct; } -template T::*member> +template T::* member> inline T const* IntrusiveList::prev(T const* current) { auto& prevnode = (current->*member).m_prev; @@ -334,7 +334,7 @@ inline T const* IntrusiveList::prev(T const* current) return prevstruct; } -template T::*member> +template T::* member> inline T* IntrusiveList::next(T* current) { auto& nextnode = (current->*member).m_next; @@ -342,7 +342,7 @@ inline T* IntrusiveList::next(T* current) return nextstruct; } -template T::*member> +template T::* member> inline T* IntrusiveList::prev(T* current) { auto& prevnode = (current->*member).m_prev; @@ -350,25 +350,25 @@ inline T* IntrusiveList::prev(T* current) return prevstruct; } -template T::*member> +template T::* member> inline typename IntrusiveList::Iterator IntrusiveList::begin() { return m_storage.m_first ? Iterator(node_to_value(*m_storage.m_first)) : Iterator(); } -template T::*member> +template T::* member> inline typename IntrusiveList::ReverseIterator IntrusiveList::rbegin() { return m_storage.m_last ? ReverseIterator(node_to_value(*m_storage.m_last)) : ReverseIterator(); } -template T::*member> +template T::* member> inline typename IntrusiveList::ConstIterator IntrusiveList::begin() const { return m_storage.m_first ? ConstIterator(node_to_value(*m_storage.m_first)) : ConstIterator(); } -template T::*member> +template T::* member> inline T* IntrusiveList::node_to_value(SubstitutedIntrusiveListNode& node) { // Note: A data member pointer is a 32-bit offset in the Windows ABI (both x86 and x86_64), @@ -422,7 +422,7 @@ inline bool IntrusiveListNode::is_in_list() const // By default, intrusive lists cannot contain null entries anyway, so switch to RefPtr // and just make the user-facing functions deref the pointers. -template> T::*member> +template> T::* member> class IntrusiveList, member> : public IntrusiveList, member> { public: [[nodiscard]] NonnullRefPtr first() const { return *IntrusiveList, member>::first(); } diff --git a/AK/IntrusiveListRelaxedConst.h b/AK/IntrusiveListRelaxedConst.h index 7faef434e62..1fc5d593514 100644 --- a/AK/IntrusiveListRelaxedConst.h +++ b/AK/IntrusiveListRelaxedConst.h @@ -11,7 +11,7 @@ namespace AK { namespace Detail { -template T::*member> +template T::* member> class IntrusiveListRelaxedConst : public IntrusiveList { AK_MAKE_NONCOPYABLE(IntrusiveListRelaxedConst); AK_MAKE_NONMOVABLE(IntrusiveListRelaxedConst); diff --git a/AK/IntrusiveRedBlackTree.h b/AK/IntrusiveRedBlackTree.h index ae20c16f02c..fb2db6b92e1 100644 --- a/AK/IntrusiveRedBlackTree.h +++ b/AK/IntrusiveRedBlackTree.h @@ -16,17 +16,17 @@ class IntrusiveRedBlackTreeNode; struct ExtractIntrusiveRedBlackTreeTypes { template - static K key(IntrusiveRedBlackTreeNode T::*x); + static K key(IntrusiveRedBlackTreeNode T::* x); template - static V value(IntrusiveRedBlackTreeNode T::*x); + static V value(IntrusiveRedBlackTreeNode T::* x); template - static Container container(IntrusiveRedBlackTreeNode T::*x); + static Container container(IntrusiveRedBlackTreeNode T::* x); }; template> using SubstitutedIntrusiveRedBlackTreeNode = IntrusiveRedBlackTreeNode::Type>; -template V::*member> +template V::* member> class IntrusiveRedBlackTree : public BaseRedBlackTree { public: @@ -200,7 +200,7 @@ public: #if !defined(AK_COMPILER_CLANG) private: - template TV::*member> + template TV::* member> friend class ::AK::Detail::IntrusiveRedBlackTree; #endif @@ -211,7 +211,7 @@ private: // Specialise IntrusiveRedBlackTree for NonnullRefPtr // By default, red black trees cannot contain null entries anyway, so switch to RefPtr // and just make the user-facing functions deref the pointers. -template> V::*member> +template> V::* member> class IntrusiveRedBlackTree, member> : public IntrusiveRedBlackTree, member> { public: [[nodiscard]] NonnullRefPtr find(K key) const { return IntrusiveRedBlackTree, member>::find(key).release_nonnull(); } diff --git a/AK/Memory.h b/AK/Memory.h index c69e498f212..c3f233df2cb 100644 --- a/AK/Memory.h +++ b/AK/Memory.h @@ -44,7 +44,7 @@ inline void secure_zero(void* ptr, size_t size) // The memory barrier is here to avoid the compiler optimizing // away the memset when we rely on it for wiping secrets. asm volatile("" :: - : "memory"); + : "memory"); } // Naive implementation of a constant time buffer comparison function. diff --git a/AK/RedBlackTree.h b/AK/RedBlackTree.h index 54aea2c74df..5a4d2c3d484 100644 --- a/AK/RedBlackTree.h +++ b/AK/RedBlackTree.h @@ -42,7 +42,7 @@ public: Node() { } - virtual ~Node() {}; + virtual ~Node() { } }; protected: diff --git a/AK/Result.h b/AK/Result.h index bf271aaa542..7c9566ea069 100644 --- a/AK/Result.h +++ b/AK/Result.h @@ -94,8 +94,8 @@ public: ~Result() = default; // For compatibility with TRY(). - void value() {}; - void release_value() {}; + void value() { } + void release_value() { } ErrorType& error() { diff --git a/AK/StdLibExtras.h b/AK/StdLibExtras.h index d0ba945aaf8..051e82dff37 100644 --- a/AK/StdLibExtras.h +++ b/AK/StdLibExtras.h @@ -132,7 +132,7 @@ requires(IsIntegral) { if (!is_constant_evaluated()) { asm volatile("" - : "+r"(value)); + : "+r"(value)); } } @@ -142,9 +142,9 @@ requires(!IsIntegral) { if (!is_constant_evaluated()) { asm volatile("" - : - : "m"(value) - : "memory"); + : + : "m"(value) + : "memory"); } } diff --git a/Libraries/LibCompress/Brotli.h b/Libraries/LibCompress/Brotli.h index 484b1c82aa6..402c710143d 100644 --- a/Libraries/LibCompress/Brotli.h +++ b/Libraries/LibCompress/Brotli.h @@ -20,7 +20,9 @@ public: CanonicalCode() = default; CanonicalCode(Vector codes, Vector values) : m_symbol_codes(move(codes)) - , m_symbol_values(move(values)) {}; + , m_symbol_values(move(values)) + { + } static ErrorOr read_prefix_code(LittleEndianInputBitStream&, size_t alphabet_size); static ErrorOr read_simple_prefix_code(LittleEndianInputBitStream&, size_t alphabet_size); diff --git a/Libraries/LibCore/Command.cpp b/Libraries/LibCore/Command.cpp index f7120b08790..a7421e5d7eb 100644 --- a/Libraries/LibCore/Command.cpp +++ b/Libraries/LibCore/Command.cpp @@ -69,7 +69,7 @@ ErrorOr Command::write_lines(Span lines) // It's possible the process dies before we can write everything to the // stdin. So make sure that we don't crash but just stop writing. - struct sigaction action_handler { }; + struct sigaction action_handler {}; action_handler.sa_handler = SIG_IGN; struct sigaction old_action_handler; diff --git a/Libraries/LibCore/Process.cpp b/Libraries/LibCore/Process.cpp index 6944b8f87f3..ce784b3d645 100644 --- a/Libraries/LibCore/Process.cpp +++ b/Libraries/LibCore/Process.cpp @@ -286,7 +286,7 @@ ErrorOr Process::is_being_debugged() if (sysctl(mib, sizeof(mib) / sizeof(*mib), &info, &size, NULL, 0) < 0) return Error::from_syscall("sysctl"sv, -errno); - // We're being debugged if the P_TRACED flag is set. + // We're being debugged if the P_TRACED flag is set. # if defined(AK_OS_MACOS) return ((info.kp_proc.p_flag & P_TRACED) != 0); # elif defined(AK_OS_FREEBSD) diff --git a/Libraries/LibCore/ThreadedPromise.h b/Libraries/LibCore/ThreadedPromise.h index 50a539b92bc..48d0cdffa77 100644 --- a/Libraries/LibCore/ThreadedPromise.h +++ b/Libraries/LibCore/ThreadedPromise.h @@ -111,7 +111,7 @@ public: // Set the callback to be called when the promise is rejected. Setting this callback // will cause the promise fulfillment to be ready to be handled. template RejectedHandler> - ThreadedPromise& when_rejected(RejectedHandler when_rejected = [](ErrorType&) {}) + ThreadedPromise& when_rejected(RejectedHandler when_rejected = [](ErrorType&) { }) { Threading::MutexLocker locker { m_mutex }; VERIFY(!m_rejection_handler); diff --git a/Libraries/LibCrypto/BigFraction/BigFraction.cpp b/Libraries/LibCrypto/BigFraction/BigFraction.cpp index 7b6b712d5f1..d51c64bf758 100644 --- a/Libraries/LibCrypto/BigFraction/BigFraction.cpp +++ b/Libraries/LibCrypto/BigFraction/BigFraction.cpp @@ -199,8 +199,8 @@ String BigFraction::to_string(unsigned rounding_threshold) const auto const number_of_digits = [](auto integer) { unsigned size = 1; for (auto division_result = integer.divided_by(UnsignedBigInteger { 10 }); - division_result.remainder == UnsignedBigInteger { 0 } && division_result.quotient != UnsignedBigInteger { 0 }; - division_result = division_result.quotient.divided_by(UnsignedBigInteger { 10 })) { + division_result.remainder == UnsignedBigInteger { 0 } && division_result.quotient != UnsignedBigInteger { 0 }; + division_result = division_result.quotient.divided_by(UnsignedBigInteger { 10 })) { ++size; } return size; diff --git a/Libraries/LibFileSystem/FileSystem.cpp b/Libraries/LibFileSystem/FileSystem.cpp index 86fdef82ec4..243087e942c 100644 --- a/Libraries/LibFileSystem/FileSystem.cpp +++ b/Libraries/LibFileSystem/FileSystem.cpp @@ -316,7 +316,7 @@ bool can_delete_or_move(StringView path) auto stat_or_empty = [](StringView path) { auto stat_or_error = Core::System::stat(path); if (stat_or_error.is_error()) { - struct stat stat { }; + struct stat stat {}; return stat; } return stat_or_error.release_value(); diff --git a/Libraries/LibGfx/Font/Typeface.cpp b/Libraries/LibGfx/Font/Typeface.cpp index ede5ab751f0..db36016fad6 100644 --- a/Libraries/LibGfx/Font/Typeface.cpp +++ b/Libraries/LibGfx/Font/Typeface.cpp @@ -60,7 +60,7 @@ NonnullRefPtr Typeface::scaled_font(float point_size) const hb_face_t* Typeface::harfbuzz_typeface() const { if (!m_harfbuzz_blob) - m_harfbuzz_blob = hb_blob_create(reinterpret_cast(buffer().data()), buffer().size(), HB_MEMORY_MODE_READONLY, nullptr, [](void*) {}); + m_harfbuzz_blob = hb_blob_create(reinterpret_cast(buffer().data()), buffer().size(), HB_MEMORY_MODE_READONLY, nullptr, [](void*) { }); if (!m_harfbuzz_face) m_harfbuzz_face = hb_face_create(m_harfbuzz_blob, ttc_index()); return m_harfbuzz_face; diff --git a/Libraries/LibGfx/Font/TypefaceSkia.cpp b/Libraries/LibGfx/Font/TypefaceSkia.cpp index 912f4f9f634..17c6a502cdf 100644 --- a/Libraries/LibGfx/Font/TypefaceSkia.cpp +++ b/Libraries/LibGfx/Font/TypefaceSkia.cpp @@ -64,9 +64,9 @@ SkTypeface const* TypefaceSkia::sk_typeface() const TypefaceSkia::TypefaceSkia(NonnullOwnPtr impl, ReadonlyBytes buffer, int ttc_index) : m_impl(move(impl)) , m_buffer(buffer) - , m_ttc_index(ttc_index) { - - }; + , m_ttc_index(ttc_index) +{ +} u32 TypefaceSkia::glyph_count() const { diff --git a/Libraries/LibGfx/ImageFormats/ImageDecoder.h b/Libraries/LibGfx/ImageFormats/ImageDecoder.h index af7a0cc9070..8b6e427049d 100644 --- a/Libraries/LibGfx/ImageFormats/ImageDecoder.h +++ b/Libraries/LibGfx/ImageFormats/ImageDecoder.h @@ -48,7 +48,7 @@ public: } protected: - virtual void fill_main_tags() const {}; + virtual void fill_main_tags() const { } mutable HashMap m_main_tags; }; diff --git a/Libraries/LibGfx/ImageFormats/JPEGLoader.cpp b/Libraries/LibGfx/ImageFormats/JPEGLoader.cpp index 402695fa121..96a86895640 100644 --- a/Libraries/LibGfx/ImageFormats/JPEGLoader.cpp +++ b/Libraries/LibGfx/ImageFormats/JPEGLoader.cpp @@ -62,7 +62,7 @@ ErrorOr JPEGLoadingContext::decode() source_manager.next_input_byte = data.data(); source_manager.bytes_in_buffer = data.size(); - source_manager.init_source = [](j_decompress_ptr) {}; + source_manager.init_source = [](j_decompress_ptr) { }; source_manager.fill_input_buffer = [](j_decompress_ptr) -> boolean { return false; }; source_manager.skip_input_data = [](j_decompress_ptr context, long num_bytes) { if (num_bytes > static_cast(context->src->bytes_in_buffer)) { @@ -73,7 +73,7 @@ ErrorOr JPEGLoadingContext::decode() context->src->bytes_in_buffer -= num_bytes; }; source_manager.resync_to_restart = jpeg_resync_to_restart; - source_manager.term_source = [](j_decompress_ptr) {}; + source_manager.term_source = [](j_decompress_ptr) { }; cinfo.src = &source_manager; diff --git a/Libraries/LibGfx/ImageFormats/JPEGWriter.cpp b/Libraries/LibGfx/ImageFormats/JPEGWriter.cpp index 8f4e9b43d94..1cadaf376b1 100644 --- a/Libraries/LibGfx/ImageFormats/JPEGWriter.cpp +++ b/Libraries/LibGfx/ImageFormats/JPEGWriter.cpp @@ -48,8 +48,8 @@ struct MemoryDestinationManager : public jpeg_destination_mgr { ErrorOr JPEGWriter::encode_impl(Stream& stream, auto const& bitmap, Options const& options, ColorSpace color_space) { - struct jpeg_compress_struct cinfo { }; - struct jpeg_error_mgr jerr { }; + struct jpeg_compress_struct cinfo {}; + struct jpeg_error_mgr jerr {}; cinfo.err = jpeg_std_error(&jerr); diff --git a/Libraries/LibGfx/ImageFormats/WebPSharedLossless.h b/Libraries/LibGfx/ImageFormats/WebPSharedLossless.h index 43b466c2d0d..25ec33f0ca7 100644 --- a/Libraries/LibGfx/ImageFormats/WebPSharedLossless.h +++ b/Libraries/LibGfx/ImageFormats/WebPSharedLossless.h @@ -41,7 +41,7 @@ private: ALWAYS_INLINE ErrorOr CanonicalCode::write_symbol(LittleEndianOutputBitStream& bit_stream, u32 symbol) const { TRY(m_code.visit( - [&](u32 single_code) __attribute__((always_inline))->ErrorOr { VERIFY(symbol == single_code); return {}; }, + [&](u32 single_code) __attribute__((always_inline)) -> ErrorOr { VERIFY(symbol == single_code); return {}; }, [&](Compress::CanonicalCode const& code) __attribute__((always_inline)) { return code.write_symbol(bit_stream, symbol); })); return {}; } diff --git a/Libraries/LibGfx/MetalContext.h b/Libraries/LibGfx/MetalContext.h index 31ebac57bf1..aefaae6d5e4 100644 --- a/Libraries/LibGfx/MetalContext.h +++ b/Libraries/LibGfx/MetalContext.h @@ -23,7 +23,7 @@ public: virtual size_t width() const = 0; virtual size_t height() const = 0; - virtual ~MetalTexture() {}; + virtual ~MetalTexture() { } }; class MetalContext : public RefCounted { @@ -33,7 +33,7 @@ public: virtual OwnPtr create_texture_from_iosurface(Core::IOSurfaceHandle const&) = 0; - virtual ~MetalContext() {}; + virtual ~MetalContext() { } }; RefPtr get_metal_context(); diff --git a/Libraries/LibGfx/SkiaBackendContext.h b/Libraries/LibGfx/SkiaBackendContext.h index e78af755838..c60cd99868b 100644 --- a/Libraries/LibGfx/SkiaBackendContext.h +++ b/Libraries/LibGfx/SkiaBackendContext.h @@ -37,10 +37,10 @@ public: static RefPtr create_metal_context(MetalContext&); #endif - SkiaBackendContext() {}; - virtual ~SkiaBackendContext() {}; + SkiaBackendContext() { } + virtual ~SkiaBackendContext() { } - virtual void flush_and_submit(SkSurface*) {}; + virtual void flush_and_submit(SkSurface*) { } virtual GrDirectContext* sk_context() const = 0; virtual MetalContext& metal_context() = 0; diff --git a/Libraries/LibJS/AST.h b/Libraries/LibJS/AST.h index cd3fe0b6c15..09845cb36b8 100644 --- a/Libraries/LibJS/AST.h +++ b/Libraries/LibJS/AST.h @@ -719,7 +719,7 @@ public: virtual bool has_name() const = 0; virtual Value instantiate_ordinary_function_expression(VM&, DeprecatedFlyString given_name) const = 0; - virtual ~FunctionNode() {}; + virtual ~FunctionNode() { } protected: FunctionNode(RefPtr name, ByteString source_text, NonnullRefPtr body, Vector parameters, i32 function_length, FunctionKind kind, bool is_strict_mode, FunctionParsingInsights parsing_insights, bool is_arrow_function, Vector local_variables_names) @@ -779,7 +779,7 @@ public: bool has_name() const override { return true; } Value instantiate_ordinary_function_expression(VM&, DeprecatedFlyString) const override { VERIFY_NOT_REACHED(); } - virtual ~FunctionDeclaration() {}; + virtual ~FunctionDeclaration() { } private: bool m_is_hoisted { false }; @@ -806,7 +806,7 @@ public: Value instantiate_ordinary_function_expression(VM&, DeprecatedFlyString given_name) const override; - virtual ~FunctionExpression() {}; + virtual ~FunctionExpression() { } private: virtual bool is_function_expression() const override { return true; } diff --git a/Libraries/LibJS/Runtime/ErrorConstructor.cpp b/Libraries/LibJS/Runtime/ErrorConstructor.cpp index 5f382848926..1e83b6c9c2e 100644 --- a/Libraries/LibJS/Runtime/ErrorConstructor.cpp +++ b/Libraries/LibJS/Runtime/ErrorConstructor.cpp @@ -101,7 +101,7 @@ ThrowCompletionOr> ErrorConstructor::construct(FunctionObject& n auto message = vm.argument(0); \ auto options = vm.argument(1); \ \ - /* 2. Let O be ? OrdinaryCreateFromConstructor(newTarget, "%NativeError.prototype%", « [[ErrorData]] »). */ \ + /* 2. Let O be ? OrdinaryCreateFromConstructor(newTarget, "%NativeError.prototype%", « [[ErrorData]] »). */ \ auto error = TRY(ordinary_create_from_constructor(vm, new_target, &Intrinsics::snake_name##_prototype)); \ \ /* 3. If message is not undefined, then */ \ diff --git a/Libraries/LibJS/Runtime/IndexedProperties.h b/Libraries/LibJS/Runtime/IndexedProperties.h index 5f01283338c..4d6b5eaf370 100644 --- a/Libraries/LibJS/Runtime/IndexedProperties.h +++ b/Libraries/LibJS/Runtime/IndexedProperties.h @@ -48,7 +48,9 @@ public: protected: explicit IndexedPropertyStorage(IsSimpleStorage is_simple_storage) - : m_is_simple_storage(is_simple_storage == IsSimpleStorage::Yes) {}; + : m_is_simple_storage(is_simple_storage == IsSimpleStorage::Yes) + { + } private: bool m_is_simple_storage { false }; @@ -57,7 +59,9 @@ private: class SimpleIndexedPropertyStorage final : public IndexedPropertyStorage { public: SimpleIndexedPropertyStorage() - : IndexedPropertyStorage(IsSimpleStorage::Yes) {}; + : IndexedPropertyStorage(IsSimpleStorage::Yes) + { + } explicit SimpleIndexedPropertyStorage(Vector&& initial_values); virtual bool has_index(u32 index) const override; @@ -99,7 +103,9 @@ class GenericIndexedPropertyStorage final : public IndexedPropertyStorage { public: explicit GenericIndexedPropertyStorage(SimpleIndexedPropertyStorage&&); explicit GenericIndexedPropertyStorage() - : IndexedPropertyStorage(IsSimpleStorage::No) {}; + : IndexedPropertyStorage(IsSimpleStorage::No) + { + } virtual bool has_index(u32 index) const override; virtual Optional get(u32 index) const override; diff --git a/Libraries/LibJS/Runtime/Intl/DurationFormat.h b/Libraries/LibJS/Runtime/Intl/DurationFormat.h index 7e564c224dc..049e314d675 100644 --- a/Libraries/LibJS/Runtime/Intl/DurationFormat.h +++ b/Libraries/LibJS/Runtime/Intl/DurationFormat.h @@ -221,7 +221,7 @@ struct DurationRecord { }; struct DurationInstanceComponent { - double DurationRecord::*value_slot; + double DurationRecord::* value_slot; DurationFormat::ValueStyle (DurationFormat::*get_style_slot)() const; void (DurationFormat::*set_style_slot)(DurationFormat::ValueStyle); DurationFormat::Display (DurationFormat::*get_display_slot)() const; diff --git a/Libraries/LibJS/Runtime/PromiseCapability.h b/Libraries/LibJS/Runtime/PromiseCapability.h index 8eb5d3cc959..04151a04fc4 100644 --- a/Libraries/LibJS/Runtime/PromiseCapability.h +++ b/Libraries/LibJS/Runtime/PromiseCapability.h @@ -44,7 +44,7 @@ private: auto&& _temporary_try_or_reject_result = (expression); \ /* 1. If value is an abrupt completion, then */ \ if (_temporary_try_or_reject_result.is_error()) { \ - /* a. Perform ? Call(capability.[[Reject]], undefined, « value.[[Value]] »). */ \ + /* a. Perform ? Call(capability.[[Reject]], undefined, « value.[[Value]] »). */ \ CALL_CHECK(JS::call(vm, *(capability)->reject(), js_undefined(), *_temporary_try_or_reject_result.release_error().value())); \ \ /* b. Return capability.[[Promise]]. */ \ @@ -70,7 +70,7 @@ private: auto&& _temporary_try_or_reject_result = (expression); \ /* 1. If value is an abrupt completion, then */ \ if (_temporary_try_or_reject_result.is_error()) { \ - /* a. Perform ? Call(capability.[[Reject]], undefined, « value.[[Value]] »). */ \ + /* a. Perform ? Call(capability.[[Reject]], undefined, « value.[[Value]] »). */ \ TRY(JS::call(vm, *(capability)->reject(), js_undefined(), *_temporary_try_or_reject_result.release_error().value())); \ \ /* b. Return capability.[[Promise]]. */ \ diff --git a/Libraries/LibJS/Runtime/Temporal/DurationPrototype.cpp b/Libraries/LibJS/Runtime/Temporal/DurationPrototype.cpp index c0d5c274a54..33d0ffdcc49 100644 --- a/Libraries/LibJS/Runtime/Temporal/DurationPrototype.cpp +++ b/Libraries/LibJS/Runtime/Temporal/DurationPrototype.cpp @@ -71,7 +71,7 @@ void DurationPrototype::initialize(Realm& realm) /* 2. Perform ? RequireInternalSlot(duration, [[InitializedTemporalDuration]]). */ \ auto duration = TRY(typed_this_object(vm)); \ \ - /* 3. Return 𝔽(duration.[[]]). */ \ + /* 3. Return 𝔽(duration.[[]]). */ \ return duration->unit(); \ } JS_ENUMERATE_DURATION_UNITS diff --git a/Libraries/LibJS/Runtime/Temporal/PlainDateTimePrototype.cpp b/Libraries/LibJS/Runtime/Temporal/PlainDateTimePrototype.cpp index e4b7bc5bb9a..7588099574e 100644 --- a/Libraries/LibJS/Runtime/Temporal/PlainDateTimePrototype.cpp +++ b/Libraries/LibJS/Runtime/Temporal/PlainDateTimePrototype.cpp @@ -145,15 +145,15 @@ JS_DEFINE_NATIVE_FUNCTION(PlainDateTimePrototype::era_year_getter) __JS_ENUMERATE(months_in_year) \ __JS_ENUMERATE(in_leap_year) -#define __JS_ENUMERATE(field) \ - JS_DEFINE_NATIVE_FUNCTION(PlainDateTimePrototype::field##_getter) \ - { \ - /* 1. Let dateTime be the this value. */ \ - /* 2. Perform ? RequireInternalSlot(dateTime, [[InitializedTemporalDateTime]]). */ \ - auto date_time = TRY(typed_this_object(vm)); \ - \ +#define __JS_ENUMERATE(field) \ + JS_DEFINE_NATIVE_FUNCTION(PlainDateTimePrototype::field##_getter) \ + { \ + /* 1. Let dateTime be the this value. */ \ + /* 2. Perform ? RequireInternalSlot(dateTime, [[InitializedTemporalDateTime]]). */ \ + auto date_time = TRY(typed_this_object(vm)); \ + \ /* 3. Return 𝔽(CalendarISOToDate(dateTime.[[Calendar]], dateTime.[[ISODateTime]].[[ISODate]]).[[]]). */ \ - return calendar_iso_to_date(date_time->calendar(), date_time->iso_date_time().iso_date).field; \ + return calendar_iso_to_date(date_time->calendar(), date_time->iso_date_time().iso_date).field; \ } JS_ENUMERATE_PLAIN_DATE_TIME_SIMPLE_DATE_FIELDS #undef __JS_ENUMERATE @@ -191,7 +191,7 @@ JS_DEFINE_NATIVE_FUNCTION(PlainDateTimePrototype::month_code_getter) /* 2. Perform ? RequireInternalSlot(dateTime, [[InitializedTemporalDateTime]]). */ \ auto date_time = TRY(typed_this_object(vm)); \ \ - /* 3. Return 𝔽(dateTime.[[ISODateTime]].[[Time]].[[]]). */ \ + /* 3. Return 𝔽(dateTime.[[ISODateTime]].[[Time]].[[]]). */ \ return date_time->iso_date_time().time.field; \ } JS_ENUMERATE_PLAIN_DATE_TIME_TIME_FIELDS diff --git a/Libraries/LibJS/Runtime/Temporal/PlainTimePrototype.cpp b/Libraries/LibJS/Runtime/Temporal/PlainTimePrototype.cpp index 715e484c1dd..e2e245c30d1 100644 --- a/Libraries/LibJS/Runtime/Temporal/PlainTimePrototype.cpp +++ b/Libraries/LibJS/Runtime/Temporal/PlainTimePrototype.cpp @@ -72,7 +72,7 @@ void PlainTimePrototype::initialize(Realm& realm) /* 2. Perform ? RequireInternalSlot(temporalTime, [[InitializedTemporalTime]]). */ \ auto temporal_time = TRY(typed_this_object(vm)); \ \ - /* 3. Return 𝔽(temporalTime.[[Time]].[[]]). */ \ + /* 3. Return 𝔽(temporalTime.[[Time]].[[]]). */ \ return temporal_time->time().field; \ } JS_ENUMERATE_PLAIN_TIME_FIELDS diff --git a/Libraries/LibJS/Runtime/Temporal/ZonedDateTimePrototype.cpp b/Libraries/LibJS/Runtime/Temporal/ZonedDateTimePrototype.cpp index b8a6ef8266b..e4ffe69f4ce 100644 --- a/Libraries/LibJS/Runtime/Temporal/ZonedDateTimePrototype.cpp +++ b/Libraries/LibJS/Runtime/Temporal/ZonedDateTimePrototype.cpp @@ -186,7 +186,7 @@ JS_DEFINE_NATIVE_FUNCTION(ZonedDateTimePrototype::era_year_getter) /* Let isoDateTime be GetISODateTimeFor(zonedDateTime.[[TimeZone]], zonedDateTime.[[EpochNanoseconds]]). */ \ auto iso_date_time = get_iso_date_time_for(zoned_date_time->time_zone(), zoned_date_time->epoch_nanoseconds()->big_integer()); \ \ - /* 3. Return 𝔽(CalendarISOToDate(zonedDateTime.[[Calendar]], isoDateTime.[[ISODate]]).[[]]). */ \ + /* 3. Return 𝔽(CalendarISOToDate(zonedDateTime.[[Calendar]], isoDateTime.[[ISODate]]).[[]]). */ \ return calendar_iso_to_date(zoned_date_time->calendar(), iso_date_time.iso_date).field; \ } JS_ENUMERATE_ZONED_DATE_TIME_SIMPLE_DATE_FIELDS @@ -231,7 +231,7 @@ JS_DEFINE_NATIVE_FUNCTION(ZonedDateTimePrototype::month_code_getter) /* Let isoDateTime be GetISODateTimeFor(zonedDateTime.[[TimeZone]], zonedDateTime.[[EpochNanoseconds]]). */ \ auto iso_date_time = get_iso_date_time_for(zoned_date_time->time_zone(), zoned_date_time->epoch_nanoseconds()->big_integer()); \ \ - /* 3. Return 𝔽(isoDateTime.[[Time]].[[]]). */ \ + /* 3. Return 𝔽(isoDateTime.[[Time]].[[]]). */ \ return iso_date_time.time.field; \ } JS_ENUMERATE_PLAIN_DATE_TIME_TIME_FIELDS diff --git a/Libraries/LibJS/Runtime/Value.cpp b/Libraries/LibJS/Runtime/Value.cpp index 22ac7e74331..8313f41b183 100644 --- a/Libraries/LibJS/Runtime/Value.cpp +++ b/Libraries/LibJS/Runtime/Value.cpp @@ -2380,8 +2380,8 @@ ThrowCompletionOr is_less_than(VM& vm, Value lhs, Value rhs, bool left // b. Let ly be the length of py. // c. For each integer i such that 0 ≤ i < min(lx, ly), in ascending order, do for (auto k = x_code_points.begin(), l = y_code_points.begin(); - k != x_code_points.end() && l != y_code_points.end(); - ++k, ++l) { + k != x_code_points.end() && l != y_code_points.end(); + ++k, ++l) { // i. Let cx be the integer that is the numeric value of the code unit at index i within px. // ii. Let cy be the integer that is the numeric value of the code unit at index i within py. if (*k != *l) { diff --git a/Libraries/LibJS/SourceTextModule.cpp b/Libraries/LibJS/SourceTextModule.cpp index 956b584ccc9..f49650edc17 100644 --- a/Libraries/LibJS/SourceTextModule.cpp +++ b/Libraries/LibJS/SourceTextModule.cpp @@ -176,7 +176,7 @@ Result, Vector> SourceTextModule::parse(S [&](ImportEntry const& import_entry) { return import_entry.local_name == entry.local_or_import_name; }) - .is_end()); + .is_end()); default_export = export_statement; } diff --git a/Libraries/LibLine/Editor.h b/Libraries/LibLine/Editor.h index 0320590eebd..cf0c752c7da 100644 --- a/Libraries/LibLine/Editor.h +++ b/Libraries/LibLine/Editor.h @@ -468,10 +468,8 @@ private: KeyCallbackMachine m_callback_machine; - struct termios m_termios { - }; - struct termios m_default_termios { - }; + struct termios m_termios {}; + struct termios m_default_termios {}; bool m_was_interrupted { false }; bool m_previous_interrupt_was_handled_as_interrupt { true }; bool m_was_resized { false }; diff --git a/Libraries/LibLine/KeyCallbackMachine.h b/Libraries/LibLine/KeyCallbackMachine.h index a18814b0197..fbc9a55807e 100644 --- a/Libraries/LibLine/KeyCallbackMachine.h +++ b/Libraries/LibLine/KeyCallbackMachine.h @@ -25,7 +25,9 @@ struct Key { Key(unsigned c) : modifiers(None) - , key(c) {}; + , key(c) + { + } Key(unsigned c, int modifiers) : modifiers(modifiers) diff --git a/Libraries/LibMedia/VideoDecoder.h b/Libraries/LibMedia/VideoDecoder.h index b7212439ffc..3bc60dfa8d3 100644 --- a/Libraries/LibMedia/VideoDecoder.h +++ b/Libraries/LibMedia/VideoDecoder.h @@ -16,7 +16,7 @@ namespace Media { class VideoDecoder { public: - virtual ~VideoDecoder() {}; + virtual ~VideoDecoder() { } virtual DecoderErrorOr receive_sample(AK::Duration timestamp, ReadonlyBytes sample) = 0; DecoderErrorOr receive_sample(AK::Duration timestamp, ByteBuffer const& sample) { return receive_sample(timestamp, sample.span()); } diff --git a/Libraries/LibTLS/TLSv12.h b/Libraries/LibTLS/TLSv12.h index f6cb127da75..74c02dea95c 100644 --- a/Libraries/LibTLS/TLSv12.h +++ b/Libraries/LibTLS/TLSv12.h @@ -194,8 +194,8 @@ struct Options { OPTION_WITH_DEFAULTS(bool, validate_certificates, true) OPTION_WITH_DEFAULTS(bool, allow_self_signed_certificates, false) OPTION_WITH_DEFAULTS(Optional>, root_certificates, ) - OPTION_WITH_DEFAULTS(Function, alert_handler, [](auto) {}) - OPTION_WITH_DEFAULTS(Function, finish_callback, [] {}) + OPTION_WITH_DEFAULTS(Function, alert_handler, [](auto) { }) + OPTION_WITH_DEFAULTS(Function, finish_callback, [] { }) OPTION_WITH_DEFAULTS(Function()>, certificate_provider, [] { return Vector {}; }) OPTION_WITH_DEFAULTS(bool, enable_extended_master_secret, true) diff --git a/Libraries/LibWeb/CSS/ColumnCount.h b/Libraries/LibWeb/CSS/ColumnCount.h index 4723bae035d..954cffe5a0e 100644 --- a/Libraries/LibWeb/CSS/ColumnCount.h +++ b/Libraries/LibWeb/CSS/ColumnCount.h @@ -34,7 +34,7 @@ private: , m_value(value) { } - ColumnCount() {}; + ColumnCount() { } Type m_type { Type::Auto }; Optional m_value; diff --git a/Libraries/LibWeb/CSS/Filter.h b/Libraries/LibWeb/CSS/Filter.h index 8fec9c5154f..7badbea8131 100644 --- a/Libraries/LibWeb/CSS/Filter.h +++ b/Libraries/LibWeb/CSS/Filter.h @@ -16,7 +16,9 @@ class Filter { public: Filter() = default; Filter(FilterValueListStyleValue const& filter_value_list) - : m_filter_value_list { filter_value_list } {}; + : m_filter_value_list { filter_value_list } + { + } static Filter make_none() { diff --git a/Libraries/LibWeb/CSS/GridTrackPlacement.h b/Libraries/LibWeb/CSS/GridTrackPlacement.h index 025269dd843..a6d3c966bf2 100644 --- a/Libraries/LibWeb/CSS/GridTrackPlacement.h +++ b/Libraries/LibWeb/CSS/GridTrackPlacement.h @@ -71,11 +71,17 @@ private: }; GridTrackPlacement() - : m_value(Auto {}) {}; + : m_value(Auto {}) + { + } GridTrackPlacement(AreaOrLine value) - : m_value(value) {}; + : m_value(value) + { + } GridTrackPlacement(Span value) - : m_value(value) {}; + : m_value(value) + { + } Variant m_value; }; diff --git a/Libraries/LibWeb/DOM/ElementFactory.cpp b/Libraries/LibWeb/DOM/ElementFactory.cpp index 88d2cd148e5..7cb9b79f692 100644 --- a/Libraries/LibWeb/DOM/ElementFactory.cpp +++ b/Libraries/LibWeb/DOM/ElementFactory.cpp @@ -263,10 +263,10 @@ bool is_unknown_html_element(FlyString const& tag_name) if (tag_name.is_one_of(HTML::TagNames::applet, HTML::TagNames::bgsound, HTML::TagNames::blink, HTML::TagNames::isindex, HTML::TagNames::keygen, HTML::TagNames::multicol, HTML::TagNames::nextid, HTML::TagNames::spacer)) return true; - // 2. If name is acronym, basefont, big, center, nobr, noembed, noframes, plaintext, rb, rtc, strike, or tt, then return HTMLElement. - // 3. If name is listing or xmp, then return HTMLPreElement. - // 4. Otherwise, if this specification defines an interface appropriate for the element type corresponding to the local name name, then return that interface. - // 5. If other applicable specifications define an appropriate interface for name, then return the interface they define. + // 2. If name is acronym, basefont, big, center, nobr, noembed, noframes, plaintext, rb, rtc, strike, or tt, then return HTMLElement. + // 3. If name is listing or xmp, then return HTMLPreElement. + // 4. Otherwise, if this specification defines an interface appropriate for the element type corresponding to the local name name, then return that interface. + // 5. If other applicable specifications define an appropriate interface for name, then return the interface they define. #define __ENUMERATE_HTML_TAG(name) \ if (tag_name == HTML::TagNames::name) \ return false; diff --git a/Libraries/LibWeb/DOMURL/URLSearchParams.cpp b/Libraries/LibWeb/DOMURL/URLSearchParams.cpp index d2d8ce07ed7..2a9dc7d7475 100644 --- a/Libraries/LibWeb/DOMURL/URLSearchParams.cpp +++ b/Libraries/LibWeb/DOMURL/URLSearchParams.cpp @@ -286,7 +286,7 @@ bool URLSearchParams::has(String const& name, Optional const& value) if (!m_list.find_if([&name, &value](auto& entry) { return entry.name == name && entry.value == value.value(); }) - .is_end()) { + .is_end()) { return true; } } @@ -295,7 +295,7 @@ bool URLSearchParams::has(String const& name, Optional const& value) if (!m_list.find_if([&name](auto& entry) { return entry.name == name; }) - .is_end()) { + .is_end()) { return true; } } diff --git a/Libraries/LibWeb/Editing/Commands.cpp b/Libraries/LibWeb/Editing/Commands.cpp index b031f42362b..c7b600bc490 100644 --- a/Libraries/LibWeb/Editing/Commands.cpp +++ b/Libraries/LibWeb/Editing/Commands.cpp @@ -600,8 +600,8 @@ bool command_insert_paragraph_action(DOM::Document& document, String const&) // 12. If container's local name is "address", "listing", or "pre": if (is(*container) && static_cast(*container) - .local_name() - .is_one_of(HTML::TagNames::address, HTML::TagNames::listing, HTML::TagNames::pre)) { + .local_name() + .is_one_of(HTML::TagNames::address, HTML::TagNames::listing, HTML::TagNames::pre)) { // 1. Let br be the result of calling createElement("br") on the context object. auto br = MUST(DOM::create_element(document, HTML::TagNames::br, Namespace::HTML)); diff --git a/Libraries/LibWeb/HTML/HTMLMediaElement.cpp b/Libraries/LibWeb/HTML/HTMLMediaElement.cpp index 3769bc2b0cf..11c59712e7d 100644 --- a/Libraries/LibWeb/HTML/HTMLMediaElement.cpp +++ b/Libraries/LibWeb/HTML/HTMLMediaElement.cpp @@ -1042,7 +1042,7 @@ WebIDL::ExceptionOr HTMLMediaElement::fetch_resource(URL::URL const& url_r // 5. Otherwise, incrementally read response's body given updateMedia, processEndOfMedia, an empty algorithm, and global. VERIFY(response->body()); - auto empty_algorithm = GC::create_function(heap(), [](JS::Value) {}); + auto empty_algorithm = GC::create_function(heap(), [](JS::Value) { }); // FIXME: We are "fully" reading the response here, rather than "incrementally". Memory concerns aside, this should be okay for now as we are // always setting byteRange to "entire resource". However, we should switch to incremental reads when that is implemented, and then diff --git a/Libraries/LibWeb/HTML/HTMLVideoElement.cpp b/Libraries/LibWeb/HTML/HTMLVideoElement.cpp index 0eff6090c5b..b96f7e16ee2 100644 --- a/Libraries/LibWeb/HTML/HTMLVideoElement.cpp +++ b/Libraries/LibWeb/HTML/HTMLVideoElement.cpp @@ -211,7 +211,7 @@ WebIDL::ExceptionOr HTMLVideoElement::determine_element_poster_frame(Optio }); VERIFY(response->body()); - auto empty_algorithm = GC::create_function(heap(), [](JS::Value) {}); + auto empty_algorithm = GC::create_function(heap(), [](JS::Value) { }); response->body()->fully_read(realm, on_image_data_read, empty_algorithm, GC::Ref { global }); }; diff --git a/Libraries/LibWeb/HTML/TraversableNavigable.cpp b/Libraries/LibWeb/HTML/TraversableNavigable.cpp index a9db18b6e63..12bb33a080a 100644 --- a/Libraries/LibWeb/HTML/TraversableNavigable.cpp +++ b/Libraries/LibWeb/HTML/TraversableNavigable.cpp @@ -688,8 +688,8 @@ TraversableNavigable::HistoryStepResult TraversableNavigable::apply_the_history_ // that is synchronous navigation steps with a target navigable not contained in navigablesThatMustWaitBeforeHandlingSyncNavigation. // 2. Remove steps from traversable's session history traversal queue's algorithm set. for (auto entry = m_session_history_traversal_queue->first_synchronous_navigation_steps_with_target_navigable_not_contained_in(navigables_that_must_wait_before_handling_sync_navigation); - entry; - entry = m_session_history_traversal_queue->first_synchronous_navigation_steps_with_target_navigable_not_contained_in(navigables_that_must_wait_before_handling_sync_navigation)) { + entry; + entry = m_session_history_traversal_queue->first_synchronous_navigation_steps_with_target_navigable_not_contained_in(navigables_that_must_wait_before_handling_sync_navigation)) { // 3. Set traversable's running nested apply history step to true. m_running_nested_apply_history_step = true; diff --git a/Libraries/LibWeb/Layout/SVGFormattingContext.cpp b/Libraries/LibWeb/Layout/SVGFormattingContext.cpp index 66abeec1b8c..6e002e5bae9 100644 --- a/Libraries/LibWeb/Layout/SVGFormattingContext.cpp +++ b/Libraries/LibWeb/Layout/SVGFormattingContext.cpp @@ -226,9 +226,9 @@ void SVGFormattingContext::run(AvailableSpace const& available_space) CSSPixelPoint offset = viewbox_offset_and_scale.offset; m_current_viewbox_transform = Gfx::AffineTransform { m_current_viewbox_transform }.multiply(Gfx::AffineTransform {} - .translate(offset.to_type()) - .scale(viewbox_offset_and_scale.scale_factor_x, viewbox_offset_and_scale.scale_factor_y) - .translate({ -viewbox->min_x, -viewbox->min_y })); + .translate(offset.to_type()) + .scale(viewbox_offset_and_scale.scale_factor_x, viewbox_offset_and_scale.scale_factor_y) + .translate({ -viewbox->min_x, -viewbox->min_y })); } if (svg_box_state.has_definite_width() && svg_box_state.has_definite_height()) { diff --git a/Libraries/LibWeb/Painting/BackingStore.h b/Libraries/LibWeb/Painting/BackingStore.h index ecd92d6a0f2..215015d6c7b 100644 --- a/Libraries/LibWeb/Painting/BackingStore.h +++ b/Libraries/LibWeb/Painting/BackingStore.h @@ -23,8 +23,8 @@ public: virtual Gfx::IntSize size() const = 0; virtual Gfx::Bitmap& bitmap() const = 0; - BackingStore() {}; - virtual ~BackingStore() {}; + BackingStore() { } + virtual ~BackingStore() { } }; class BitmapBackingStore final : public BackingStore { diff --git a/Libraries/LibWeb/Painting/PaintStyle.h b/Libraries/LibWeb/Painting/PaintStyle.h index 188a86af6e7..5cf1d3816b3 100644 --- a/Libraries/LibWeb/Painting/PaintStyle.h +++ b/Libraries/LibWeb/Painting/PaintStyle.h @@ -47,7 +47,7 @@ public: ReadonlySpan color_stops() const { return m_color_stops; } Optional repeat_length() const { return m_repeat_length; } - virtual ~SVGGradientPaintStyle() {}; + virtual ~SVGGradientPaintStyle() { } protected: Vector m_color_stops; diff --git a/Libraries/LibWeb/SVG/SVGAnimatedTransformList.cpp b/Libraries/LibWeb/SVG/SVGAnimatedTransformList.cpp index 89a07fd9c49..356e141c1c8 100644 --- a/Libraries/LibWeb/SVG/SVGAnimatedTransformList.cpp +++ b/Libraries/LibWeb/SVG/SVGAnimatedTransformList.cpp @@ -20,7 +20,9 @@ GC::Ref SVGAnimatedTransformList::create(JS::Realm& re SVGAnimatedTransformList::SVGAnimatedTransformList(JS::Realm& realm, GC::Ref base_val, GC::Ref anim_val) : PlatformObject(realm) , m_base_val(base_val) - , m_anim_val(anim_val) {}; + , m_anim_val(anim_val) +{ +} SVGAnimatedTransformList::~SVGAnimatedTransformList() = default; diff --git a/Libraries/LibWeb/SVG/SVGTransform.cpp b/Libraries/LibWeb/SVG/SVGTransform.cpp index c2580a4ea94..d84d8eca099 100644 --- a/Libraries/LibWeb/SVG/SVGTransform.cpp +++ b/Libraries/LibWeb/SVG/SVGTransform.cpp @@ -19,7 +19,9 @@ GC::Ref SVGTransform::create(JS::Realm& realm) } SVGTransform::SVGTransform(JS::Realm& realm) - : PlatformObject(realm) {}; + : PlatformObject(realm) +{ +} SVGTransform::~SVGTransform() = default; diff --git a/Libraries/LibWeb/XHR/XMLHttpRequestEventTarget.h b/Libraries/LibWeb/XHR/XMLHttpRequestEventTarget.h index 67aceed7f14..0cab035f6a8 100644 --- a/Libraries/LibWeb/XHR/XMLHttpRequestEventTarget.h +++ b/Libraries/LibWeb/XHR/XMLHttpRequestEventTarget.h @@ -24,7 +24,7 @@ class XMLHttpRequestEventTarget : public DOM::EventTarget { WEB_PLATFORM_OBJECT(XMLHttpRequestEventTarget, DOM::EventTarget); public: - virtual ~XMLHttpRequestEventTarget() override {}; + virtual ~XMLHttpRequestEventTarget() override { } #undef __ENUMERATE #define __ENUMERATE(attribute_name, event_name) \ diff --git a/Libraries/LibWebView/EventLoop/EventLoopImplementationMacOS.mm b/Libraries/LibWebView/EventLoop/EventLoopImplementationMacOS.mm index 8652d2c5267..a4ceea854cd 100644 --- a/Libraries/LibWebView/EventLoop/EventLoopImplementationMacOS.mm +++ b/Libraries/LibWebView/EventLoop/EventLoopImplementationMacOS.mm @@ -92,7 +92,7 @@ public: SignalHandlers::SignalHandlers(int signal_number, CFFileDescriptorCallBack handle_signal) : m_signal_number(signal_number) - , m_original_handler(signal(signal_number, [](int) {})) + , m_original_handler(signal(signal_number, [](int) { })) { m_kevent_fd = kqueue(); if (m_kevent_fd < 0) { @@ -320,7 +320,7 @@ static void handle_signal(CFFileDescriptorRef f, CFOptionFlags callback_types, v VERIFY(callback_types & kCFFileDescriptorReadCallBack); auto* signal_handlers = static_cast(info); - struct kevent event { }; + struct kevent event {}; // returns number of events that have occurred since last call (void)::kevent(CFFileDescriptorGetNativeDescriptor(f), nullptr, 0, &event, 1, nullptr); diff --git a/Libraries/LibWebView/ProcessManager.cpp b/Libraries/LibWebView/ProcessManager.cpp index c0b2e29bedd..de447ea6a01 100644 --- a/Libraries/LibWebView/ProcessManager.cpp +++ b/Libraries/LibWebView/ProcessManager.cpp @@ -47,7 +47,7 @@ StringView process_name_from_type(ProcessType type) } ProcessManager::ProcessManager() - : on_process_exited([](Process&&) {}) + : on_process_exited([](Process&&) { }) { m_signal_handle = Core::EventLoop::register_signal(SIGCHLD, [this](int) { auto result = Core::System::waitpid(-1, WNOHANG); diff --git a/Libraries/LibWebView/ViewImplementation.cpp b/Libraries/LibWebView/ViewImplementation.cpp index a1398dae10e..dd7467af858 100644 --- a/Libraries/LibWebView/ViewImplementation.cpp +++ b/Libraries/LibWebView/ViewImplementation.cpp @@ -488,8 +488,8 @@ void ViewImplementation::did_allocate_iosurface_backing_stores(i32 front_id, Cor auto bytes_per_row = front_iosurface.bytes_per_row(); - auto front_bitmap = Gfx::Bitmap::create_wrapper(Gfx::BitmapFormat::BGRA8888, Gfx::AlphaType::Premultiplied, front_size, bytes_per_row, front_iosurface.data(), [handle = move(front_iosurface)] {}); - auto back_bitmap = Gfx::Bitmap::create_wrapper(Gfx::BitmapFormat::BGRA8888, Gfx::AlphaType::Premultiplied, back_size, bytes_per_row, back_iosurface.data(), [handle = move(back_iosurface)] {}); + auto front_bitmap = Gfx::Bitmap::create_wrapper(Gfx::BitmapFormat::BGRA8888, Gfx::AlphaType::Premultiplied, front_size, bytes_per_row, front_iosurface.data(), [handle = move(front_iosurface)] { }); + auto back_bitmap = Gfx::Bitmap::create_wrapper(Gfx::BitmapFormat::BGRA8888, Gfx::AlphaType::Premultiplied, back_size, bytes_per_row, back_iosurface.data(), [handle = move(back_iosurface)] { }); m_client_state.front_bitmap.bitmap = front_bitmap.release_value_but_fixme_should_propagate_errors(); m_client_state.front_bitmap.id = front_id; diff --git a/Services/WebWorker/PageHost.h b/Services/WebWorker/PageHost.h index 86a008e1aad..8587a15a715 100644 --- a/Services/WebWorker/PageHost.h +++ b/Services/WebWorker/PageHost.h @@ -31,8 +31,8 @@ public: virtual Web::CSS::PreferredColorScheme preferred_color_scheme() const override; virtual Web::CSS::PreferredContrast preferred_contrast() const override; virtual Web::CSS::PreferredMotion preferred_motion() const override; - virtual void paint_next_frame() override {}; - virtual void process_screenshot_requests() override {}; + virtual void paint_next_frame() override { } + virtual void process_screenshot_requests() override { } virtual void paint(Web::DevicePixelRect const&, Web::Painting::BackingStore&, Web::PaintOptions = {}) override; virtual void request_file(Web::FileRequest) override; virtual bool is_ready_to_paint() const override { return true; } diff --git a/Tests/LibGC/TestHeap.cpp b/Tests/LibGC/TestHeap.cpp index 661fbdbc84a..7b7d5b9e9f3 100644 --- a/Tests/LibGC/TestHeap.cpp +++ b/Tests/LibGC/TestHeap.cpp @@ -10,6 +10,6 @@ GC::Heap& test_gc_heap() { // FIXME: The GC heap should become thread aware! - thread_local GC::Heap heap(nullptr, [](auto&) {}); + thread_local GC::Heap heap(nullptr, [](auto&) { }); return heap; }