More spelling fixes

src/Compilers/Core/CodeAnalysisTest/AsyncQueueTests.cs

@@ -188,7 +188,7 @@ public async Task TaskCompletesAsyncWithComplete()
             await queue.WhenCompletedTask.ConfigureAwait(false);
 
             // The AsyncQueue<T>.Task property won't complete until all of the 
-            // exitsing DequeueAsync values have also completed.
+            // existing DequeueAsync values have also completed.
             Assert.True(task.IsCompleted);
         }
 

src/Compilers/Core/CodeAnalysisTest/Text/TextSpanTest.cs

@@ -139,7 +139,7 @@ public void TextSpanContains01()
             TextSpan span_10_11 = new TextSpan(10, 1);
             TextSpan span_00_03 = new TextSpan(0, 3);
 
-            // nonoverlapping
+            // non-overlapping
             Assert.False(span_05_15.Contains(span_00_03));
             Assert.False(span_00_03.Contains(span_05_15));
 
@@ -157,7 +157,7 @@ public void TextSpanContains01()
         [Fact]
         public void TextSpanContainsEmpty()
         {
-            // nonoverlapping
+            // non-overlapping
             Assert.False(new TextSpan(2, 5).Contains(new TextSpan(0, 0)));
             Assert.False(new TextSpan(2, 5).Contains(new TextSpan(10, 0)));
 
@@ -177,7 +177,7 @@ public void TextSpanContainsEmpty()
         [Fact]
         public void TextSpanEmptyContains()
         {
-            // nonoverlapping
+            // non-overlapping
             Assert.False(new TextSpan(0, 0).Contains(new TextSpan(2, 5)));
             Assert.False(new TextSpan(10, 0).Contains(new TextSpan(2, 5)));
 

src/Compilers/Core/CodeAnalysisTest/VersionHelperTests.cs

@@ -19,7 +19,7 @@ public void ParseGood()
             Assert.True(VersionHelper.TryParseAssemblyVersion("3.2.*", allowWildcard: true, version: out version));
             Assert.Equal(3, version.Major);
             Assert.Equal(2, version.Minor);
-            //number of days since jan 1, 2000
+            //number of days since Jan 1, 2000
             Assert.Equal((int)(DateTime.Now - new DateTime(2000, 1, 1)).TotalDays, version.Build);
             //number of seconds since midnight divided by two
             int s = (int)DateTime.Now.TimeOfDay.TotalSeconds / 2;

src/Compilers/Core/MSBuildTask/BuildClient.cs

@@ -192,7 +192,7 @@ private static int HandleResponse(BuildResponse response, string clientDir, stri
             // they are show-stoppers then they will crash the in-proc
             // compilation as well
             // TODO: Put in non-fatal Watson code so we still get info
-            // when things unexpectedely fail
+            // when things unexpectedly fail
             catch { }
             return Task.FromResult<BuildResponse>(null);
         }
@@ -221,7 +221,7 @@ private static int HandleResponse(BuildResponse response, string clientDir, stri
                     return null;
                 }
 
-                // Wait for the compilation and a monitor to dectect if the server disconnects
+                // Wait for the compilation and a monitor to detect if the server disconnects
                 var serverCts = CancellationTokenSource.CreateLinkedTokenSource(cancellationToken);
 
                 Log("Begin reading response");
@@ -332,7 +332,7 @@ private static int HandleResponse(BuildResponse response, string clientDir, stri
                     return pipe;
                 }
 
-                // Append an integer counter to the pipename
+                // Append an integer counter to the pipe name
                 pipeName = basePipeName + "." + counter.ToString(CultureInfo.InvariantCulture);
             }
             newPipeName = pipeName;

src/Compilers/Core/MSBuildTask/CanonicalError.cs

@@ -293,7 +293,7 @@ internal static Parts Parse(string message)
             //      -------------- ------------ ------- ------  ----------------------------------------------
             //      Origin         SubCategory  Cat.    Code    Text
             // 
-            // To accomodate absolute filenames in Origin, tolerate a colon in the second position
+            // To accommodate absolute filenames in Origin, tolerate a colon in the second position
             // as long as its preceded by a letter.
             //
             // Localization Note:

src/Compilers/Core/MSBuildTask/ManagedCompiler.cs

@@ -641,7 +641,7 @@ private void AddAdditionalFilesToCommandLine(CommandLineBuilderExtension command
         }
 
         /// <summary>
-        /// Configure the debug switches which will be placed on the compiler commandline.
+        /// Configure the debug switches which will be placed on the compiler command-line.
         /// The matrix of debug type and symbol inputs and the desired results is as follows:
         /// 
         /// Debug Symbols              DebugType   Desired Results

src/Compilers/Core/MSBuildTask/RCWForCurrentContext.cs

@@ -31,7 +31,7 @@ internal class RCWForCurrentContext<T> : IDisposable where T : class
         public RCWForCurrentContext(T rcw)
         {
             // To improve performance we create a new RCW for the current context so we get 
-            // the caching behaviour of the marshaled pointer. 
+            // the caching behavior of the marshaled pointer. 
             // See RCW::GetComIPForMethodTableFromCache in ndp\clr\src\VM\RuntimeCallableWrapper.cpp
             IntPtr iunknownPtr = Marshal.GetIUnknownForObject(rcw);
             Object objInCurrentCtx = null;

src/Compilers/Core/MSBuildTask/Vbc.cs

@@ -757,7 +757,7 @@ internal void ParseVBErrorOrWarning(string singleLine, MessageImportance message
         /// <summary>
         /// Many VisualStudio VB projects have values for the DefineConstants property that
         /// contain quotes and spaces.  Normally we don't allow parameters passed into the
-        /// task to contain quotes, because if we weren't careful, we might accidently
+        /// task to contain quotes, because if we weren't careful, we might accidentally
         /// allow a parameter injection attach.  But for "DefineConstants", we have to allow
         /// it.
         /// So this method prepares the string to be passed in on the /define: command-line

src/Compilers/Core/SharedCollections/ArrayBuilder.cs

@@ -244,7 +244,7 @@ public void Free()
                 // We do not want to retain (potentially indefinitely) very large builders 
                 // while the chance that we will need their size is diminishingly small.
                 // It makes sense to constrain the size to some "not too small" number. 
-                // Overal perf does not seem to be very sensitive to this number, so I picked 128 as a limit.
+                // Overall perf does not seem to be very sensitive to this number, so I picked 128 as a limit.
                 if (this.Count < 128)
                 {
                     if (this.Count != 0)

src/Compilers/Core/VBCSCompiler/BuildProtocol.cs

@@ -441,7 +441,7 @@ public enum RequestLanguage
             VisualBasicCompile = 0x44532522,
         }
 
-        // Arugments for CSharp and VB Compiler
+        // Arguments for CSharp and VB Compiler
         public enum ArgumentId
         {
             // The current directory of the client

src/Compilers/Core/VBCSCompilerTests/CompilerServerTests.cs

@@ -444,7 +444,7 @@ public void CompilerBinariesAreNotX86()
         /// <summary>
         /// This method tests that when a 64-bit compiler server loads a 
         /// 64-bit mscorlib with /platform:x86 enabled no warning about
-        /// emitting a refence to a 64-bit assembly is produced.
+        /// emitting a reference to a 64-bit assembly is produced.
         /// The test should pass on x86 or amd64, but can only fail on
         /// amd64.
         /// </summary>
@@ -1298,7 +1298,7 @@ public void SimpleMSBuild()
             using (var resultFile = GetResultFile(_tempDirectory, @"bin\debug\helloproj.exe"))
             {
                 // once we stop issuing BC40998 (NYI), we can start making stronger assertions
-                // about our ouptut in the general case
+                // about our output in the general case
                 if (result.ExitCode != 0)
                 {
                     Assert.Equal("", result.Output);

src/Test/PdbUtilities/Pdb/Token2SourceLineExporter.cs

@@ -381,7 +381,7 @@ private static int GetPrime(int minSize)
             // Deleted entries have their key set to buckets
 
             // The hash table data.
-            // This cannot be serialised
+            // This cannot be serialized
             private struct bucket
             {
                 internal int key;

src/Test/PdbUtilities/Shared/CustomDebugInfoReader.cs

@@ -621,7 +621,7 @@ public static bool TryParseCSharpImportString(string import, out string alias, o
 
                 case 'E': // C# (namespace) using
                     // NOTE: Dev12 has related cases "I" and "O" in EMITTER::ComputeDebugNamespace,
-                    // but they were probably implementation details that do not affect roslyn.
+                    // but they were probably implementation details that do not affect Roslyn.
                     if (!TrySplit(import, 1, ' ', out target, out externAlias))
                     {
                         return false;

src/Test/Utilities/CLRHelpers.cs

@@ -26,7 +26,7 @@ static CLRHelpers()
         {
             // Work around CLR bug: 
             // PE Verifier adds a handler to ReflectionOnlyAssemblyResolve event in AppDomain.EnableResolveAssembliesForIntrospection
-            // (called from ValidateWorker in Validator.cpp) i nwhich it directly calls Assembly.ReflectionOnlyLoad.
+            // (called from ValidateWorker in Validator.cpp) in which it directly calls Assembly.ReflectionOnlyLoad.
             // If that happens before we get a chance to resolve the assembly the resolution fails.
             // 
             // The handlers are invoked in the order they were added until one of them returns non-null assembly.

src/Test/Utilities/ConditionalFactAttribute.cs

@@ -81,7 +81,7 @@ private static string GetSkipReason(ClrOnlyReason reason)
             switch (reason)
             {
                 case ClrOnlyReason.Ilasm:
-                    return "Mono ilasm doesn't suupport all of the features we need";
+                    return "Mono ilasm doesn't support all of the features we need";
                 case ClrOnlyReason.MemberOrder:
                     return "Mono returns certain symbols in different order than we are expecting";
                 case ClrOnlyReason.Pdb:

src/Test/Utilities/HostedRuntimeEnvironment.cs

@@ -1006,7 +1006,7 @@ internal static class SerializationInfoExtensions
         public static void AddArray<T>(this SerializationInfo info, string name, ImmutableArray<T> value) where T : class
         {
             // we will copy the content into an array and serialize the copy
-            // we could serialize elementwise, but that would require serializing
+            // we could serialize element-wise, but that would require serializing
             // name and type for every serialized element which seems worse than creating a copy.
             info.AddValue(name, value.IsDefault ? null : value.ToArray(), typeof(T[]));
         }
@@ -1020,7 +1020,7 @@ internal static class SerializationInfoExtensions
         public static void AddByteArray(this SerializationInfo info, string name, ImmutableArray<byte> value)
         {
             // we will copy the content into an array and serialize the copy
-            // we could serialize elementwise, but that would require serializing
+            // we could serialize element-wise, but that would require serializing
             // name and type for every serialized element which seems worse than creating a copy.
             info.AddValue(name, value.IsDefault ? null : value.ToArray(), typeof(byte[]));
         }

src/Test/Utilities/MarkupTestFile.cs

@@ -15,7 +15,7 @@ namespace Roslyn.Test.Utilities
     /// To aid with testing, we define a special type of text file that can encode additional
     /// information in it.  This prevents a test writer from having to carry around multiple sources
     /// of information that must be reconstituted.  For example, instead of having to keep around the
-    /// contents of a file *and* and the location of of the cursor, the tester can just provide a
+    /// contents of a file *and* and the location of the cursor, the tester can just provide a
     /// string with the "$" character in it.  This allows for easy creation of "FIT" tests where all
     /// that needs to be provided are strings that encode every bit of state necessary in the string
     /// itself.