Everywhere: Fix a bunch of typos

Author: https://github.com/linusg
Commit: https://github.com/SerenityOS/serenity/commit/0aab7743436

AK/Variant.h

@@ -190,7 +190,7 @@ struct InheritFromPacks<IndexSequence<Is...>, Ps...>
     using InheritFromUniqueEntries<Is, Ps, IndexSequence<Is...>, Ps...>::InheritFromUniqueEntries...;
 };
 
-// Just a nice wrapper around InheritFromPacks, which will wrap any parameter packs in ParameterPack (unless it alread is one).
+// Just a nice wrapper around InheritFromPacks, which will wrap any parameter packs in ParameterPack (unless it already is one).
 template<typename... Ps>
 using MergeAndDeduplicatePacks = InheritFromPacks<MakeIndexSequence<sizeof...(Ps)>, Conditional<IsBaseOf<ParameterPackTag, Ps>, Ps, ParameterPack<Ps>>...>;
 

Base/usr/share/man/man1/rev.md

@@ -11,7 +11,7 @@ $ rev [files...]
 ## Description
 
 `rev` reads the specified files line by line, and prints them to standard
-output with each line being reversed characterwise. If no files are specifed,
+output with each line being reversed characterwise. If no files are specified,
 then `rev` will read from standard input.
 
 ## Examples

Documentation/CLionConfiguration.md

