Fix typos (dotnet#2065)

* begining -> beginning

* paramterized -> parameterized

* folow -> follow

* occurance -> occurrence

* Fix bad changes

* Apply suggestion + minor space fix

src/coreclr/src/ToolBox/superpmi/superpmi-shared/icorjitinfoimpl.h

@@ -901,7 +901,7 @@ InfoAccessType constructStringLiteral(CORINFO_MODULE_HANDLE module, mdToken meta
 InfoAccessType emptyStringLiteral(void** ppValue);
 
 // (static fields only) given that 'field' refers to thread local store,
-// return the ID (TLS index), which is used to find the begining of the
+// return the ID (TLS index), which is used to find the beginning of the
 // TLS data area for the particular DLL 'field' is associated with.
 DWORD getFieldThreadLocalStoreID(CORINFO_FIELD_HANDLE field, void** ppIndirection = NULL);
 

src/coreclr/src/ToolBox/superpmi/superpmi-shim-collector/icorjitinfo.cpp

@@ -1896,7 +1896,7 @@ InfoAccessType interceptor_ICJI::emptyStringLiteral(void** ppValue)
 }
 
 // (static fields only) given that 'field' refers to thread local store,
-// return the ID (TLS index), which is used to find the begining of the
+// return the ID (TLS index), which is used to find the beginning of the
 // TLS data area for the particular DLL 'field' is associated with.
 DWORD interceptor_ICJI::getFieldThreadLocalStoreID(CORINFO_FIELD_HANDLE field, void** ppIndirection)
 {

src/coreclr/src/ToolBox/superpmi/superpmi-shim-counter/icorjitinfo.cpp

@@ -1464,7 +1464,7 @@ InfoAccessType interceptor_ICJI::emptyStringLiteral(void** ppValue)
 }
 
 // (static fields only) given that 'field' refers to thread local store,
-// return the ID (TLS index), which is used to find the begining of the
+// return the ID (TLS index), which is used to find the beginning of the
 // TLS data area for the particular DLL 'field' is associated with.
 DWORD interceptor_ICJI::getFieldThreadLocalStoreID(CORINFO_FIELD_HANDLE field, void** ppIndirection)
 {

src/coreclr/src/ToolBox/superpmi/superpmi-shim-simple/icorjitinfo.cpp

@@ -1311,7 +1311,7 @@ InfoAccessType interceptor_ICJI::emptyStringLiteral(void** ppValue)
 }
 
 // (static fields only) given that 'field' refers to thread local store,
-// return the ID (TLS index), which is used to find the begining of the
+// return the ID (TLS index), which is used to find the beginning of the
 // TLS data area for the particular DLL 'field' is associated with.
 DWORD interceptor_ICJI::getFieldThreadLocalStoreID(CORINFO_FIELD_HANDLE field, void** ppIndirection)
 {

src/coreclr/src/ToolBox/superpmi/superpmi/icorjitinfo.cpp

@@ -1569,7 +1569,7 @@ InfoAccessType MyICJI::emptyStringLiteral(void** ppValue)
 }
 
 // (static fields only) given that 'field' refers to thread local store,
-// return the ID (TLS index), which is used to find the begining of the
+// return the ID (TLS index), which is used to find the beginning of the
 // TLS data area for the particular DLL 'field' is associated with.
 DWORD MyICJI::getFieldThreadLocalStoreID(CORINFO_FIELD_HANDLE field, void** ppIndirection)
 {

src/coreclr/src/debug/di/rspriv.h

@@ -4786,7 +4786,7 @@ class CordbType : public CordbBase, public ICorDebugType, public ICorDebugType2
                           CordbType ** ppResultType);
 
     // Some derived constructors...  Use this one if the type is definitely not
-    // a paramterized type, e.g. to implement functions on the API where types cannot
+    // a parameterized type, e.g. to implement functions on the API where types cannot
     // be parameterized.
     static HRESULT MkUnparameterizedType(CordbAppDomain *appdomain, CorElementType et, CordbClass *cl, CordbType **ppType);
 

src/coreclr/src/debug/ildbsymlib/symread.cpp

@@ -233,7 +233,7 @@ HRESULT SymReader::InitializeFromStream(
     BYTE *pSignature;
     HRESULT hr = S_OK;
 
-    // Reset the stream to the begining
+    // Reset the stream to the beginning
     LARGE_INTEGER li;
     li.u.HighPart = 0;
     li.u.LowPart = 0;

src/coreclr/src/inc/gcdecoder.cpp

@@ -530,7 +530,7 @@ const unsigned callCommonDelta[4] = { 6,8,10,12 };
  *
  *  Note that ARG_MASK is the mask of pushed args that contain GC pointers
  *   since the first two arguments are always passed in registers it is
- *   a fairly rare occurance to push a GC pointer as an argument, since it
+ *   a fairly rare occurrence to push a GC pointer as an argument, since it
  *   only occurs for nested calls, when the third or later argument for the
  *   outer call contains a GC ref.
  *

src/coreclr/src/inc/sigparser.h

@@ -490,7 +490,7 @@ class SigParser
 
         //------------------------------------------------------------------------
         // Is this at the Sentinal (the ... in a varargs signature) that marks
-        // the begining of varguments that are not decared at the target
+        // the beginning of varguments that are not decared at the target
 
         bool AtSentinel() const
         {

src/coreclr/src/jit/codegencommon.cpp

@@ -7116,7 +7116,7 @@ void CodeGen::genFnProlog()
     if (compiler->compJitHaltMethod())
     {
         /* put a nop first because the debugger and other tools are likely to
-           put an int3 at the begining and we don't want to confuse them */
+           put an int3 at the beginning and we don't want to confuse them */
 
         instGen(INS_nop);
         instGen(INS_BREAKPOINT);

src/coreclr/src/jit/compiler.h

@@ -6153,9 +6153,9 @@ class Compiler
         unsigned csdDefWtCnt; // weighted def count
         unsigned csdUseWtCnt; // weighted use count  (excluding the implicit uses at defs)
 
-        GenTree*    csdTree;  // treenode containing the 1st occurance
-        Statement*  csdStmt;  // stmt containing the 1st occurance
-        BasicBlock* csdBlock; // block containing the 1st occurance
+        GenTree*    csdTree;  // treenode containing the 1st occurrence
+        Statement*  csdStmt;  // stmt containing the 1st occurrence
+        BasicBlock* csdBlock; // block containing the 1st occurrence
 
         treeStmtLst* csdTreeList; // list of matching tree nodes: head
         treeStmtLst* csdTreeLast; // list of matching tree nodes: tail

src/coreclr/src/jit/importer.cpp

@@ -13063,7 +13063,7 @@ void Compiler::impImportBlockCode(BasicBlock* block)
             {
                 if (tiVerificationNeeded)
                 {
-                    // Dup could start the begining of delegate creation sequence, remember that
+                    // Dup could start the beginning of delegate creation sequence, remember that
                     delegateCreateStart = codeAddr - 1;
                     impStackTop(0);
                 }
@@ -13384,7 +13384,7 @@ void Compiler::impImportBlockCode(BasicBlock* block)
 
                 if (tiVerificationNeeded)
                 {
-                    // LDFTN could start the begining of delegate creation sequence, remember that
+                    // LDFTN could start the beginning of delegate creation sequence, remember that
                     delegateCreateStart = codeAddr - 2;
 
                     // check any constraints on the callee's class and type parameters

src/coreclr/src/jit/morph.cpp

@@ -356,7 +356,7 @@ GenTree* Compiler::fgMorphCast(GenTree* tree)
     {
         // We are casting away GC information.  we would like to just
         // change the type to int, however this gives the emitter fits because
-        // it believes the variable is a GC variable at the begining of the
+        // it believes the variable is a GC variable at the beginning of the
         // instruction group, but is not turned non-gc by the code generator
         // we fix this by copying the GC pointer to a non-gc pointer temp.
         noway_assert(!varTypeIsGC(dstType) && "How can we have a cast to a GCRef here?");

src/coreclr/src/jit/valuenum.cpp

@@ -6975,7 +6975,7 @@ void Compiler::fgValueNumberTree(GenTree* tree)
 
                     // The ValueNum returned must represent the full-sized IL-Stack value
                     // If we need to widen this value then we need to introduce a VNF_Cast here to represent
-                    // the widened value.    This is necessary since the CSE package can replace all occurances
+                    // the widened value.    This is necessary since the CSE package can replace all occurrences
                     // of a given ValueNum with a LclVar that is a full-sized IL-Stack value
                     //
                     if (varTypeIsSmall(tree->TypeGet()))

src/coreclr/src/pal/tests/palsuite/c_runtime/strpbrk/test1/test1.cpp

@@ -8,7 +8,7 @@
 **
 ** Purpose:
 ** Search a string for characters in a given character set and ensure the 
-** pointer returned points to the first occurance.  Check to see that the
+** pointer returned points to the first occurrence.  Check to see that the
 ** function returns NULL if the character is not found.
 **
 **

src/coreclr/src/pal/tests/palsuite/c_runtime/strpbrk/test1/testinfo.dat

@@ -10,5 +10,5 @@ TYPE = DEFAULT
 EXE1 = test1
 Description
 = Search a string for characters in a given character set and ensure the 
-= pointer returned points to the first occurance.  Check to see that the
+= pointer returned points to the first occurrence.  Check to see that the
 = function returns NULL if the character is not found.

src/coreclr/src/pal/tests/palsuite/miscellaneous/InterlockedCompareExchange64/test2/test.cpp

@@ -9,7 +9,7 @@
 ** Purpose: InterlockedIncrement64() function
 **
 **	The test case spawns MAX_THREADS Threads, and each thread call InterlockedDecrement Function to decrement a 
-**	global counter REPEAT_COUNT Times. The Test case sets the global counter to Zero at the begining of the test.  
+**	global counter REPEAT_COUNT Times. The Test case sets the global counter to Zero at the beginning of the test.  
 **	The test cases passes if at the end the test the value of the global counter is  MAX_THREADS * REPEAT_COUNT.
 **
 **

src/coreclr/src/pal/tests/palsuite/miscellaneous/InterlockedDecrement/test2/test.cpp

@@ -10,7 +10,7 @@
 **
 **	The test case spawns MAX_THREADS Threads, and each thread call InterlockedDecrement Function to decrement a 
 **	global counter REPEAT_COUNT Times. The Test case sets the global counter to MAX_THREADS * REPEAT_COUNT 
-**	at the begining of the test.  The test cases passes if at the end the test the value of the global counter is Zero.
+**	at the beginning of the test.  The test cases passes if at the end the test the value of the global counter is Zero.
 **
 **
 **=========================================================*/

src/coreclr/src/pal/tests/palsuite/miscellaneous/InterlockedDecrement64/test2/test.cpp

@@ -10,7 +10,7 @@
 **
 **	The test case spawns MAX_THREADS Threads, and each thread call InterlockedDecrement Function to decrement a 
 **	global counter REPEAT_COUNT Times. The Test case sets the global counter to MAX_THREADS * REPEAT_COUNT 
-**	at the begining of the test.  The test cases passes if at the end the test the value of the global counter is Zero.
+**	at the beginning of the test.  The test cases passes if at the end the test the value of the global counter is Zero.
 **
 **
 **=========================================================*/

src/coreclr/src/pal/tests/palsuite/miscellaneous/InterlockedIncrement/test2/test.cpp

@@ -9,7 +9,7 @@
 ** Purpose: InterlockedIncrement() function
 **
 **	The test case spawns MAX_THREADS Threads, and each thread call InterlockedDecrement Function to decrement a 
-**	global counter REPEAT_COUNT Times. The Test case sets the global counter to Zero at the begining of the test.  
+**	global counter REPEAT_COUNT Times. The Test case sets the global counter to Zero at the beginning of the test.  
 **	The test cases passes if at the end the test the value of the global counter is  MAX_THREADS * REPEAT_COUNT.
 **
 **

src/coreclr/src/pal/tests/palsuite/miscellaneous/InterlockedIncrement64/test2/test.cpp

@@ -9,7 +9,7 @@
 ** Purpose: InterlockedIncrement64() function
 **
 **	The test case spawns MAX_THREADS Threads, and each thread call InterlockedDecrement Function to decrement a 
-**	global counter REPEAT_COUNT Times. The Test case sets the global counter to Zero at the begining of the test.  
+**	global counter REPEAT_COUNT Times. The Test case sets the global counter to Zero at the beginning of the test.  
 **	The test cases passes if at the end the test the value of the global counter is  MAX_THREADS * REPEAT_COUNT.
 **
 **

src/coreclr/src/palrt/path.cpp

@@ -214,7 +214,7 @@ LPCWSTR GetPCEnd(LPCWSTR lpszStart)
 
 //
 // Given a pointer to the end of a path component, return a pointer to
-// its begining.
+// its beginning.
 // ie return a pointer to the previous backslash (or start of the string).
 //
 LPCWSTR PCStart(LPCWSTR lpszStart, LPCWSTR lpszEnd)

src/coreclr/src/tools/crossgen2/ILCompiler.ReadyToRun/IBC/IBCDataModel.cs

@@ -436,7 +436,7 @@ public enum SectionFormat : int
     public enum BlobType : int
     {
         // IMPORTANT: Keep the first four enums together in the same order and at the
-        //  very begining of this enum.  See MetaModelPub.h for the order
+        //  very beginning of this enum.  See MetaModelPub.h for the order
 
         MetadataStringPool = 0,
         MetadataGuidPool = 1,

src/coreclr/src/tools/crossgen2/ILCompiler.Reflection.ReadyToRun/ReadyToRunHeader.cs

@@ -21,7 +21,7 @@ public class ReadyToRunHeader
         public const uint READYTORUN_SIGNATURE = 0x00525452; // 'RTR'
 
         /// <summary>
-        /// RVA to the begining of the ReadyToRun header
+        /// RVA to the beginning of the ReadyToRun header
         /// </summary>
         public int RelativeVirtualAddress { get; set; }
 

src/coreclr/src/utilcode/util.cpp

@@ -1041,7 +1041,7 @@ DWORD LCM(DWORD u, DWORD v)
     CONTRACTL_END;
 
     // CPUGroupInfo needs to be initialized only once. This could happen in three cases
-    // 1. CLR initialization at begining of EEStartup, or
+    // 1. CLR initialization at beginning of EEStartup, or
     // 2. Sometimes, when hosted by ASP.NET, the hosting process may initialize ThreadPool
     //    before initializing CLR, thus require CPUGroupInfo to be initialized to determine
     //    if CPU group support should/could be enabled.

src/coreclr/src/vm/codeman.cpp

@@ -5231,7 +5231,7 @@ PCODE ExecutionManager::getNextJumpStub(MethodDesc* pMD, PCODE target,
 
             if ((m_LCG_JumpStubLookup > 0) && ((m_normal_JumpStubBlockFullCount % 5) == 1))
             {
-                // Every 5 occurance of the above we also
+                // Every 5 occurrence of the above we also
                 // Log "LCG JumpStubBlock status" along with the four counter values
                 STRESS_LOG4(LF_JIT, LL_INFO1000, "LCG JumpStubBlock status - (%u, %u, %u, %u)\n",
                             m_LCG_JumpStubLookup, m_LCG_JumpStubUnique,

src/coreclr/src/vm/comcache.cpp

@@ -197,7 +197,7 @@ HRESULT wCoMarshalInterThreadInterfaceInStream(REFIID riid,
 
     if (SUCCEEDED(hr))
     {
-        // Reset the stream to the begining
+        // Reset the stream to the beginning
         LARGE_INTEGER li;
         LISet32(li, 0);
         ULARGE_INTEGER li2;
@@ -707,7 +707,7 @@ IUnknown* IUnkEntry::UnmarshalIUnknownForCurrContext()
                 }
                 else
                 {
-                    // Reset the stream to the begining
+                    // Reset the stream to the beginning
                     LARGE_INTEGER li;
                     LISet32(li, 0);
                     ULARGE_INTEGER li2;
@@ -734,7 +734,7 @@ IUnknown* IUnkEntry::UnmarshalIUnknownForCurrContext()
                     }
                     else
                     {
-                        // Reset the stream to the begining
+                        // Reset the stream to the beginning
                         LISet32(li, 0);
                         m_pStream->Seek(li, STREAM_SEEK_SET, &li2);
 

src/coreclr/src/vm/comutilnative.cpp

@@ -846,7 +846,7 @@ FCIMPL0(UINT64, GCInterface::GetSegmentSize)
 FCIMPLEND
 
 /*================================CollectionCount=================================
-**Action: Returns the number of collections for this generation since the begining of the life of the process
+**Action: Returns the number of collections for this generation since the beginning of the life of the process
 **Returns: The collection count.
 **Arguments: args->generation -- The generation
 **Exceptions: Argument exception if args->generation is < 0 or > GetMaxGeneration();

src/coreclr/src/vm/eventtrace.cpp

@@ -5707,7 +5707,7 @@ VOID ETW::EnumerationLog::ProcessShutdown()
 
 /****************************************************************************/
 /****************************************************************************/
-/* Begining of helper functions */
+/* beginning of helper functions */
 /****************************************************************************/
 /****************************************************************************/
 

src/coreclr/src/vm/fcall.h

@@ -852,7 +852,7 @@ class FCallCheck : public ForbidGC {
 #endif
     bool          didGCPoll;            // GC poll was done
     bool          notNeeded;            // GC poll not needed
-    unsigned __int64 startTicks;        // tick count at begining of FCall
+    unsigned __int64 startTicks;        // tick count at beginning of FCall
 };
 
         // FC_COMMON_PROLOG is used for both FCalls and HCalls

src/coreclr/src/vm/frames.h

@@ -1018,7 +1018,7 @@ class TransitionFrame : public Frame
 // TransitionFrames for exceptions
 //-----------------------------------------------------------------------
 
-// The define USE_FEF controls how this class is used.  Look for occurances
+// The define USE_FEF controls how this class is used.  Look for occurrences
 //  of USE_FEF.
 
 class FaultingExceptionFrame : public Frame

src/coreclr/src/vm/gcenv.ee.cpp

@@ -914,7 +914,7 @@ void GCToEEInterface::StompWriteBarrier(WriteBarrierParameters* args)
         if (stompWBCompleteActions & SWB_EE_RESTART)
         {
             assert(!args->is_runtime_suspended &&
-                "if runtime was suspended in patching routines then it was in running state at begining");
+                "if runtime was suspended in patching routines then it was in running state at beginning");
             ThreadSuspend::RestartEE(FALSE, TRUE);
         }
         return; // unlike other branches we have already done cleanup so bailing out here
@@ -990,7 +990,7 @@ void GCToEEInterface::StompWriteBarrier(WriteBarrierParameters* args)
     if (stompWBCompleteActions & SWB_EE_RESTART)
     {
         assert(!args->is_runtime_suspended &&
-            "if runtime was suspended in patching routines then it was in running state at begining");
+            "if runtime was suspended in patching routines then it was in running state at beginning");
         ThreadSuspend::RestartEE(FALSE, TRUE);
     }
 }

src/coreclr/src/vm/ibclogger.h

@@ -323,7 +323,7 @@ class IBCLoggingDisabler
 private:
     ThreadLocalIBCInfo* m_pInfo;
     bool                m_fDisabled;  // true if this holder actually disable the logging
-                                      // false when this is a nested occurance and logging was already disabled
+                                      // false when this is a nested occurrence and logging was already disabled
 };
 
 //

src/coreclr/src/vm/interoputil.cpp

@@ -2427,7 +2427,7 @@ void SafeReleaseStream(IStream *pStream)
         LogInterop(logStr);
         if (hr != S_OK)
         {
-            // Reset the stream to the begining
+            // Reset the stream to the beginning
             LARGE_INTEGER li;
             LISet32(li, 0);
             ULARGE_INTEGER li2;

src/coreclr/src/vm/object.h

@@ -620,7 +620,7 @@ class ArrayBase : public Object
         return(GetMethodTable()->IsMultiDimArray());
     }
 
-        // Get pointer to the begining of the bounds (counts for each dim)
+        // Get pointer to the beginning of the bounds (counts for each dim)
         // Works for any array type
     PTR_INT32 GetBoundsPtr() const {
         WRAPPER_NO_CONTRACT;

src/coreclr/src/vm/proftoeeinterfaceimpl.cpp

@@ -8386,7 +8386,7 @@ HRESULT ProfToEEInterfaceImpl::DoStackSnapshotHelper(Thread * pThreadToSnapshot,
         if ((pParam->pData->infoFlags & COR_PRF_SNAPSHOT_X86_OPTIMIZED) != 0)
         {
 #ifndef _TARGET_X86_
-            // If check in the begining of DoStackSnapshot (to return E_INVALIDARG) should
+            // If check in the beginning of DoStackSnapshot (to return E_INVALIDARG) should
             // make this unreachable
             _ASSERTE(!"COR_PRF_SNAPSHOT_X86_OPTIMIZED on non-X86 should be unreachable!");
 #else

src/coreclr/src/vm/threadpoolrequest.h

@@ -45,7 +45,7 @@ class IPerAppDomainTPCount{
     virtual void ResetState() = 0;
     virtual BOOL IsRequestPending() = 0;
 
-	//This functions marks the begining of requests queued for the domain.
+    //This functions marks the beginning of requests queued for the domain.
     //It needs to notify the scheduler of work-arrival among other things.
     virtual void SetAppDomainRequestsActive() = 0;