Fix typos

docs/design/features/additional-deps.md

@@ -102,7 +102,7 @@ The proposed ordering change for 2.1 is:
 
 In addition, the additional-deps will always look for assembly and file version information present in the deps files in order to support "upgrade" scenarios where the additional-deps brings a newer version of a given assembly. Note that these version checks only occur for managed assemblies, not native files nor resource assemblies.
 
-## 2.1 proposal (add runtimeconfig knob to to disable `%DOTNET_ADDITIONAL_DEPS%`)
+## 2.1 proposal (add runtimeconfig knob to disable `%DOTNET_ADDITIONAL_DEPS%`)
 <strike>
 Add an `additionalDepsLookup` option to the runtimeconfig with these values:
 

docs/design/specs/Ecma-335-Augments.md

@@ -801,7 +801,7 @@ to allow `callvirt` instructions to be made in a uniform way independent of whet
 When followed by the `call` instruction or the `ldftn` instruction,
 the method must refer to a virtual static method defined on an interface. The behavior of the
 `constrained.` prefix is to change the method that the `call` or `ldftn`
-instruction refers to to be the method on `implementorType` which implements the
+instruction refers to be the method on `implementorType` which implements the
 virtual static method (paragraph *II.10.3*).
 
 (Edit the paragraph "Correctness:" second from the bottom of page 316:)

eng/testing/performance/performance-setup.sh

