From dd31d19871d914e1894104ace14978abecf1cf9f Mon Sep 17 00:00:00 2001 From: Pascal Borreli Date: Wed, 18 Mar 2015 21:00:39 +0000 Subject: [PATCH] Fixed typos --- README.md | 2 +- src/Framework/ProjectStartedEventArgs.cs | 2 +- src/Shared/INodePacketTranslator.cs | 2 +- src/Shared/UnitTests/MockLogger.cs | 2 +- src/Utilities/MuxLogger.cs | 8 ++++---- src/Utilities/UnitTests/MockLogger.cs | 2 +- .../BuildRequestEngine/BuildRequestEntry.cs | 2 +- .../Components/Communications/NodeManager.cs | 2 +- .../Components/Logging/EventSourceSink.cs | 2 +- .../Components/Logging/ILoggingService.cs | 2 +- .../BackEnd/Components/Logging/LoggingService.cs | 16 ++++++++-------- .../Logging/LoggingServiceLogMethods.cs | 2 +- .../RequestBuilder/IRequestBuilderCallback.cs | 2 +- .../RequestBuilder/ITargetBuilderCallback.cs | 2 +- .../BackEnd/Components/Scheduler/Scheduler.cs | 2 +- .../Solution/SolutionConfigurationInSolution.cs | 2 +- .../Definition/ProjectCollection.cs | 2 +- .../Conditionals/FunctionCallExpressionNode.cs | 2 +- .../Logging/NullCentralLogger.cs | 4 ++-- .../UnitTests/BackEnd/BuildManager_Tests.cs | 4 ++-- .../BackEnd/EventRedirectorToSink_Tests.cs | 2 +- .../UnitTests/BackEnd/EventSourceSink_Tests.cs | 2 +- .../BackEnd/LoggingServicesLogMethod_Tests.cs | 2 +- .../UnitTests/BackEnd/TaskHost_Tests.cs | 2 +- .../UnitTests/QaTests/ITestDataProvider.cs | 2 +- .../UnitTests/QaTests/MockResultsCache.cs | 2 +- .../AssemblyDependency/NoMatchReason.cs | 2 +- src/XMakeTasks/CommandLineBuilderExtension.cs | 2 +- .../Microsoft.Build.Tasks.Suppressions.cs | 6 +++--- src/XMakeTasks/ManifestUtil/FileAssociation.cs | 2 +- .../UnitTests/ResolveAssemblyReference_Tests.cs | 6 +++--- src/XMakeTasks/Vbc.cs | 2 +- 32 files changed, 48 insertions(+), 48 deletions(-) diff --git a/README.md b/README.md index 586436e7ebe..0c808c1c3f1 100644 --- a/README.md +++ b/README.md @@ -19,7 +19,7 @@ or grab a [prebuilt Azure VM image](http://blogs.msdn.com/b/visualstudioalm/arch 3. Open src/MSBuild.sln solution in Visual Studio 2015. ## How to Engage, Contribute and Provide Feedback -Before you contribute, please read through the contributing and developer guides to get an idea of what kinds of pull requsts we will or won't accept. +Before you contribute, please read through the contributing and developer guides to get an idea of what kinds of pull requests we will or won't accept. * [Contributing Guide](https://github.com/Microsoft/msbuild/wiki/Contributing-Code) * [Developer Guide](https://github.com/Microsoft/msbuild/wiki/Building-Testing-and-Debugging) diff --git a/src/Framework/ProjectStartedEventArgs.cs b/src/Framework/ProjectStartedEventArgs.cs index 404ff7df3b1..d87c3aaf515 100644 --- a/src/Framework/ProjectStartedEventArgs.cs +++ b/src/Framework/ProjectStartedEventArgs.cs @@ -11,7 +11,7 @@ namespace Microsoft.Build.Framework { /// - /// Arguements for project started events + /// Arguments for project started events /// /// /// WARNING: marking a type [Serializable] without implementing diff --git a/src/Shared/INodePacketTranslator.cs b/src/Shared/INodePacketTranslator.cs index 93fb31a5ef6..d74a549d02c 100644 --- a/src/Shared/INodePacketTranslator.cs +++ b/src/Shared/INodePacketTranslator.cs @@ -18,7 +18,7 @@ namespace Microsoft.Build.BackEnd { /// - /// This delegate is used for objects which do not have public paramaterless constructors and must be constructed using + /// This delegate is used for objects which do not have public parameterless constructors and must be constructed using /// another method. When invoked, this delegate should return a new object which has been translated appropriately. /// /// The type to be translated. diff --git a/src/Shared/UnitTests/MockLogger.cs b/src/Shared/UnitTests/MockLogger.cs index 8bcfe4b9bf8..4adf0272765 100644 --- a/src/Shared/UnitTests/MockLogger.cs +++ b/src/Shared/UnitTests/MockLogger.cs @@ -283,7 +283,7 @@ public void Shutdown() /* * Method: LoggerEventHandler * - * Recieves build events and logs them the way we like. + * Receives build events and logs them the way we like. * */ internal void LoggerEventHandler(object sender, BuildEventArgs eventArgs) diff --git a/src/Utilities/MuxLogger.cs b/src/Utilities/MuxLogger.cs index f42a1f810c0..4013fb1ead2 100644 --- a/src/Utilities/MuxLogger.cs +++ b/src/Utilities/MuxLogger.cs @@ -15,12 +15,12 @@ namespace Microsoft.Build.Utilities { /// /// This is a multiplexing logger. The purpose of this logger is to allow the registration and deregistration of - /// multiple loggers during the build. This is to support the VS IDE scenario where loggers are regisered and unregistered + /// multiple loggers during the build. This is to support the VS IDE scenario where loggers are registered and unregistered /// for each project system's build request. This means one physical build may have multiple logical builds /// each with their own set of loggers. /// /// The Mux logger will register itself with the build manager as a regular central /l style logger. - /// It will be responsible for recieving messages from the build manager and route them to the correct + /// It will be responsible for receiving messages from the build manager and route them to the correct /// logger based on the logical build the message came from. /// /// Requirements: @@ -34,7 +34,7 @@ namespace Microsoft.Build.Utilities /// /// 3) The MUX logger will be shutdown when the build is finished in end build . At this time it will un-register any loggers attached to it. /// - /// 4) The MUX logger will log the build finsished event when the project finished event for the first project started event is seen for each logger. + /// 4) The MUX logger will log the build finished event when the project finished event for the first project started event is seen for each logger. /// /// Registering Loggers: /// @@ -46,7 +46,7 @@ namespace Microsoft.Build.Utilities /// UnRegisterLogger: /// When a build submission is completed the UnregisterLogger method will be called with the submission ID. /// At this point we will look up the success state of the project finished event for the submission ID and log a build finished event to the logger. - /// The event source will be cleaned up. This may be interesting because the unregister will come from a thread other than what is doign the logging. + /// The event source will be cleaned up. This may be interesting because the unregister will come from a thread other than what is doing the logging. /// This may create a Synchronization issue, if unregister is called while events are being logged. /// /// UNDONE: If we can use ErrorUtilities, replace all InvalidOperation and Argument exceptions with the appropriate calls. diff --git a/src/Utilities/UnitTests/MockLogger.cs b/src/Utilities/UnitTests/MockLogger.cs index 3c5c8ffb6f3..027d38700fc 100644 --- a/src/Utilities/UnitTests/MockLogger.cs +++ b/src/Utilities/UnitTests/MockLogger.cs @@ -201,7 +201,7 @@ public void Shutdown() /* * Method: LoggerEventHandler * - * Recieves build events and logs them the way we like. + * Receives build events and logs them the way we like. * */ internal void LoggerEventHandler(object sender, BuildEventArgs eventArgs) diff --git a/src/XMakeBuildEngine/BackEnd/Components/BuildRequestEngine/BuildRequestEntry.cs b/src/XMakeBuildEngine/BackEnd/Components/BuildRequestEngine/BuildRequestEntry.cs index 8a4e421ece9..f4691bcc77f 100644 --- a/src/XMakeBuildEngine/BackEnd/Components/BuildRequestEngine/BuildRequestEntry.cs +++ b/src/XMakeBuildEngine/BackEnd/Components/BuildRequestEngine/BuildRequestEntry.cs @@ -60,7 +60,7 @@ internal enum BuildRequestEntryState /// must be non-null. /// /// Transitions: - /// Ready: All of the results which caused the build request to wait have been recieved + /// Ready: All of the results which caused the build request to wait have been received /// Waiting, diff --git a/src/XMakeBuildEngine/BackEnd/Components/Communications/NodeManager.cs b/src/XMakeBuildEngine/BackEnd/Components/Communications/NodeManager.cs index ebdce676f3b..10a740d9068 100644 --- a/src/XMakeBuildEngine/BackEnd/Components/Communications/NodeManager.cs +++ b/src/XMakeBuildEngine/BackEnd/Components/Communications/NodeManager.cs @@ -307,7 +307,7 @@ static internal IBuildComponent CreateComponent(BuildComponentType type) } /// - /// We have recieved the node shutdown packet for this node, we should remove it from our list of providers. + /// We have received the node shutdown packet for this node, we should remove it from our list of providers. /// private void RemoveNodeFromMapping(int nodeId) { diff --git a/src/XMakeBuildEngine/BackEnd/Components/Logging/EventSourceSink.cs b/src/XMakeBuildEngine/BackEnd/Components/Logging/EventSourceSink.cs index 60401f8d498..034000a0663 100644 --- a/src/XMakeBuildEngine/BackEnd/Components/Logging/EventSourceSink.cs +++ b/src/XMakeBuildEngine/BackEnd/Components/Logging/EventSourceSink.cs @@ -1,7 +1,7 @@ // Copyright (c) Microsoft. All rights reserved. // Licensed under the MIT license. See LICENSE file in the project root for full license information. // -// Sink which will take in a build event and raiise it on its internal event source +// Sink which will take in a build event and raise it on its internal event source //----------------------------------------------------------------------- using System; diff --git a/src/XMakeBuildEngine/BackEnd/Components/Logging/ILoggingService.cs b/src/XMakeBuildEngine/BackEnd/Components/Logging/ILoggingService.cs index 3f70f430e2d..482a8e83665 100644 --- a/src/XMakeBuildEngine/BackEnd/Components/Logging/ILoggingService.cs +++ b/src/XMakeBuildEngine/BackEnd/Components/Logging/ILoggingService.cs @@ -160,7 +160,7 @@ bool RunningOnRemoteNode /// /// Allows the registering of an ICentralLogger and a forwarding logger pair /// - /// Central logger which is to recieve the events created by the forwarding logger + /// Central logger which is to receive the events created by the forwarding logger /// A description of the forwarding logger /// True if the central and forwarding loggers were registered. False if the central logger or the forwarding logger were already registered bool RegisterDistributedLogger(ILogger centralLogger, LoggerDescription forwardingLogger); diff --git a/src/XMakeBuildEngine/BackEnd/Components/Logging/LoggingService.cs b/src/XMakeBuildEngine/BackEnd/Components/Logging/LoggingService.cs index 29a61384f5f..9016e2c1ed5 100644 --- a/src/XMakeBuildEngine/BackEnd/Components/Logging/LoggingService.cs +++ b/src/XMakeBuildEngine/BackEnd/Components/Logging/LoggingService.cs @@ -642,7 +642,7 @@ public void ShutdownComponent() } /// - /// Will recieve a logging packet and send it to the correct + /// Will receive a logging packet and send it to the correct /// sink which is registered to the LoggingServices. /// PacketReceived should be called from a single thread. /// @@ -669,7 +669,7 @@ public void PacketReceived(int node, INodePacket packet) /// /// Register an instantiated logger which implements the ILogger interface. This logger will be registered to a specific event - /// source (the central logger event source) which will recieve all logging messages for a given build. + /// source (the central logger event source) which will receive all logging messages for a given build. /// This should not be used on a node, Loggers are not to be registered on a child node. /// /// ILogger @@ -760,12 +760,12 @@ public void UnregisterAllLoggers() /// /// The central logger is initialized before the distributed logger /// - /// Central logger to recieve messages from the forwarding logger, This logger cannot have been registered before - /// Logger description which describes how to create the forwarding logger, the logger description canot have been used before + /// Central logger to receive messages from the forwarding logger, This logger cannot have been registered before + /// Logger description which describes how to create the forwarding logger, the logger description cannot have been used before /// True if the distributed and central logger were registered, false if they either were already registered /// If forwardingLogger is null - /// If a logger exception is thown while creating or initializing the distributed or central logger - /// If any exception (other than a loggerException)is thown while creating or initializing the distributed or central logger, we will wrap these exceptions in an InternalLoggerException + /// If a logger exception is thrown while creating or initializing the distributed or central logger + /// If any exception (other than a loggerException)is thrown while creating or initializing the distributed or central logger, we will wrap these exceptions in an InternalLoggerException public bool RegisterDistributedLogger(ILogger centralLogger, LoggerDescription forwardingLogger) { lock (_lockObject) @@ -1111,7 +1111,7 @@ private void LoggingEventProcessor(object loggingEvent) bool cultureSet = false; try { - // If we have a componenthost then set the culture on the first message we recieve + // If we have a componenthost then set the culture on the first message we receive if (_componentHost != null) { originalCultureInfo = Thread.CurrentThread.CurrentCulture; @@ -1205,7 +1205,7 @@ private void RouteBuildEvent(BuildEventArgs eventArg) _filterEventSource.Consume(eventArg); // Now that the forwarding loggers have been given the chance to log the build started and finished events we need to check the - // central logger sinks to see if they have recieved the events or not. If the sink has not recieved the event we need to send it to the + // central logger sinks to see if they have received the events or not. If the sink has not received the event we need to send it to the // logger for backwards compatibility with orcas. // In addition we need to make sure we manually forward the events because in orcas the forwarding loggers were not allowed to // forward build started or build finished events. In the new OM we allow the loggers to forward the events. However since orcas did not forward them diff --git a/src/XMakeBuildEngine/BackEnd/Components/Logging/LoggingServiceLogMethods.cs b/src/XMakeBuildEngine/BackEnd/Components/Logging/LoggingServiceLogMethods.cs index 4472a0783e1..7a6f2f1d7af 100644 --- a/src/XMakeBuildEngine/BackEnd/Components/Logging/LoggingServiceLogMethods.cs +++ b/src/XMakeBuildEngine/BackEnd/Components/Logging/LoggingServiceLogMethods.cs @@ -134,7 +134,7 @@ public void LogError(BuildEventContext location, BuildEventFileInfo file, string /// File information about where the error happened /// String index into the string.resx file /// Arguments for the format string in the resource file - /// MessageResouceName is null + /// MessageResourceName is null public void LogError(BuildEventContext buildEventContext, string subcategoryResourceName, BuildEventFileInfo file, string messageResourceName, params object[] messageArgs) { lock (_lockObject) diff --git a/src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/IRequestBuilderCallback.cs b/src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/IRequestBuilderCallback.cs index 2145b800740..e4fde1250f1 100644 --- a/src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/IRequestBuilderCallback.cs +++ b/src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/IRequestBuilderCallback.cs @@ -51,7 +51,7 @@ internal interface IRequestBuilderCallback /// /// Instructs the RequestBuilder that next Build request from a task should post its request - /// and immeditely return so that the thread may be freed up. May not be nested. + /// and immediately return so that the thread may be freed up. May not be nested. /// void EnterMSBuildCallbackState(); diff --git a/src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/ITargetBuilderCallback.cs b/src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/ITargetBuilderCallback.cs index e355806e485..39e11e96721 100644 --- a/src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/ITargetBuilderCallback.cs +++ b/src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/ITargetBuilderCallback.cs @@ -16,7 +16,7 @@ namespace Microsoft.Build.BackEnd { /// - /// Interface implemted by the Target Builder which allows tasks to invoke targets and build projects. + /// Interface implemented by the Target Builder which allows tasks to invoke targets and build projects. /// internal interface ITargetBuilderCallback : IRequestBuilderCallback { diff --git a/src/XMakeBuildEngine/BackEnd/Components/Scheduler/Scheduler.cs b/src/XMakeBuildEngine/BackEnd/Components/Scheduler/Scheduler.cs index caaba2a8024..feeed2c33e7 100644 --- a/src/XMakeBuildEngine/BackEnd/Components/Scheduler/Scheduler.cs +++ b/src/XMakeBuildEngine/BackEnd/Components/Scheduler/Scheduler.cs @@ -362,7 +362,7 @@ public IEnumerable ReportResult(int nodeId, BuildResult result ScheduleResponse response = TrySatisfyRequestFromCache(request.Parent.AssignedNode, request.BuildRequest, skippedResultsAreOK: false); // response may be null if the result was never added to the cache. This can happen if the result has an exception in it - // or the results could not be satisfied becasue the initial or default targets have been skipped. If that is the case + // or the results could not be satisfied because the initial or default targets have been skipped. If that is the case // we need to report the result directly since it contains an exception if (response == null) { diff --git a/src/XMakeBuildEngine/Construction/Solution/SolutionConfigurationInSolution.cs b/src/XMakeBuildEngine/Construction/Solution/SolutionConfigurationInSolution.cs index 75a5e130a11..edbecbfc148 100644 --- a/src/XMakeBuildEngine/Construction/Solution/SolutionConfigurationInSolution.cs +++ b/src/XMakeBuildEngine/Construction/Solution/SolutionConfigurationInSolution.cs @@ -16,7 +16,7 @@ namespace Microsoft.Build.Construction public sealed class SolutionConfigurationInSolution { /// - /// Default seperator between configuration and platform in configuration + /// Default separator between configuration and platform in configuration /// full names /// internal const char ConfigurationPlatformSeparator = '|'; diff --git a/src/XMakeBuildEngine/Definition/ProjectCollection.cs b/src/XMakeBuildEngine/Definition/ProjectCollection.cs index 432b8f3b18c..8cedd54fcd5 100644 --- a/src/XMakeBuildEngine/Definition/ProjectCollection.cs +++ b/src/XMakeBuildEngine/Definition/ProjectCollection.cs @@ -308,7 +308,7 @@ public ProjectCollection(IDictionary globalProperties, IEnumerab } /// - /// Handler to recieve which project got added to the project collection. + /// Handler to receive which project got added to the project collection. /// [SuppressMessage("Microsoft.Design", "CA1034:NestedTypesShouldNotBeVisible", Justification = "This has been API reviewed")] public delegate void ProjectAddedEventHandler(object sender, ProjectAddedToProjectCollectionEventArgs e); diff --git a/src/XMakeBuildEngine/Evaluation/Conditionals/FunctionCallExpressionNode.cs b/src/XMakeBuildEngine/Evaluation/Conditionals/FunctionCallExpressionNode.cs index 20c59ee549c..4cb6c1aa078 100644 --- a/src/XMakeBuildEngine/Evaluation/Conditionals/FunctionCallExpressionNode.cs +++ b/src/XMakeBuildEngine/Evaluation/Conditionals/FunctionCallExpressionNode.cs @@ -64,7 +64,7 @@ internal override bool BoolEvaluate(ConditionEvaluator.IConditionEvaluationState // Ignore invalid characters or path related exceptions - // We will ignore the PathTooLong exception caused by GetFullPath becasue in single proc this code + // We will ignore the PathTooLong exception caused by GetFullPath because in single proc this code // is not executed and the condition is just evaluated to false as File.Exists and Directory.Exists does not throw in this situation. // To be consistant with that we will return a false in this case also. // DevDiv Bugs: 46035 diff --git a/src/XMakeBuildEngine/Logging/NullCentralLogger.cs b/src/XMakeBuildEngine/Logging/NullCentralLogger.cs index a23f373fbdc..7f5eb1d4d58 100644 --- a/src/XMakeBuildEngine/Logging/NullCentralLogger.cs +++ b/src/XMakeBuildEngine/Logging/NullCentralLogger.cs @@ -10,7 +10,7 @@ namespace Microsoft.Build.BackEnd.Logging { /// - /// This class will throw an exception when it recieves any event except for the build started or build finished event + /// This class will throw an exception when it receives any event except for the build started or build finished event /// this logger is good to use if a distributed logger is attached but does not want to forward any events /// internal class NullCentralLogger : INodeLogger @@ -56,7 +56,7 @@ public void AnyEventRaisedHandler(object sender, BuildEventArgs e) { if (!(e is BuildStartedEventArgs) && !(e is BuildFinishedEventArgs)) { - ErrorUtilities.VerifyThrowInvalidOperation(false, "Should not recieve any events other than build started or finished"); + ErrorUtilities.VerifyThrowInvalidOperation(false, "Should not receive any events other than build started or finished"); } } diff --git a/src/XMakeBuildEngine/UnitTests/BackEnd/BuildManager_Tests.cs b/src/XMakeBuildEngine/UnitTests/BackEnd/BuildManager_Tests.cs index 33dded39dea..0aa63d628ee 100644 --- a/src/XMakeBuildEngine/UnitTests/BackEnd/BuildManager_Tests.cs +++ b/src/XMakeBuildEngine/UnitTests/BackEnd/BuildManager_Tests.cs @@ -336,7 +336,7 @@ public void RunOutOfProcBuild(Action buildParametersModifier) } /// - /// Make sure when we are doign an inprocess build that even if the environment variable msbuildforwardpropertiesfromchild is set that we still + /// Make sure when we are doing an inprocess build that even if the environment variable msbuildforwardpropertiesfromchild is set that we still /// get all of the initial properties. /// [TestMethod] @@ -389,7 +389,7 @@ public void InProcForwardPropertiesFromChild() } /// - /// Make sure when we are doign an inprocess build that even if the environment variable MsBuildForwardAllPropertiesFromChild is set that we still + /// Make sure when we are doing an inprocess build that even if the environment variable MsBuildForwardAllPropertiesFromChild is set that we still /// get all of the initial properties. /// [TestMethod] diff --git a/src/XMakeBuildEngine/UnitTests/BackEnd/EventRedirectorToSink_Tests.cs b/src/XMakeBuildEngine/UnitTests/BackEnd/EventRedirectorToSink_Tests.cs index 4b09a5c0997..290c2c41f10 100644 --- a/src/XMakeBuildEngine/UnitTests/BackEnd/EventRedirectorToSink_Tests.cs +++ b/src/XMakeBuildEngine/UnitTests/BackEnd/EventRedirectorToSink_Tests.cs @@ -53,7 +53,7 @@ public void TestConstructorValidInputs() } /// - /// Verify when an event is forwarded, the event that was put in is the same event that was recieved on the event source + /// Verify when an event is forwarded, the event that was put in is the same event that was received on the event source /// also make sure the sinkId has been updated by the event redirector. /// [TestMethod] diff --git a/src/XMakeBuildEngine/UnitTests/BackEnd/EventSourceSink_Tests.cs b/src/XMakeBuildEngine/UnitTests/BackEnd/EventSourceSink_Tests.cs index bbc801d3df2..966395c94bd 100644 --- a/src/XMakeBuildEngine/UnitTests/BackEnd/EventSourceSink_Tests.cs +++ b/src/XMakeBuildEngine/UnitTests/BackEnd/EventSourceSink_Tests.cs @@ -469,7 +469,7 @@ public bool EnteredStatusEventHandler /// /// Which event was raised on the event source, this can be asserted upon /// to verify the event passed to the event source is the same one which was - /// recieved by the event handlers + /// received by the event handlers /// public BuildEventArgs RaisedEvent { diff --git a/src/XMakeBuildEngine/UnitTests/BackEnd/LoggingServicesLogMethod_Tests.cs b/src/XMakeBuildEngine/UnitTests/BackEnd/LoggingServicesLogMethod_Tests.cs index 99422d9c5e1..2df91b9739e 100644 --- a/src/XMakeBuildEngine/UnitTests/BackEnd/LoggingServicesLogMethod_Tests.cs +++ b/src/XMakeBuildEngine/UnitTests/BackEnd/LoggingServicesLogMethod_Tests.cs @@ -566,7 +566,7 @@ public void LogWarningTests() #region LogWarningErrorFromText /// - /// Verify an InternalErrorException is thown when a null BuildEventContext is passed in + /// Verify an InternalErrorException is thrown when a null BuildEventContext is passed in /// [TestMethod] [ExpectedException(typeof(InternalErrorException))] diff --git a/src/XMakeBuildEngine/UnitTests/BackEnd/TaskHost_Tests.cs b/src/XMakeBuildEngine/UnitTests/BackEnd/TaskHost_Tests.cs index 6e96fc61297..a2c4fdd3d1d 100644 --- a/src/XMakeBuildEngine/UnitTests/BackEnd/TaskHost_Tests.cs +++ b/src/XMakeBuildEngine/UnitTests/BackEnd/TaskHost_Tests.cs @@ -1083,7 +1083,7 @@ public void Shutdown() } /// - /// Log if we have recieved any event. + /// Log if we have received any event. /// internal void EventSource_AnyEventRaised(object sender, BuildEventArgs e) { diff --git a/src/XMakeBuildEngine/UnitTests/QaTests/ITestDataProvider.cs b/src/XMakeBuildEngine/UnitTests/QaTests/ITestDataProvider.cs index 306f8b1f5f5..b23c217eeaf 100644 --- a/src/XMakeBuildEngine/UnitTests/QaTests/ITestDataProvider.cs +++ b/src/XMakeBuildEngine/UnitTests/QaTests/ITestDataProvider.cs @@ -12,7 +12,7 @@ namespace Microsoft.Build.UnitTests.QA { /// - /// An interfacing representing a build request defination cache. + /// An interfacing representing a build request definition cache. /// internal interface ITestDataProvider { diff --git a/src/XMakeBuildEngine/UnitTests/QaTests/MockResultsCache.cs b/src/XMakeBuildEngine/UnitTests/QaTests/MockResultsCache.cs index 36c456911cf..cc1d7e99f05 100644 --- a/src/XMakeBuildEngine/UnitTests/QaTests/MockResultsCache.cs +++ b/src/XMakeBuildEngine/UnitTests/QaTests/MockResultsCache.cs @@ -10,7 +10,7 @@ namespace Microsoft.Build.UnitTests.QA /// /// Mock Implementation of the results cache which does captures some counters and provides it /// back to the tests for validation purposes. Most of the implementation is routed to the default component - /// by means of aggrigration + /// by means of aggregation /// internal class QAResultsCache : IResultsCache, IBuildComponent { diff --git a/src/XMakeTasks/AssemblyDependency/NoMatchReason.cs b/src/XMakeTasks/AssemblyDependency/NoMatchReason.cs index 5dc77b9376c..309c9371df3 100644 --- a/src/XMakeTasks/AssemblyDependency/NoMatchReason.cs +++ b/src/XMakeTasks/AssemblyDependency/NoMatchReason.cs @@ -51,7 +51,7 @@ internal enum NoMatchReason NotAFileNameOnDisk, /// - /// The processor architecure does not match the targeted processor architecture. + /// The processor architecture does not match the targeted processor architecture. /// ProcessorArchitectureDoesNotMatch } diff --git a/src/XMakeTasks/CommandLineBuilderExtension.cs b/src/XMakeTasks/CommandLineBuilderExtension.cs index 3f9616eb44a..47fd8a24f44 100644 --- a/src/XMakeTasks/CommandLineBuilderExtension.cs +++ b/src/XMakeTasks/CommandLineBuilderExtension.cs @@ -201,7 +201,7 @@ internal void AppendSwitchWithSplitting(string switchName, string parameter, str /// /// Returns true if the parameter is empty in spirits, - /// even if it contains the seperators and white space only + /// even if it contains the separators and white space only /// Split on the characters provided. /// /// diff --git a/src/XMakeTasks/FxCopExclusions/Microsoft.Build.Tasks.Suppressions.cs b/src/XMakeTasks/FxCopExclusions/Microsoft.Build.Tasks.Suppressions.cs index cfa4ec54dc4..7349ce57fe8 100644 --- a/src/XMakeTasks/FxCopExclusions/Microsoft.Build.Tasks.Suppressions.cs +++ b/src/XMakeTasks/FxCopExclusions/Microsoft.Build.Tasks.Suppressions.cs @@ -9,9 +9,9 @@ #if CODE_ANALYSIS [module: SuppressMessage("Microsoft.Naming","CA1704:IdentifiersShouldBeSpelledCorrectly", MessageId="0", Scope="module", Target="microsoft.build.tasks.core.dll", Justification="Already shipped several versions with a name like this")] -[module: SuppressMessage("Microsoft.Usage","CA1806:DoNotIgnoreMethodResults", MessageId="Microsoft.Build.Tasks.NativeMethods.GetCachePath(Microsoft.Build.Tasks.AssemblyCacheFlags,System.Text.StringBuilder,System.Int32@)", Scope="member", Target="Microsoft.Build.Tasks.GlobalAssemblyCache.#GetGacPath()", Justification="We do not use the method result we use one of the out parameters as a check instead. This is becasue the method can have a failed hresult but return a object in the out parameter.")] -[module: SuppressMessage("Microsoft.Usage","CA1806:DoNotIgnoreMethodResults", MessageId="Microsoft.Build.Tasks.IAssemblyCache.QueryAssemblyInfo(System.UInt32,System.String,Microsoft.Build.Tasks.ASSEMBLY_INFO@)", Scope="member", Target="Microsoft.Build.Tasks.GlobalAssemblyCache.#GetLocationImpl(Microsoft.Build.Shared.AssemblyNameExtension,System.String)", Justification="We do not use the method result we use one of the out parameters as a check instead. This is becasue the method can have a failed hresult but return a object in the out parameter.")] -[module: SuppressMessage("Microsoft.Usage","CA1806:DoNotIgnoreMethodResults", MessageId="Microsoft.Build.Tasks.NativeMethods.CreateAssemblyCache(Microsoft.Build.Tasks.IAssemblyCache@,System.UInt32)", Scope="member", Target="Microsoft.Build.Tasks.GlobalAssemblyCache.#GetLocationImpl(Microsoft.Build.Shared.AssemblyNameExtension,System.String)", Justification="We do not use the method result we use one of the out parameters as a check instead. This is becasue the method can have a failed hresult but return a object in the out parameter.")] +[module: SuppressMessage("Microsoft.Usage","CA1806:DoNotIgnoreMethodResults", MessageId="Microsoft.Build.Tasks.NativeMethods.GetCachePath(Microsoft.Build.Tasks.AssemblyCacheFlags,System.Text.StringBuilder,System.Int32@)", Scope="member", Target="Microsoft.Build.Tasks.GlobalAssemblyCache.#GetGacPath()", Justification="We do not use the method result we use one of the out parameters as a check instead. This is because the method can have a failed hresult but return a object in the out parameter.")] +[module: SuppressMessage("Microsoft.Usage","CA1806:DoNotIgnoreMethodResults", MessageId="Microsoft.Build.Tasks.IAssemblyCache.QueryAssemblyInfo(System.UInt32,System.String,Microsoft.Build.Tasks.ASSEMBLY_INFO@)", Scope="member", Target="Microsoft.Build.Tasks.GlobalAssemblyCache.#GetLocationImpl(Microsoft.Build.Shared.AssemblyNameExtension,System.String)", Justification="We do not use the method result we use one of the out parameters as a check instead. This is because the method can have a failed hresult but return a object in the out parameter.")] +[module: SuppressMessage("Microsoft.Usage","CA1806:DoNotIgnoreMethodResults", MessageId="Microsoft.Build.Tasks.NativeMethods.CreateAssemblyCache(Microsoft.Build.Tasks.IAssemblyCache@,System.UInt32)", Scope="member", Target="Microsoft.Build.Tasks.GlobalAssemblyCache.#GetLocationImpl(Microsoft.Build.Shared.AssemblyNameExtension,System.String)", Justification="We do not use the method result we use one of the out parameters as a check instead. This is because the method can have a failed hresult but return a object in the out parameter.")] [module: SuppressMessage("Microsoft.Usage","CA1806:DoNotIgnoreMethodResults", MessageId="Microsoft.Build.Tasks.NativeMethods.CompareAssemblyIdentity(System.String,System.Boolean,System.String,System.Boolean,System.Boolean@,Microsoft.Build.Tasks.NativeMethods+AssemblyComparisonResult@)", Scope="member", Target="Microsoft.Build.Tasks.ReferenceTable.#ResolveAssemblyNameConflict(Microsoft.Build.Tasks.AssemblyNameReference,Microsoft.Build.Tasks.AssemblyNameReference)", Justification="We do use this")] [module: SuppressMessage("Microsoft.Design","CA1056:UriPropertiesShouldNotBeStrings", Scope="member", Target="Microsoft.Build.Tasks.RequiresFramework35SP1Assembly", Justification="These are public properties on public tasks, these cannot have their type changed due to backwards compatibility")] [module: SuppressMessage("Microsoft.Design","CA1056:UriPropertiesShouldNotBeStrings", Scope="member", Target="Microsoft.Build.Tasks.Deployment.ManifestUtilities.DeployManifest.XmlTrustUrlParameters", Justification="These are public properties on public tasks, these cannot have their type changed due to backwards compatibility")] diff --git a/src/XMakeTasks/ManifestUtil/FileAssociation.cs b/src/XMakeTasks/ManifestUtil/FileAssociation.cs index 117a144f15e..53977a28fe5 100644 --- a/src/XMakeTasks/ManifestUtil/FileAssociation.cs +++ b/src/XMakeTasks/ManifestUtil/FileAssociation.cs @@ -11,7 +11,7 @@ namespace Microsoft.Build.Tasks.Deployment.ManifestUtilities { /// - /// Describes a fileAssocation for an application manifest + /// Describes a fileAssociation for an application manifest /// [ComVisible(false)] public sealed class FileAssociation diff --git a/src/XMakeTasks/UnitTests/ResolveAssemblyReference_Tests.cs b/src/XMakeTasks/UnitTests/ResolveAssemblyReference_Tests.cs index d7a986b3464..0d0e6be3810 100644 --- a/src/XMakeTasks/UnitTests/ResolveAssemblyReference_Tests.cs +++ b/src/XMakeTasks/UnitTests/ResolveAssemblyReference_Tests.cs @@ -2826,7 +2826,7 @@ protected static bool Execute(ResolveAssemblyReference t) /// /// Execute the task. Without confirming that the number of files resolved with and without find dependencies is identical. - /// This is becasue profiles could cause the number of primary references to be different. + /// This is because profiles could cause the number of primary references to be different. /// protected static bool Execute(ResolveAssemblyReference t, bool buildConsistencyCheck) { @@ -7194,7 +7194,7 @@ public void ResolveWinmdWithArchitectureDependencyAnyCPUNative() /// /// Resolve a winmd file which depends on a native implementation dll that has an invalid pe header. - /// A warning or error is expectected in the log depending on the WarnOrErrorOnTargetArchitecture property value. + /// A warning or error is expected in the log depending on the WarnOrErrorOnTargetArchitecture property value. /// [TestMethod] public void ResolveWinmdWithArchitectureDependency() @@ -13092,7 +13092,7 @@ public void RedistListDifferentNameToSubSet() /// /// Test the case where the subset list has the same name as the redist list but it has no entries In this case - /// the black list should contain ALL redist list entries becasue there are no white list files to remove from the black list. + /// the black list should contain ALL redist list entries because there are no white list files to remove from the black list. /// [TestMethod] public void RedistListEmptySubsetMatchingName() diff --git a/src/XMakeTasks/Vbc.cs b/src/XMakeTasks/Vbc.cs index 5f1e1bc6fee..abba77965ba 100644 --- a/src/XMakeTasks/Vbc.cs +++ b/src/XMakeTasks/Vbc.cs @@ -38,7 +38,7 @@ public class Vbc : ManagedCompiler // The following 1 fields are used, set and re-set in LogEventsFromTextOutput() /// - /// This stores the origional lines and error priority together in the order in which they were recieved. + /// This stores the origional lines and error priority together in the order in which they were received. /// private Queue _vbErrorLines = new Queue();