@@ -12,7 +12,7 @@ and set the following fields: (Assuming you use `Ninja` as the build system and
 ## Excluding Build Artifacts
 
 Source files are copied to the `Build` directory during the build, if you do not exclude them from CLion indexing they will show up
-in search results. This is often confusing, unintuitive, and can result in your loosing changes you have made to files. To exclude
+in search results. This is often confusing, unintuitive, and can result in you losing changes you have made to files. To exclude
 these files navigate to the `Project` tool window, right-click the `Build` folder and select `Mark Directory as | Excluded`. If you
 want exclude Toolchain files as well, follow the same procedure with the following paths:
 - `Toolchain/Local`

Documentation/RunningTests.md

@@ -34,7 +34,7 @@ export SERENITY_SOURCE_DIR=${PWD}/..
 ninja && ninja test
 ```
 
-To see the stdout/stderr output of failing tests, the reccommended way is to set the environment variable [`CTEST_OUTPUT_ON_FAILURE`](https://cmake.org/cmake/help/latest/manual/ctest.1.html#options) to 1.
+To see the stdout/stderr output of failing tests, the recommended way is to set the environment variable [`CTEST_OUTPUT_ON_FAILURE`](https://cmake.org/cmake/help/latest/manual/ctest.1.html#options) to 1.
 
 ```sh
 CTEST_OUTPUT_ON_FAILURE=1 ninja test

Kernel/Graphics/GraphicsManagement.cpp

@@ -97,13 +97,13 @@ UNMAP_AFTER_INIT bool GraphicsManagement::initialize()
      * 2. The bootloader specified settings of a pre-set framebuffer, and the 
      * kernel does not have a native driver for a detected display adapter, 
      * therefore the kernel will use the pre-set framebuffer. Modesetting is not
-     * availabe in this situation.
+     * available in this situation.
      * 3. The bootloader didn't specify settings of a pre-set framebuffer, and 
      * the kernel does not have a native driver for a detected display adapter, 
      * therefore the kernel will try to initialize a VGA text mode console.
      * In that situation, the kernel will assume that VGA text mode was already
      * initialized, but will still try to modeset it. No switching to graphical 
-     * enviroment is allowed in this case.
+     * environment is allowed in this case.
      * 
      * By default, the kernel assumes that no framebuffer was created until it
      * was proven that there's an existing framebuffer or we can modeset the 

Userland/DevTools/UserspaceEmulator/SoftCPU.cpp

@@ -1853,8 +1853,8 @@ void SoftCPU::FLD_RM80(const X86::Instruction& insn)
     VERIFY(!insn.modrm().is_register());
 
     // long doubles can be up to 128 bits wide in memory for reasons (alignment) and only uses 80 bits of precision
-    // gcc uses 12 byte in 32 bit and 16 byte in 64 bit mode
-    // so in the 32 bit case we read a bit to much, but that shouldnt be that bad
+    // GCC uses 12 bytes in 32 bit and 16 bytes in 64 bit mode
+    // so in the 32 bit case we read a bit to much, but that shouldn't be an issue.
     auto new_f80 = insn.modrm().read128(*this, insn);
     // FIXME: Respect shadow values
     fpu_push(*(long double*)new_f80.value().bytes());

Userland/Libraries/LibCrypto/ASN1/ASN1.cpp

@@ -127,7 +127,7 @@ Optional<Core::DateTime> parse_generalized_time(const StringView& time)
     auto month = lexer.consume(2).to_uint();
     auto day = lexer.consume(2).to_uint();
     auto hour = lexer.consume(2).to_uint();
-    Optional<unsigned> minute, seconds, miliseconds, offset_hours, offset_minutes;
+    Optional<unsigned> minute, seconds, milliseconds, offset_hours, offset_minutes;
     [[maybe_unused]] bool negative_offset = false;
     if (!lexer.is_eof()) {
         if (lexer.consume_specific('Z'))
@@ -152,8 +152,8 @@ Optional<Core::DateTime> parse_generalized_time(const StringView& time)
         }
 
         if (lexer.consume_specific('.')) {
-            miliseconds = lexer.consume(3).to_uint();
-            if (!miliseconds.has_value()) {
+            milliseconds = lexer.consume(3).to_uint();
+            if (!milliseconds.has_value()) {
                 return {};
             }
             if (lexer.consume_specific('Z'))
@@ -182,7 +182,7 @@ done_parsing:;
     if (offset_hours.has_value() || offset_minutes.has_value())
         dbgln("FIXME: Implement GeneralizedTime with offset!");
 
-    // Unceremonially drop the miliseconds on the floor.
+    // Unceremonially drop the milliseconds on the floor.
     return Core::DateTime::create(year.value(), month.value(), day.value(), hour.value(), minute.value_or(0), seconds.value_or(0));
 }
 

Userland/Libraries/LibGL/SoftwareGLContext.cpp

@@ -127,7 +127,7 @@ void SoftwareGLContext::gl_end()
     // 2.   Transform all of the vertices from eye space into clip space by multiplying by the projection matrix
     // 3.   If culling is enabled, we cull the desired faces (https://learnopengl.com/Advanced-OpenGL/Face-culling)
     // 4.   Each element of the vertex is then divided by w to bring the positions into NDC (Normalized Device Coordinates)
-    // 5.   The vertices are sorted (for the rasteriser, how are we doing this? 3Dfx did this top to bottom in terms of vertex y co-ordinates)
+    // 5.   The vertices are sorted (for the rasteriser, how are we doing this? 3Dfx did this top to bottom in terms of vertex y coordinates)
     // 6.   The vertices are then sent off to the rasteriser and drawn to the screen
 
     float scr_width = m_frontbuffer->width();

Userland/Libraries/LibGfx/TextDirection.cpp

@@ -9,7 +9,7 @@
 
 namespace Gfx {
 
-// FIXME: These should be parsed from the official UnicodeData.txt that specifies the class for each character (this function doesnt take into account a large amount of characters)
+// FIXME: These should be parsed from the official UnicodeData.txt that specifies the class for each character (this function doesn't take into account a large amount of characters)
 static consteval Array<BidirectionalClass, 0x1F000> generate_char_bidi_class_lookup_table()
 {
     Array<BidirectionalClass, 0x1F000> lookup_table {};
@@ -22,7 +22,7 @@ static consteval Array<BidirectionalClass, 0x1F000> generate_char_bidi_class_loo
         if ((ch >= 0x30 && ch <= 0x39) || (ch >= 0x660 && ch <= 0x669) || (ch >= 0x10D30 && ch <= 0x10E7E))
             char_class = BidirectionalClass::WEAK_NUMBERS; // Numerals
         if ((ch >= 0x23 && ch <= 0x25) || (ch >= 0x2B && ch <= 0x2F) || (ch == 0x3A))
-            char_class = BidirectionalClass::WEAK_SEPARATORS; // Seperators
+            char_class = BidirectionalClass::WEAK_SEPARATORS; // Separators
         if ((ch >= 0x9 && ch <= 0xD) || (ch >= 0x1C && ch <= 0x22) || (ch >= 0x26 && ch <= 0x2A) || (ch >= 0x3B && ch <= 0x40) || (ch >= 0x5B && ch <= 0x60) || (ch >= 0x7B && ch <= 0x7E))
             char_class = BidirectionalClass::NEUTRAL;
         lookup_table[ch] = char_class;

Userland/Libraries/LibGfx/TextDirection.h

@@ -28,7 +28,7 @@ constexpr BidirectionalClass get_char_bidi_class(u32 ch)
     return char_bidi_class_lookup_table[ch];
 }
 
-// FIXME: These should be parsed from the official BidiMirroring.txt that specifies the mirroring character for each character (this function doesnt take into account a large amount of characters)
+// FIXME: These should be parsed from the official BidiMirroring.txt that specifies the mirroring character for each character (this function doesn't take into account a large amount of characters)
 constexpr u32 get_mirror_char(u32 ch)
 {
     if (ch == 0x28)

Userland/Libraries/LibPthread/pthread.cpp

@@ -612,7 +612,7 @@ int pthread_spin_lock(pthread_spinlock_t* lock)
 int pthread_spin_trylock(pthread_spinlock_t* lock)
 {
     // We expect the current value to be unlocked, as the specification
-    // states that trylock should lock ony if it is not held by ANY thread.
+    // states that trylock should lock only if it is not held by ANY thread.
     auto current = spinlock_unlock_sentinel;
     auto desired = gettid();
 

Userland/Libraries/LibSQL/Parser.cpp

@@ -319,7 +319,7 @@ NonnullRefPtr<Select> Parser::parse_select_statement(RefPtr<CommonTableExpressio
         if (consume_if(TokenType::Offset)) {
             offset_expression = parse_expression();
         } else {
-            // Note: The limit clause may instead be definied as "offset-expression, limit-expression", effectively reversing the
+            // Note: The limit clause may instead be defined as "offset-expression, limit-expression", effectively reversing the
             // order of the expressions. SQLite notes "this is counter-intuitive" and "to avoid confusion, programmers are strongly
             // encouraged to ... avoid using a LIMIT clause with a comma-separated offset."
             VERIFY(!consume_if(TokenType::Comma));
@@ -907,7 +907,7 @@ NonnullRefPtr<ResultColumn> Parser::parse_result_column()
         return create_ast_node<ResultColumn>();
 
     // If we match an identifier now, we don't know whether it is a table-name of the form "table-name.*", or if it is the start of a
-    // column-name-expression, until we try to parse the asterisk. So if we consume an indentifier and a period, but don't find an
+    // column-name-expression, until we try to parse the asterisk. So if we consume an identifier and a period, but don't find an
     // asterisk, hold onto that information to form a column-name-expression later.
     String table_name;
     bool parsed_period = false;

Userland/Libraries/LibWeb/CSS/Parser/Parser.cpp

@@ -89,7 +89,7 @@ Vector<CSS::Selector::ComplexSelector> Parser::parse_selectors(Vector<String> pa
     // TODO:
     // This is a mess because the prelude is parsed as a string.
     // It should really be parsed as its class, but the cpp gods have forsaken me
-    // and i cant make it work due to cyclic includes.
+    // and I can't make it work due to cyclic includes.
 
     Vector<CSS::Selector::ComplexSelector> selectors;
 

Userland/Libraries/LibWebSocket/WebSocket.cpp

@@ -219,7 +219,7 @@ void WebSocket::read_server_handshake()
         }
         if (parts[1] != "101") {
             // 1. If the status code is not 101, handle as per HTTP procedures.
-            // FIXME : This could be a redirect or a 401 authentification request, which we do not handle.
+            // FIXME : This could be a redirect or a 401 authentication request, which we do not handle.
             dbgln("WebSocket: Server HTTP Handshake return status {} which isn't supported", parts[1]);
             fatal_error(WebSocket::Error::ConnectionUpgradeFailed);
             return;