@@ -193,7 +193,7 @@ while (($# > 0)); do
       echo ""
       echo "Advanced settings:"
       echo "  --framework <value>            The framework to run, if not running in master"
-      echo "  --compliationmode <value>      The compilation mode if not passing --configurations"
+      echo "  --compilationmode <value>      The compilation mode if not passing --configurations"
       echo "  --sourcedirectory <value>      The directory of the sources. Defaults to env:BUILD_SOURCESDIRECTORY"
       echo "  --repository <value>           The name of the repository in the <owner>/<repository name> format. Defaults to env:BUILD_REPOSITORY_NAME"
       echo "  --branch <value>               The name of the branch. Defaults to env:BUILD_SOURCEBRANCH"
@@ -464,4 +464,4 @@ Write-PipelineSetVariable -Name 'iOSLlvmBuild' -Value "$iosllvmbuild" -is_multi_
 Write-PipelineSetVariable -name "OnlySanityCheck" -value "$only_sanity" -is_multi_job_variable false
 
 # Put it back to what was set on top of this script
-set -x
+set -x

src/coreclr/.nuget/Microsoft.CrossOsDiag.Private.CoreCLR/Microsoft.CrossOsDiag.Private.CoreCLR.proj

@@ -6,7 +6,7 @@
 
   <Target Name="VerifyCrossDacProperties" BeforeTargets="Build;Pack">
     <Error Condition="'$(CrossDacArtifactsDir)' == '' or !Exists('$(CrossDacArtifactsDir)')"
-          Text="Packaging the Cross OS DAC requires property CrossDacArtifactsDir to to a path containing all cross OS DAC assets needed." />
+          Text="Packaging the Cross OS DAC requires property CrossDacArtifactsDir to a path containing all cross OS DAC assets needed." />
   </Target>
 
   <Target Name="FilterSupportedCrossOsDacPackages" BeforeTargets="Build;Pack">

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

@@ -573,7 +573,7 @@ internal Type[] GetTypesNoLock()
         [RequiresUnreferencedCode("Types might be removed")]
         private Type? GetTypeNoLock(string className, bool throwOnError, bool ignoreCase)
         {
-            // public API to to a type. The reason that we need this function override from module
+            // public API to a type. The reason that we need this function override from module
             // is because clients might need to get foo[] when foo is being built. For example, if
             // foo class contains a data member of type foo[].
             // This API first delegate to the Module.GetType implementation. If succeeded, great!

src/coreclr/debug/daccess/dacdbiimpl.cpp

@@ -3611,7 +3611,7 @@ void DacDbiInterfaceImpl::EnumerateMemRangesForLoaderAllocator(PTR_LoaderAllocat
     heapsToEnumerate.Push(pLoaderAllocator->GetStubHeap());
 
     // GetVirtualCallStubManager returns VirtualCallStubManager*, but it's really an address to target as
-    // pLoaderAllocator is DACized. Cast it so we don't try to to a Host to Target translation.
+    // pLoaderAllocator is DACized. Cast it so we don't try to a Host to Target translation.
     VirtualCallStubManager *pVcsMgr = pLoaderAllocator->GetVirtualCallStubManager();
     LOG((LF_CORDB, LL_INFO10000, "DDBII::EMRFLA: VirtualCallStubManager 0x%x\n", PTR_HOST_TO_TADDR(pVcsMgr)));
     if (pVcsMgr)

src/coreclr/debug/daccess/enummem.cpp

@@ -888,7 +888,7 @@ HRESULT ClrDataAccess::EnumMemWalkStackHelper(CLRDataEnumMemoryFlags flags,
                                 {
                                     // This method has a generic type token which is required to figure out the exact instantiation
                                     // of the method.
-                                    // We need to to use the variable index of the generic type token in order to do the look up.
+                                    // We need to use the variable index of the generic type token in order to do the look up.
                                     CLRDATA_ADDRESS address = NULL;
                                     DWORD dwExactGenericArgsTokenIndex = 0;
                                     ReleaseHolder<IXCLRDataValue> pDV(NULL);

src/coreclr/debug/dbgutil/dbgutil.cpp

@@ -29,7 +29,7 @@ namespace
         DWORD* directoryRVA)
     {
         // Fun code ahead... below is a hand written PE decoder with some of the file offsets hardcoded.
-        // It supports no more than what we absolutely have to to get to the PE directory we need. Any of the
+        // It supports no more than what we absolutely have to get to the PE directory we need. Any of the
         // magic numbers used below can be determined by using the public documentation on the web.
         //
         // Yes utilcode has a PE decoder, no it does not support reading its data through a datatarget

src/coreclr/debug/di/process.cpp

@@ -5802,7 +5802,7 @@ void CordbProcess::RawDispatchEvent(
             FramePointer fp = pEvent->ExceptionCallback2.framePointer;
             if (fp != LEAF_MOST_FRAME)
             {
-                // The interface forces us to to pass a FramePointer via an ICorDebugFrame.
+                // The interface forces us to pass a FramePointer via an ICorDebugFrame.
                 // However, we can't get a real ICDFrame without a stackwalk, and we don't
                 // want to do a stackwalk now. so pass a netuered proxy frame. The shim
                 // can map this to a real frame.

src/coreclr/debug/ee/debugger.cpp

@@ -16383,7 +16383,7 @@ HRESULT DebuggerHeap::Init(BOOL fExecutable)
 #endif
 
 #ifdef USE_INTEROPSAFE_CANARY
-// Small header to to prefix interop-heap blocks.
+// Small header to prefix interop-heap blocks.
 // This lets us enforce that we don't delete interopheap data from a non-interop heap.
 struct InteropHeapCanary
 {

src/coreclr/gc/handletablepriv.h

@@ -796,7 +796,7 @@ void SegmentTrimExcessPages(TableSegment *pSegment);
 /*
  * TableAllocBulkHandles
  *
- * Attempts to allocate the requested number of handes of the specified type.
+ * Attempts to allocate the requested number of handles of the specified type.
  *
  * Returns the number of handles that were actually allocated.  This is always
  * the same as the number of handles requested except in out-of-memory conditions,

src/coreclr/inc/cvinfo.h

@@ -3052,7 +3052,7 @@ typedef struct CV_LVARFLAGS {
     unsigned short fIsParam          :1; // variable is a parameter
     unsigned short fAddrTaken        :1; // address is taken
     unsigned short fCompGenx         :1; // variable is compiler generated
-    unsigned short fIsAggregate      :1; // the symbol is splitted in temporaries,
+    unsigned short fIsAggregate      :1; // the symbol is split in temporaries,
                                          // which are treated by compiler as
                                          // independent entities
     unsigned short fIsAggregated     :1; // Counterpart of fIsAggregate - tells

src/coreclr/inc/eetwain.h

@@ -254,7 +254,7 @@ virtual OBJECTREF GetInstance(PREGDISPLAY     pContext,
                               EECodeInfo*     pCodeInfo) = 0;
 
 /*
-    Returns the extra argument passed to to shared generic code if it is still alive.
+    Returns the extra argument passed to shared generic code if it is still alive.
     Returns NULL in all other cases.
 */
 virtual PTR_VOID GetParamTypeArg(PREGDISPLAY     pContext,
@@ -503,7 +503,7 @@ OBJECTREF GetInstance(
                 EECodeInfo *    pCodeInfo);
 
 /*
-    Returns the extra argument passed to to shared generic code if it is still alive.
+    Returns the extra argument passed to shared generic code if it is still alive.
     Returns NULL in all other cases.
 */
 virtual

src/coreclr/jit/assertionprop.cpp

@@ -3684,7 +3684,7 @@ GenTree* Compiler::optAssertionProp_LclVar(ASSERT_VALARG_TP assertions, GenTreeL
 {
     // If we have a var definition then bail or
     // If this is the address of the var then it will have the GTF_DONT_CSE
-    // flag set and we don't want to to assertion prop on it.
+    // flag set and we don't want to assertion prop on it.
     if (tree->gtFlags & (GTF_VAR_DEF | GTF_DONT_CSE))
     {
         return nullptr;
@@ -3789,7 +3789,7 @@ GenTree* Compiler::optAssertionProp_LclFld(ASSERT_VALARG_TP assertions, GenTreeL
 {
     // If we have a var definition then bail or
     // If this is the address of the var then it will have the GTF_DONT_CSE
-    // flag set and we don't want to to assertion prop on it.
+    // flag set and we don't want to assertion prop on it.
     if (tree->gtFlags & (GTF_VAR_DEF | GTF_DONT_CSE))
     {
         return nullptr;

src/coreclr/jit/emitarm64.h

@@ -115,7 +115,7 @@ bool IsRedundantLdStr(instruction ins, regNumber reg1, regNumber reg2, ssize_t i
 
 /************************************************************************
 *
-* This union is used to to encode/decode the special ARM64 immediate values
+* This union is used to encode/decode the special ARM64 immediate values
 * that is listed as imm(N,r,s) and referred to as 'bitmask immediate'
 */
 
@@ -141,7 +141,7 @@ static INT64 emitDecodeBitMaskImm(const emitter::bitMaskImm bmImm, emitAttr size
 
 /************************************************************************
 *
-* This union is used to to encode/decode the special ARM64 immediate values
+* This union is used to encode/decode the special ARM64 immediate values
 * that is listed as imm(i16,hw) and referred to as 'halfword immediate'
 */
 
@@ -192,7 +192,7 @@ static UINT32 emitDecodeByteShiftedImm(const emitter::byteShiftedImm bsImm, emit
 
 /************************************************************************
 *
-* This union is used to to encode/decode the special ARM64 immediate values
+* This union is used to encode/decode the special ARM64 immediate values
 * that are use for FMOV immediate and referred to as 'float 8-bit immediate'
 */
 
@@ -217,7 +217,7 @@ static double emitDecodeFloatImm8(const emitter::floatImm8 fpImm);
 
 /************************************************************************
 *
-*  This union is used to to encode/decode the cond, nzcv and imm5 values for
+*  This union is used to encode/decode the cond, nzcv and imm5 values for
 *   instructions that use them in the small constant immediate field
 */
 

src/coreclr/jit/morph.cpp

@@ -5651,7 +5651,7 @@ bool Compiler::fgCanFastTailCall(GenTreeCall* callee, const char** failReason)
 #ifdef TARGET_ARM
         if (arg.AbiInfo.IsSplit())
         {
-            reportFastTailCallDecision("Splitted argument in callee is not supported on ARM32");
+            reportFastTailCallDecision("Argument splitting in callee is not supported on ARM32");
             return false;
         }
 #endif // TARGET_ARM
@@ -5662,7 +5662,7 @@ bool Compiler::fgCanFastTailCall(GenTreeCall* callee, const char** failReason)
 #ifdef TARGET_ARM
     if (compHasSplitParam)
     {
-        reportFastTailCallDecision("Splitted argument in caller is not supported on ARM32");
+        reportFastTailCallDecision("Argument splitting in caller is not supported on ARM32");
         return false;
     }
 

src/coreclr/jit/scopeinfo.cpp

@@ -359,7 +359,7 @@ void CodeGenInterface::siVarLoc::siFillStackVarLoc(
 //
 // Notes:
 //    The "varLoc" argument is filled depending of the "type" argument but as a VLT_REG... variation.
-//    "baseReg" and "offset" are used .for not 64 bit and values that are splitted in two parts.
+//    "baseReg" and "offset" are used .for not 64 bit and values that are split in two parts.
 void CodeGenInterface::siVarLoc::siFillRegisterVarLoc(
     const LclVarDsc* varDsc, var_types type, regNumber baseReg, int offset, bool isFramePointerUsed)
 {

src/coreclr/md/compiler/emit.cpp

@@ -1312,7 +1312,7 @@ STDMETHODIMP RegMeta::SetParent(                      // S_OK or error.
 
     IfFailGo(m_pStgdb->m_MiniMd.GetMemberRefRecord(RidFromToken(mr), &pMemberRef));
 
-    // If the token is nil set it to to m_tdModule.
+    // If the token is nil set it to m_tdModule.
     tk = IsNilToken(tk) ? m_tdModule : tk;
 
     // Set the parent.

src/coreclr/pal/inc/pal.h

@@ -3637,7 +3637,7 @@ to the variable that 'Addend' points to.
 Parameters
 
 lpAddend
-[in/out] Pointer to the variable to to added.
+[in/out] Pointer to the variable to added.
 
 Return Values
 

src/coreclr/pal/src/synchmgr/synchmanager.cpp

@@ -613,7 +613,7 @@ namespace CorUnix
             // Signal the object and trigger thread awakening
             psdSynchData->Signal(pthrCurrent, 1, false);
 
-            // Release reference to to SynchData
+            // Release reference to SynchData
             psdSynchData->Release(pthrCurrent);
 
             // Return node to the cache
@@ -1500,7 +1500,7 @@ namespace CorUnix
 
         if ((NULL == pSynchManager) || ((LONG)SynchMgrStatusRunning != s_lInitStatus))
         {
-            ERROR("Trying to to create worker thread in invalid state\n");
+            ERROR("Trying to create worker thread in invalid state\n");
             return ERROR_INTERNAL_ERROR;
         }
 

src/coreclr/tools/aot/ILCompiler.ReadyToRun/Compiler/CompilationModuleGroup.ReadyToRun.cs

@@ -79,7 +79,7 @@ partial class CompilationModuleGroup
         /// </summary>
         /// <param name="type">Type to get a module token for</param>
         /// <param name="token">Module-based token for the type</param>
-        /// <returns>Returns true the type was referenced by any of the input modules in the current compliation</returns>
+        /// <returns>Returns true when the type was referenced by any of the input modules in the current compilation</returns>
         public abstract bool TryGetModuleTokenForExternalType(TypeDesc type, out ModuleToken token);
 
         /// <summary>

src/coreclr/vm/eetwain.cpp

@@ -5616,7 +5616,7 @@ GenericParamContextType EECodeManager::GetParamContextType(PREGDISPLAY     pCont
 
 /*****************************************************************************
  *
- *  Returns the extra argument passed to to shared generic code if it is still alive.
+ *  Returns the extra argument passed to shared generic code if it is still alive.
  *  Returns NULL in all other cases.
  */
 PTR_VOID EECodeManager::GetParamTypeArg(PREGDISPLAY     pContext,

src/coreclr/vm/multicorejit.cpp

@@ -187,7 +187,7 @@ HRESULT WriteData(IStream * pStream, const void * pData, unsigned len)
     return hr;
 }
 
-// Write string, round to to DWORD alignment
+// Write string, round to DWORD alignment
 HRESULT WriteString(const void * pString, unsigned len, IStream * pStream)
 {
     CONTRACTL

src/coreclr/vm/proftoeeinterfaceimpl.cpp

@@ -8720,7 +8720,7 @@ HRESULT ProfToEEInterfaceImpl::DoStackSnapshot(ThreadID thread,
         // If the profiler did not specify a seed context of its own, use the current one we
         // just produced.
         //
-        // Failing to seed the walk can cause us to to "miss" functions on the stack.  This is
+        // Failing to seed the walk can cause us to "miss" functions on the stack.  This is
         // because StackWalkFrames(), when doing an unseeded stackwalk, sets the
         // starting regdisplay's IP/SP to 0.  This, in turn causes StackWalkFramesEx
         // to set cf.isFrameless = (pEEJM != NULL); (which is FALSE, since we have no

src/coreclr/vm/threads.cpp

@@ -2526,7 +2526,7 @@ int Thread::DecExternalCount(BOOL holdingLock)
             CONTRACT_VIOLATION(ModeViolation);
 
             // Clear the handle and leave the lock.
-            // We do not have to to DisablePreemptiveGC here, because
+            // We do not have to DisablePreemptiveGC here, because
             // we just want to put NULL into a handle.
             StoreObjectInHandle(m_StrongHndToExposedObject, NULL);
 

src/coreclr/vm/virtualcallstub.cpp

@@ -1996,7 +1996,7 @@ PCODE VirtualCallStubManager::ResolveWorker(StubCallSite* pCallSite,
 
 /*
 Resolve the token in the context of the method table, and set the target to point to
-the address that we should go to to get to the implementation.  Return a boolean indicating
+the address that we should go to get to the implementation.  Return a boolean indicating
 whether or not this is a permenent choice or a temporary choice.  For example, if the code has
 not been jitted yet, return FALSE and set the target to the prestub.  If the target is set to NULL,
 it means that the token is not resolvable.
@@ -2399,7 +2399,7 @@ VirtualCallStubManager::GetTarget(
 //----------------------------------------------------------------------------
 /*
 Resolve the token in the context of the method table, and set the target to point to
-the address that we should go to to get to the implementation.  Return a boolean indicating
+the address that we should go to get to the implementation.  Return a boolean indicating
 whether or not this is a permenent choice or a temporary choice.  For example, if the code has
 not been jitted yet, return FALSE and set the target to the prestub.  If the target is set to NULL,
 it means that the token is not resolvable.

src/libraries/Microsoft.VisualBasic.Core/src/Microsoft/VisualBasic/CompilerServices/ConversionResolution.vb

@@ -1040,7 +1040,7 @@ Namespace Microsoft.VisualBasic.CompilerServices
                 End Sub
 
                 Friend Sub New(ByVal size As Integer)
-                    'Populate the cache list with the maximum number of entires.
+                    'Populate the cache list with the maximum number of entries.
                     'This simplifies the insertion code for a small upfront cost.
                     _size = size
 
@@ -1143,7 +1143,7 @@ Namespace Microsoft.VisualBasic.CompilerServices
                 End Sub
 
                 Friend Sub New(ByVal size As Integer)
-                    'Populate the list with the maximum number of entires.
+                    'Populate the list with the maximum number of entries.
                     'This simplifies the insertion code for a small upfront cost.
                     _size = size
 

src/libraries/System.ComponentModel.Composition/src/System/ComponentModel/Composition/Hosting/AtomicComposition.cs

@@ -14,7 +14,7 @@ namespace System.ComponentModel.Composition.Hosting
     /// queryable state along with a sequence of actions necessary to complete the state when
     /// the atomicComposition is no longer in danger of being rolled back.  State is completed or
     /// rolled back when the atomicComposition is disposed, depending on the state of the
-    /// CompleteOnDipose property which defaults to false.  The using(...) pattern in C# is a
+    /// CompleteOnDispose property which defaults to false.  The using(...) pattern in C# is a
     /// convenient mechanism for defining atomicComposition scopes.
     ///
     /// The least obvious aspects of AtomicComposition deal with nesting.

src/libraries/System.DirectoryServices.AccountManagement/src/System/DirectoryServices/AccountManagement/AD/ADDNLinkedAttrSet.cs

@@ -257,7 +257,7 @@ private bool GetNextSearchResult()
                 memberFound = _memberSearchResultsEnumerator.MoveNext();
 
                 // The search is complete.
-                // Dipose the searcher and search results.
+                // Dispose the searcher and search results.
                 if (!memberFound)
                 {
                     _currentMembersSearcher.Dispose();

src/libraries/System.DirectoryServices.AccountManagement/src/System/DirectoryServices/AccountManagement/AD/SDSCache.cs

@@ -12,7 +12,7 @@
 namespace System.DirectoryServices.AccountManagement
 {
     /// <summary>
-    /// This is a class designed to cache DirectoryEntires instead of creating them every time.
+    /// This is a class designed to cache DirectoryEntries instead of creating them every time.
     /// </summary>
     internal sealed class SDSCache
     {

src/libraries/System.Drawing.Common/src/System/Drawing/Graphics.cs

@@ -3297,7 +3297,7 @@ private void PopContext(int currentContextState)
                 {
                     _previousContext = context.Previous;
 
-                    // This will dipose all context object up the stack.
+                    // This will dispose all context object up the stack.
                     context.Dispose();
                     return;
                 }