Merge pull request #14289 from Sintendo/typos

Fix various typos and spelling mistakes

Source/Core/Common/Assembler/GekkoIRGen.cpp

@@ -844,7 +844,7 @@ void GekkoIRPlugin::EvalTerminalAbs(Terminal type, const AssemblerToken& tok)
 
   case Terminal::NumLabFwd:
     m_owner->EmitErrorHere(
-        fmt::format("Forward label references not supported in fully resolved expressons"));
+        fmt::format("Forward label references not supported in fully resolved expressions"));
     break;
 
   case Terminal::NumLabBwd:

Source/Core/Common/Buffer.h

@@ -7,7 +7,7 @@
 
 // UniqueBuffer<T> and SharedBuffer<T> are a lighter alternative to std::vector<T>.
 // The main benefit is that elements are not value-initialized like in vector.
-// That can be quite a bit of unecessary overhead when allocating a large buffer.
+// That can be quite a bit of unnecessary overhead when allocating a large buffer.
 
 namespace Common
 {

Source/Core/Common/StringUtil.h

@@ -314,7 +314,7 @@ bool CaseInsensitiveContains(std::string_view haystack, std::string_view needle)
 // 'std::less'-like comparison function object type for case-insensitive strings.
 struct CaseInsensitiveLess
 {
-  using is_transparent = void;  // Allow heterogenous lookup.
+  using is_transparent = void;  // Allow heterogeneous lookup.
   bool operator()(std::string_view a, std::string_view b) const;
 };
 

Source/Core/Core/DSP/DSPAssembler.h

@@ -57,7 +57,7 @@ public:
   explicit DSPAssembler(const AssemblerSettings& settings);
   ~DSPAssembler();
 
-  // line_numbers is optional (and not yet implemented). It'll receieve a list of ints,
+  // line_numbers is optional (and not yet implemented). It'll receive a list of ints,
   // one for each word of code, indicating the source assembler code line number it came from.
 
   // If returns false, call GetErrorString to get some text to present to the user.

Source/Core/Core/DSP/DSPDisassembler.cpp

@@ -108,7 +108,7 @@ std::string DSPDisassembler::DisassembleParameters(const DSPOPCTemplate& opc, u1
         if (opc.params[j].mask == 0x003f)
         {
           // Left and right shifts function essentially as a single shift by a 7-bit signed value,
-          // but are split into two intructions for clarity.
+          // but are split into two instructions for clarity.
           buf += fmt::format("#{}", (val & 0x20) != 0 ? (int(val) - 64) : int(val));
         }
         else

Source/Core/Core/DSP/Jit/x64/DSPJitRegCache.cpp

@@ -16,7 +16,7 @@ using namespace Gen;
 
 namespace DSP::JIT::x64
 {
-// Ordered in order of prefered use.
+// Ordered in order of preferred use.
 // Not all of these are actually available
 constexpr std::array<X64Reg, 15> s_allocation_order = {
     {R8, R9, R10, R11, R12, R13, R14, R15, RSI, RDI, RBX, RCX, RDX, RAX, RBP}};

Source/Core/Core/Debugger/BranchWatch.h

@@ -98,7 +98,7 @@ struct BranchWatchSelectionValueType
   BranchWatchCollection::value_type* collection_ptr;
   bool is_virtual;
   bool condition;
-  // This is moreso a GUI thing, but it works best in the Core code for multiple reasons.
+  // This is more so a GUI thing, but it works best in the Core code for multiple reasons.
   Inspection inspection;
 };
 

Source/Core/Core/Debugger/CodeTrace.cpp

@@ -263,7 +263,7 @@ HitType CodeTrace::TraceLogic(const TraceOutput& current_instr, bool first_hit)
   if (!match_reg0 && !match_reg123 && !mem_hit)
     return HitType::SKIP;
 
-  // Checks if the intstruction is a type that needs special handling.
+  // Checks if the instruction is a type that needs special handling.
   const auto CompareInstruction = [](std::string_view instruction, const auto& type_compare) {
     return std::ranges::any_of(
         type_compare, [&instruction](std::string_view s) { return instruction.starts_with(s); });

Source/Core/Core/FifoPlayer/FifoDataFile.cpp

@@ -42,7 +42,7 @@ struct FileHeader
   u32 flags;
   u64 texMemOffset;
   u32 texMemSize;
-  // These are for overriden RAM sizes.  Otherwise the FIFO Player
+  // These are for overridden RAM sizes.  Otherwise the FIFO Player
   // will crash and burn with mismatched settings.  See PR #8722.
   u32 mem1_size;
   u32 mem2_size;
@@ -136,7 +136,7 @@ bool FifoDataFile::Save(const std::string& filename)
   FileHeader header;
   header.fileId = FILE_ID;
   header.file_version = VERSION_NUMBER;
-  // Maintain backwards compatability so long as the RAM sizes aren't overridden.
+  // Maintain backwards compatibility so long as the RAM sizes aren't overridden.
   if (Config::Get(Config::MAIN_RAM_OVERRIDE_ENABLE))
     header.min_loader_version = MIN_LOADER_VERSION_FOR_RAM_OVERRIDE;
   else

Source/Core/Core/FifoPlayer/FifoPlayer.cpp

@@ -315,7 +315,7 @@ void FifoPlayer::SetFileLoadedCallback(CallbackFunc callback)
 {
   m_FileLoadedCb = std::move(callback);
 
-  // Trigger the callback immediatly if the file is already loaded.
+  // Trigger the callback immediately if the file is already loaded.
   if (GetFile() != nullptr)
   {
     m_FileLoadedCb();

Source/Core/Core/HLE/HLE.cpp

@@ -103,7 +103,7 @@ void PatchFixedFunctions(Core::System& system)
   // Not part of the binary itself, but either we or Gecko OS might insert
   // this, and it doesn't clear the icache properly.
   Patch(system, Gecko::ENTRY_POINT, "GeckoCodehandler");
-  // This has to always be installed even if cheats are not enabled because of the possiblity of
+  // This has to always be installed even if cheats are not enabled because of the possibility of
   // loading a savestate where PC is inside the code handler while cheats are disabled.
   Patch(system, Gecko::HLE_TRAMPOLINE_ADDRESS, "GeckoHandlerReturnTrampoline");
 }

Source/Core/Core/HW/AddressSpace.h

@@ -32,7 +32,7 @@ struct Accessors
   virtual u8 ReadU8(const Core::CPUThreadGuard& guard, u32 address) const = 0;
   virtual void WriteU8(const Core::CPUThreadGuard& guard, u32 address, u8 value) = 0;
 
-  // overrideable naive implementations of below are defined
+  // overridable naive implementations of below are defined
   virtual u16 ReadU16(const Core::CPUThreadGuard& guard, u32 address) const;
   virtual void WriteU16(const Core::CPUThreadGuard& guard, u32 address, u16 value);
   virtual u32 ReadU32(const Core::CPUThreadGuard& guard, u32 address) const;

Source/Core/Core/HW/DSPHLE/UCodes/ASnd.cpp

@@ -31,7 +31,7 @@ constexpr u32 MAIL_INPUT_NEXT_SAMPLES = 0x0222;
 // "send the samples for the internal buffer to the external buffer"
 constexpr u32 MAIN_SEND_SAMPLES = 0x0666;
 // "special: to dump the IROM Datas (remember disable others functions from the interrupt vector to
-// use) (CMBH+0x8000) countain the address of IROM" (not used)
+// use) (CMBH+0x8000) contains the address of IROM" (not used)
 constexpr u32 MAIL_ROM_DUMP_WORD = 0x0777;
 // "Used for test" (not used)
 constexpr u32 MAIL_TEST = 0x0888;

Source/Core/Core/HW/DSPHLE/UCodes/ASnd.h

@@ -103,7 +103,7 @@ private:
     u32 flags;
     // internal addr counter
     u32 start_addr;
-    // end voice physical pointer (bytes without alignament, but remember it reads in blocks of 32
+    // end voice physical pointer (bytes without alignment, but remember it reads in blocks of 32
     // bytes (use padding to the end))
     u32 end_addr;
     // freq operation
@@ -116,7 +116,7 @@ private:
     u16 volume_l, volume_r;
     // initial voice2 physical pointer (bytes aligned 32 bytes) (to do a ring)
     u32 start_addr2;
-    // end voice2 physical pointer (bytes without alignament, but remember it reads in blocks of 32
+    // end voice2 physical pointer (bytes without alignment, but remember it reads in blocks of 32
     // bytes (use padding to the end))
     u32 end_addr2;
     // volume (from 0 to 256) for voice 2

Source/Core/Core/HW/DSPHLE/UCodes/GBA.cpp

@@ -95,7 +95,7 @@ void GBAUCode::HandleMail(u32 mail)
   {
     PrepareBootUCode(mail);
     // The GBA ucode ignores the first 3 mails (mram_dest_addr, mram_size, mram_dram_addr)
-    // but we currently don't handle that (they're read when they shoudln't be, but DSP HLE doesn't
+    // but we currently don't handle that (they're read when they shouldn't be, but DSP HLE doesn't
     // implement them so it's fine).
     return;
   }
@@ -106,7 +106,7 @@ void GBAUCode::HandleMail(u32 mail)
   {
     if (mail == REQUEST_MAIL)
     {
-      INFO_LOG_FMT(DSPHLE, "GBAUCode - Recieved request mail");
+      INFO_LOG_FMT(DSPHLE, "GBAUCode - Received request mail");
       m_mail_state = MailState::WaitingForAddress;
     }
     else

Source/Core/Core/HW/DVD/DVDMath.cpp

@@ -52,7 +52,7 @@ constexpr double LONG_SEEK_VELOCITY_INVERSE = 4.5;  // inverse: s/m
 // In practice this yields good-enough numbers as a more exact formula
 // involving the integral over a polar equation (too complex to describe here)
 // or the approximation of a DVD as a set of concentric circles (which is a
-// better approximation, but makes futher derivations more complicated than
+// better approximation, but makes further derivations more complicated than
 // they need to be).
 //
 // From the area approximation, we end up with this formula:

Source/Core/Core/HW/EXI/BBA/XLINK_KAI_BBA.cpp

@@ -113,7 +113,7 @@ bool CEXIETHERNET::XLinkNetworkInterface::SendFrame(const u8* frame, u32 size)
       m_bba_failure_notified = true;
     }
 
-    // Return true because this isnt an error dolphin needs to handle
+    // Return true because this isn't an error dolphin needs to handle
     return true;
   }
 
@@ -160,7 +160,7 @@ void CEXIETHERNET::XLinkNetworkInterface::ReadThreadHandler(
       ERROR_LOG_FMT(SP1, "Failed to read from BBA, err={}", bytes_read);
     }
 
-    // Make sure *anything* was recieved before going any further
+    // Make sure *anything* was received before going any further
     if (bytes_read < 1)
       continue;
 
@@ -193,7 +193,7 @@ void CEXIETHERNET::XLinkNetworkInterface::ReadThreadHandler(
         }
       }
     }
-    // Otherwise we recieved control data or junk
+    // Otherwise we received control data or junk
     else
     {
       const std::string control_msg(self->m_in_frame, self->m_in_frame + bytes_read);

Source/Core/Core/HW/GCMemcard/GCMemcard.cpp

@@ -165,10 +165,10 @@ std::pair<GCMemcardErrorCode, std::optional<GCMemcard>> GCMemcard::Open(std::str
   //
   // This rule only applies for errors within a single block! That is, invalid checksums for both
   // types, and free block mismatch for the BATs. Once two valid blocks have been selected but it
-  // later turns out they do not match eachother (eg. claimed block count of a file in the directory
-  // does not match the actual block count arrived at by following BAT), the card will be treated as
-  // corrupted, even if perhaps a different combination of the two blocks would result in a valid
-  // memory card.
+  // later turns out they do not match each other (eg. claimed block count of a file in the
+  // directory does not match the actual block count arrived at by following BAT), the card will be
+  // treated as corrupted, even if perhaps a different combination of the two blocks would result in
+  // a valid memory card.
 
   // can return invalid checksum, data in unused area
   GCMemcardErrorCode dir_block_0_error_code = card.m_directory_blocks[0].CheckForErrors();
@@ -821,7 +821,7 @@ GCMemcardRemoveFileRetVal GCMemcard::RemoveFile(u8 index)  // index in the direc
 
   // TODO: Deleting a file via the GC BIOS sometimes leaves behind an extra updated directory block
   // here that has an empty file with the filename "Broken File000" where the actual deleted file
-  // was. Determine when exactly this happens and if this is neccessary for anything.
+  // was. Determine when exactly this happens and if this is necessary for anything.
 
   memset(reinterpret_cast<u8*>(&UpdatedDir.m_dir_entries[index]), 0xFF, DENTRY_SIZE);
   UpdatedDir.m_update_counter = UpdatedDir.m_update_counter + 1;

Source/Core/Core/HW/GCMemcard/GCMemcard.h

@@ -177,7 +177,7 @@ struct HeaderData
   // 4 bytes at 0x001c: VI DTV status register value (u16 from 0xCC00206E)
   u32 m_dtv_status;
 
-  // 2 bytes at 0x0020: 0 if formated in slot A, 1 if formated in slot B
+  // 2 bytes at 0x0020: 0 if formatted in slot A, 1 if formatted in slot B
   Common::BigEndianValue<u16> m_device_id;
 
   // 2 bytes at 0x0022: Size of memcard in Mbits

Source/Core/Core/HW/GCMemcard/GCMemcardDirectory.cpp

@@ -575,7 +575,7 @@ s32 GCMemcardDirectory::DirectoryWrite(u32 dest_address, u32 length, const u8* s
     // first 58 bytes should always be 0xff
     // needed to update the update ctr, checksums
     // could check for writes to the 6 important bytes but doubtful that it improves performance
-    // noticably
+    // noticeably
     memcpy((u8*)(dest) + offset, src_address, length);
     SyncSaves();
   }