Fix typos (#73122)

Author: am11

eng/build.ps1

@@ -66,7 +66,7 @@ function Get-Help() {
   Write-Host "  -restore                Restore dependencies."
   Write-Host "  -sign                   Sign build outputs."
   Write-Host "  -test (-t)              Incrementally builds and runs tests."
-  Write-Host "                          Use in conjuction with -testnobuild to only run tests."
+  Write-Host "                          Use in conjunction with -testnobuild to only run tests."
   Write-Host ""
 
   Write-Host "Libraries settings:"
@@ -216,7 +216,7 @@ if ($vs) {
     # Respect the RuntimeConfiguration variable for building inside VS with different runtime configurations
     $env:RUNTIMECONFIGURATION=$runtimeConfiguration
   }
-  
+
   # Launch Visual Studio with the locally defined environment variables
   ."$vs"
 

eng/build.sh

@@ -56,7 +56,7 @@ usage()
   echo "  --restore (-r)             Restore dependencies."
   echo "  --sign                     Sign build outputs."
   echo "  --test (-t)                Incrementally builds and runs tests."
-  echo "                             Use in conjuction with --testnobuild to only run tests."
+  echo "                             Use in conjunction with --testnobuild to only run tests."
   echo ""
 
   echo "Libraries settings:"

src/coreclr/System.Private.CoreLib/src/System/GC.CoreCLR.cs

@@ -785,7 +785,7 @@ private static unsafe void Callback(void* configurationContext, void* name, void
         }
 
         /// <summary>
-        /// Gets the Configurations used by the Garbage Collector. The value of these configurations used don't neccessarily have to be the same as the ones that are passed by the user.
+        /// Gets the Configurations used by the Garbage Collector. The value of these configurations used don't necessarily have to be the same as the ones that are passed by the user.
         /// For example for the "GCHeapCount" configuration, if the user supplies a value higher than the number of CPUs, the configuration that will be used is that of the number of CPUs.
         /// <returns> A Read Only Dictionary with configuration names and values of the configuration as the keys and values of the dictionary, respectively.</returns>
         /// </summary>

src/coreclr/System.Private.CoreLib/src/System/Reflection/Emit/TypeBuilderInstantiation.cs

@@ -107,7 +107,7 @@ public override Type MakeArrayType(int rank)
         [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2055:UnrecognizedReflectionPattern",
             Justification = "The entire TypeBuilderInstantiation is serving the MakeGenericType implementation. " +
                             "Currently this is not supported by linker. Once it is supported the outercall (Type.MakeGenericType)" +
-                            "will validate that the types fullfill the necessary requirements of annotations on type parameters." +
+                            "will validate that the types fulfill the necessary requirements of annotations on type parameters." +
                             "As such the actual internals of the implementation are not interesting.")]
         private Type Substitute(Type[] substitutes)
         {

src/coreclr/binder/assemblybindercommon.cpp

@@ -1089,7 +1089,7 @@ namespace BINDER_SPACE
                 // Lock the application context
                 CRITSEC_Holder contextLock(pApplicationContext->GetCriticalSectionCookie());
 
-                // Only perform costly validation if other binds succeded before us
+                // Only perform costly validation if other binds succeeded before us
                 if (kContextVersion != pApplicationContext->GetVersion())
                 {
                     IF_FAIL_GO(AssemblyBinderCommon::OtherBindInterfered(pApplicationContext,

src/coreclr/classlibnative/bcltype/oavariant.cpp

@@ -5,7 +5,7 @@
 //
 
 //
-// Purpose: Wrapper for Ole Automation compatable math ops.
+// Purpose: Wrapper for Ole Automation compatible math ops.
 // Calls through to OleAut.dll
 //
 

src/coreclr/classlibnative/bcltype/oavariant.h

@@ -5,7 +5,7 @@
 //
 
 //
-// Purpose: Wrapper for Ole Automation compatable math ops.
+// Purpose: Wrapper for Ole Automation compatible math ops.
 // Calls through to OleAut.dll
 //
 

src/coreclr/classlibnative/bcltype/objectnative.cpp

@@ -224,7 +224,7 @@ FCIMPL1(Object*, ObjectNative::AllocateUninitializedClone, Object* pObjUNSAFE)
 {
     FCALL_CONTRACT;
 
-    // Delegate error handling to managed side (it will throw NullRefenceException)
+    // Delegate error handling to managed side (it will throw NullReferenceException)
     if (pObjUNSAFE == NULL)
         return NULL;
 

src/coreclr/debug/daccess/dacdbiimplstackwalk.cpp

@@ -830,7 +830,7 @@ void DacDbiInterfaceImpl::InitFrameData(StackFrameIterator *   pIter,
         //  2. !pCF->HasFaulted() - It wasn't a "hardware" exception (Access violation, dev by 0, etc.)
         //  3. !pCF->IsIPadjusted() - It hasn't been previously adjusted to point to [call IL_Throw]
         //  4. pJITFuncData->nativeOffset != 0 - nativeOffset contains something that looks like a real return address.
-        pJITFuncData->jsutAfterILThrow = pCF->IsInterrupted()
+        pJITFuncData->justAfterILThrow = pCF->IsInterrupted()
                                      && !pCF->HasFaulted()
                                      && !pCF->IsIPadjusted()
                                      && pJITFuncData->nativeOffset != 0;

src/coreclr/debug/daccess/dacfn.cpp

@@ -1455,7 +1455,7 @@ void DacEnumCodeForStackwalk(TADDR taCallEnd)
     // Note that this only handles absolute indirect calls (ModR/M byte of 0x15), all the other forms of
     // indirect calls are register-relative, and so we'd have to do a much more complicated decoding based
     // on the register context.  Regardless, it seems like this is fundamentally error-prone because it's
-    // aways possible that the call instruction was not 6 bytes long, and we could have some other instructions
+    // always possible that the call instruction was not 6 bytes long, and we could have some other instructions
     // that happen to match the pattern we're looking for.
     PTR_BYTE callCode = PTR_BYTE(taCallEnd - 6);
     PTR_BYTE callMrm = PTR_BYTE(taCallEnd - 5);

src/coreclr/debug/daccess/dacimpl.h

@@ -652,7 +652,7 @@ struct DumpMemoryReportStatics
     TSIZE_T    m_cbModuleList;         // number of bytes that we report for module list directly
     TSIZE_T    m_cbClrStatics;         // number of bytes that we report for CLR statics
     TSIZE_T    m_cbClrHeapStatics;     // number of bytes that we report for CLR heap statics
-    TSIZE_T    m_cbImplicity;          // number of bytes that we report implicitly
+    TSIZE_T    m_cbImplicitly;          // number of bytes that we report implicitly
 };
 
 
@@ -1592,7 +1592,7 @@ class DefaultCOMImpl : public T
 };
 
 
-// A stuct representing a thread's allocation context.
+// A struct representing a thread's allocation context.
 struct AllocInfo
 {
     CORDB_ADDRESS Ptr;

src/coreclr/debug/daccess/enummem.cpp

@@ -329,7 +329,7 @@ HRESULT ClrDataAccess::EnumMemoryRegionsWorkerHeap(IN CLRDataEnumMemoryFlags fla
     CATCH_ALL_EXCEPT_RETHROW_COR_E_OPERATIONCANCELLED( g_pDebugger->EnumMemoryRegions(flags); )
 
     // now dump the memory get dragged in by using DAC API implicitly.
-    m_dumpStats.m_cbImplicity = m_instances.DumpAllInstances(m_enumMemCb);
+    m_dumpStats.m_cbImplicitly = m_instances.DumpAllInstances(m_enumMemCb);
 
     // Do not let any remaining implicitly enumerated memory leak out.
     Flush();
@@ -340,7 +340,7 @@ HRESULT ClrDataAccess::EnumMemoryRegionsWorkerHeap(IN CLRDataEnumMemoryFlags fla
 //+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 //
 // Helper function for skinny mini-dump
-// Pass in an managed object, this function will dump the EEClass hierachy
+// Pass in an managed object, this function will dump the EEClass hierarchy
 // and field desc of object so SOS's !DumpObj will work
 //
 //
@@ -1615,7 +1615,7 @@ HRESULT ClrDataAccess::EnumMemoryRegionsWorkerSkinny(IN CLRDataEnumMemoryFlags f
 #endif // FEATURE_MINIMETADATA_IN_TRIAGEDUMPS
 
     // now dump the memory get dragged in by using DAC API implicitly.
-    m_dumpStats.m_cbImplicity = m_instances.DumpAllInstances(m_enumMemCb);
+    m_dumpStats.m_cbImplicitly = m_instances.DumpAllInstances(m_enumMemCb);
 
     // Do not let any remaining implicitly enumerated memory leak out.
     Flush();
@@ -1665,7 +1665,7 @@ HRESULT ClrDataAccess::EnumMemoryRegionsWorkerMicroTriage(IN CLRDataEnumMemoryFl
 #endif // FEATURE_MINIMETADATA_IN_TRIAGEDUMPS
 
     // now dump the memory get dragged in by using DAC API implicitly.
-    m_dumpStats.m_cbImplicity = m_instances.DumpAllInstances(m_enumMemCb);
+    m_dumpStats.m_cbImplicitly = m_instances.DumpAllInstances(m_enumMemCb);
 
     // Do not let any remaining implicitly enumerated memory leak out.
     Flush();
@@ -1756,7 +1756,7 @@ HRESULT ClrDataAccess::EnumMemoryRegionsWorkerCustom()
         // we are done...
 
         // now dump the memory get dragged in implicitly
-        m_dumpStats.m_cbImplicity = m_instances.DumpAllInstances(m_enumMemCb);
+        m_dumpStats.m_cbImplicitly = m_instances.DumpAllInstances(m_enumMemCb);
 
     }
     else if (eFlavor == DUMP_FLAVOR_CriticalCLRState)
@@ -1783,7 +1783,7 @@ HRESULT ClrDataAccess::EnumMemoryRegionsWorkerCustom()
         // we are done...
 
         // now dump the memory get dragged in implicitly
-        m_dumpStats.m_cbImplicity = m_instances.DumpAllInstances(m_enumMemCb);
+        m_dumpStats.m_cbImplicitly = m_instances.DumpAllInstances(m_enumMemCb);
 
     }
     else if (eFlavor == DUMP_FLAVOR_NonHeapCLRState)

src/coreclr/debug/di/module.cpp

@@ -1860,7 +1860,7 @@ CordbFunction * CordbModule::CreateFunction(mdMethodDef funcMetaDataToken, SIZE_
 //
 // Arguments:
 //    funcMetaDataToken - the functions methodDef token in this module
-//    enCVerison        - The new version number of this function
+//    enCVersion        - The new version number of this function
 //    ppFunction        - Output param for the new instance - optional
 //
 // Assumptions:

src/coreclr/debug/di/process.cpp

@@ -5616,7 +5616,7 @@ void CordbProcess::RawDispatchEvent(
             // Neutering will dump it back down to zero.
             this->AddToNeuterOnExitList(pMDA);
 
-            // We bump up and down the external ref so that even if the callback doensn't touch the refs,
+            // We bump up and down the external ref so that even if the callback doesn't touch the refs,
             // our Ext-Release here will still cause a 1->0 ext-ref transition, which will get it
             // swept on the neuter list.
             RSExtSmartPtr<ICorDebugMDA> pExternalMDARef;
@@ -5967,7 +5967,7 @@ void CordbProcess::RawDispatchEvent(
 //---------------------------------------------------------------------------------------
 // Callback for prepopulating threads.
 //
-// Arugments:
+// Arguments:
 //    vmThread - thread as part of the eunmeration.
 //    pUserData - data supplied with callback. It's a CordbProcess* object.
 //

src/coreclr/debug/di/rsclass.cpp

@@ -621,7 +621,7 @@ HRESULT CordbClass::SetJMCStatus(BOOL fIsUserCode)
 // We have to go the EE to find out if a class is a value
 // class or not.  This is because there is no flag for this, but rather
 // it depends on whether the class subclasses System.ValueType (apart
-// from System.Enum...).  Replicating all that resoultion logic
+// from System.Enum...).  Replicating all that resolution logic
 // does not seem like a good plan.
 //
 // We also accept other "evidence" that the class is or isn't a VC, in

src/coreclr/debug/di/rspriv.h

@@ -1561,7 +1561,7 @@ _____Neuter_Status_Already_Marked = 0; \
 // 1) it means that we have no synchronization (can't take the Stop-Go lock)
 // 2) none of our backpointers are usable (they may be nulled out at anytime by another thread).
 //    - this also means we absolutely can't send IPC events (since that requires a CordbProcess)
-// 3) The only safe data are blittalbe embedded fields (eg, a pid or stack range)
+// 3) The only safe data are blittable embedded fields (eg, a pid or stack range)
 //
 // Any usage of this macro should clearly specify why this is safe.
 #define OK_IF_NEUTERED(pThis) \
@@ -11203,7 +11203,7 @@ class DbgRSThread
     void NotifyTakeLock(RSLock * pLock);
     void NotifyReleaseLock(RSLock * pLock);
 
-    // Used to map other resources (like thread access) into the lock hierachy.
+    // Used to map other resources (like thread access) into the lock hierarchy.
     // Note this only effects lock leveling checks and doesn't effect HoldsAnyLock().
     void TakeVirtualLock(RSLock::ERSLockLevel level);
     void ReleaseVirtualLock(RSLock::ERSLockLevel level);

src/coreclr/debug/di/rsstackwalk.cpp

@@ -714,13 +714,13 @@ HRESULT CordbStackWalk::GetFrameWorker(ICorDebugFrame ** ppFrame)
             // instruction in the leaf frame. In the past we didn't always achieve this,
             // but we are being more deliberate about this behavior now.
 
-            // If jsutAfterILThrow is true, it means nativeOffset points to the return address of IL_Throw
+            // If justAfterILThrow is true, it means nativeOffset points to the return address of IL_Throw
             // (or another JIT exception helper) after an exception has been thrown.
             // In such cases we want to adjust nativeOffset, so it will point an actual exception callsite.
             // By subtracting STACKWALK_CONTROLPC_ADJUST_OFFSET from nativeOffset you can get
             // an address somewhere inside CALL instruction.
             // This ensures more consistent placement of exception line highlighting in Visual Studio
-            DWORD nativeOffsetToMap = pJITFuncData->jsutAfterILThrow ?
+            DWORD nativeOffsetToMap = pJITFuncData->justAfterILThrow ?
                                (DWORD)pJITFuncData->nativeOffset - STACKWALK_CONTROLPC_ADJUST_OFFSET :
                                (DWORD)pJITFuncData->nativeOffset;
             CorDebugMappingResult mappingType;

src/coreclr/debug/di/rsthread.cpp

@@ -3007,7 +3007,7 @@ HRESULT CordbUnmanagedThread::RestoreLeafSeh()
 // pRead is optional. This makes sense when '0' is a valid default value.
 // 1) On success (block exists in LS, we can read it),
 //    return value of data in the slot, *pRead = true
-// 2) On failure to read block (block doens't exist yet, any other failure)
+// 2) On failure to read block (block doesn't exist yet, any other failure)
 //    return value == 0 (assumed default, *pRead = false
 REMOTE_PTR CordbUnmanagedThread::GetPreDefTlsSlot(SIZE_T offset)
 {

src/coreclr/debug/di/shimprocess.cpp

@@ -1563,7 +1563,7 @@ void ShimProcess::PreDispatchEvent(bool fRealCreateProcessEvent /*= false*/)
         // Remember that we're processing the first managed event so that we only call HandleFirstRCEvent() once
         m_fFirstManagedEvent = true;
 
-        // This can fail with the incompatable version HR. The process has already been terminated if this
+        // This can fail with the incompatible version HR. The process has already been terminated if this
         // is the case. This will dispatch an Error callback
         // If this fails, the process is in an undefined state.
         // @dbgtodo ipc-block: this will go away once we get rid

src/coreclr/debug/ee/controller.cpp

@@ -5555,7 +5555,7 @@ bool DebuggerStepper::TrapStepInto(ControllerStackInfo *info,
     // If we're calling from managed code, this should either succeed
     // or become an ecall into mscorwks.
     // @Todo - what about stubs in mscorwks.
-    // @todo - if this fails, we want to provde as much info as possible.
+    // @todo - if this fails, we want to provide as much info as possible.
     if (!g_pEEInterface->TraceStub(ip, &trace)
         || !g_pEEInterface->FollowTrace(&trace))
     {

src/coreclr/debug/ee/dactable.cpp

@@ -98,7 +98,7 @@ const DacGlobals _DacGlobals::s_dacGlobals =
 
 // DacGlobals::Initialize is a no-op on MSVC builds as we statically initialize the table,
 // however, it provides a nice mechanism for us to get back into the right scope to validate the usage of DEFINE_DACVAR and family
-// without needing to make all of the DAC varables public or include all of the headers in daccess.h.
+// without needing to make all of the DAC variables public or include all of the headers in daccess.h.
 void DacGlobals::Initialize()
 {
 #define DEFINE_DACVAR(size, id, var) static_assert(!is_type_template_instantiation<decltype(var), Volatile>::m_value, "DAC variables defined with DEFINE_DACVAR must not be instantiations of Volatile<T>.");

src/coreclr/debug/ee/debugger.cpp

@@ -2343,7 +2343,7 @@ HRESULT Debugger::RequestFavor(FAVORCALLBACK fp, void * pData)
     if (m_pRCThread == NULL ||
         m_pRCThread->GetRCThreadId() == GetCurrentThreadId())
     {
-        // Since favors are only used internally, we know that the helper should alway be up and ready
+        // Since favors are only used internally, we know that the helper should always be up and ready
         // to handle them. Also, since favors can be used in low-stack scenarios, there's not any
         // extra initialization needed for them.
         _ASSERTE(!"Helper not initialized for favors.");
@@ -3370,7 +3370,7 @@ void Debugger::getVars(MethodDesc * md, ULONG32 *cVars, ICorDebugInfo::ILVarInfo
 
     // Just tell the JIT to extend everything.
     // Note that if optimizations are enabled, the native compilers are
-    // free to ingore *extendOthers
+    // free to ignore *extendOthers
     *extendOthers = true;
 
     DWORD bits = md->GetModule()->GetDebuggerInfoBits();
@@ -4747,7 +4747,7 @@ BOOL IsDuplicatePatch(SIZE_T *rgEntries,
 /******************************************************************************
 // HRESULT Debugger::MapAndBindFunctionBreakpoints():  For each breakpoint
 //      that we've set in any version of the existing function,
-//      set a correponding breakpoint in the new function if we haven't moved
+//      set a corresponding breakpoint in the new function if we haven't moved
 //      the patch to the new version already.
 //
 //      This must be done _AFTER_ the MethodDesc has been updated
@@ -4790,7 +4790,7 @@ HRESULT Debugger::MapAndBindFunctionPatches(DebuggerJitInfo *djiNew,
         "Ver:0x%04x (DJI:0x%p)\n", djiNew?djiNew->m_methodInfo->GetCurrentEnCVersion():0, djiNew));
 
     // We need to traverse the patch list while under the controller lock (small lock).
-    // But we can only send BreakpointSetErros while under the debugger lock (big lock).
+    // But we can only send BreakpointSetErrors while under the debugger lock (big lock).
     // So to avoid a lock violation, we queue any errors we find under the small lock,
     // and then send the whole list when under the big lock.
     PATCH_UNORDERED_ARRAY listUnbindablePatches;
@@ -8465,7 +8465,7 @@ BOOL Debugger::ShouldAutoAttach()
 
     LOG((LF_CORDB, LL_INFO1000000, "D::SAD\n"));
 
-    // Check if the user has specified a seting in the registry about what he
+    // Check if the user has specified a setting in the registry about what he
     // wants done when an unhandled exception occurs.
     DebuggerLaunchSetting dls = GetDbgJITDebugLaunchSetting();
 
@@ -8810,7 +8810,7 @@ Debugger::ATTACH_ACTION Debugger::ShouldAttachDebugger(bool fIsUserBreakpoint)
         return ATTACH_NO;
     }
 
-    // Check if the user has specified a seting in the registry about what he wants done when an unhandled exception
+    // Check if the user has specified a setting in the registry about what he wants done when an unhandled exception
     // occurs.
     DebuggerLaunchSetting dls = GetDbgJITDebugLaunchSetting();
 

src/coreclr/debug/ee/funceval.cpp

@@ -3158,7 +3158,7 @@ static void RecordFuncEvalException(DebuggerEval *pDE,
  * domain (steps 1a & 1b).  This has to be a separate function from GCProtectArgsAndDoNormalFuncEval
  * because otherwise we can't reliably find the right GCFrames to pop when unwinding the stack due to
  * an exception on 64-bit platforms (we have some GCFrames outside of the TRY, and some inside,
- * and they won't necesarily be laid out sequentially on the stack if they are all in the same function).
+ * and they won't necessarily be laid out sequentially on the stack if they are all in the same function).
  *
  * Parameters:
  *    pDE - pointer to the DebuggerEval object being processed.

src/coreclr/debug/ee/functioninfo.cpp

@@ -1230,7 +1230,7 @@ void DebuggerJitInfo::SetBoundaries(ULONG32 cMap, ICorDebugInfo::OffsetMapping *
 // Init a DJI after it's jitted.
 void DebuggerJitInfo::Init(TADDR newAddress)
 {
-    // Shouldn't initialize while holding the lock b/c initialzing may call functions that lock,
+    // Shouldn't initialize while holding the lock b/c initializing may call functions that lock,
     // and thus we'd have a locking violation.
     _ASSERTE(!g_pDebugger->HasDebuggerDataLock());