remove some redundant and unreachable code.

src/Shared/HybridDictionary.cs

@@ -674,7 +674,6 @@ private void AddToOrUpdateList(List<KeyValuePair<TKey, TValue>> list, TKey key,
                 }
 
                 list.Add(new KeyValuePair<TKey, TValue>(key, value));
-                return;
             }
             else
             {
@@ -694,7 +693,6 @@ private void AddToOrUpdateList(List<KeyValuePair<TKey, TValue>> list, TKey key,
                 }
 
                 _store = newDictionary;
-                return;
             }
         }
 

src/Shared/LogMessagePacketBase.cs

@@ -281,7 +281,7 @@ internal void WriteToStream(INodePacketTranslator translator)
                 // Make sure the other side knows what sort of serialization is coming
                 translator.Translate(ref packetVersion);
 
-                bool eventCanSerializeItself = (methodInfo != null) ? true : false;
+                bool eventCanSerializeItself = methodInfo != null;
                 translator.Translate(ref eventCanSerializeItself);
 
                 if (eventCanSerializeItself)

src/XMakeBuildEngine/BackEnd/BuildManager/BuildParameters.cs

@@ -1057,7 +1057,6 @@ private void FindMSBuildExe()
             }
 
             // Well, we just can't find it.  Maybe they will only build in-proc and won't need it...
-            return;
         }
 
         /// <summary>

src/XMakeBuildEngine/BackEnd/Components/Communications/NodeProviderOutOfProcTaskHost.cs

@@ -570,7 +570,7 @@ internal bool CreateNode(TaskHostContext hostContext, INodePacketFactory factory
                 return false;
             }
 
-            CommunicationsUtilities.Trace("For a host context of {0}, spawning executable from {1}.", hostContext.ToString(), msbuildLocation ?? "MSBuild.exe");
+            CommunicationsUtilities.Trace("For a host context of {0}, spawning executable from {1}.", hostContext.ToString(), msbuildLocation);
 
             // Make it here.
             NodeContext context = GetNode

src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/Lookup.cs

@@ -1085,7 +1085,7 @@ public MetadataModifications Clone()
             /// </summary>
             public bool HasChanges
             {
-                get { return _modifications.Count > 0 || _keepOnlySpecified == true; }
+                get { return _modifications.Count > 0 || _keepOnlySpecified; }
             }
 
             /// <summary>

src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/TargetBuilder.cs

@@ -520,8 +520,6 @@ private async Task ProcessTargetStack(ITaskBuilder taskBuilder)
                         break;
                 }
             }
-
-            return;
         }
 
         /// <summary>

src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/TargetEntry.cs

@@ -662,7 +662,6 @@ internal async Task ExecuteTarget(ITaskBuilder taskBuilder, BuildRequestEntry re
                 DataCollection.CommentMarkProfile(8801, endTargetBuild);
             }
 #endif
-            return;
         }
 
         /// <summary>

src/XMakeBuildEngine/BackEnd/Components/RequestBuilder/TaskBuilder.cs

@@ -314,7 +314,7 @@ private async Task<WorkUnitResult> ExecuteTask(TaskExecutionMode mode, Lookup lo
                 // Only create a hash table if there are more than one bucket as this is the only time a property can be overridden
                 if (buckets.Count > 1)
                 {
-                    lookupHash = lookupHash ?? new Dictionary<string, string>(MSBuildNameIgnoreCaseComparer.Default);
+                    lookupHash = new Dictionary<string, string>(MSBuildNameIgnoreCaseComparer.Default);
                 }
 
                 WorkUnitResult aggregateResult = new WorkUnitResult();

src/XMakeCommandLine/OutOfProcTaskHostNode.cs

@@ -728,9 +728,6 @@ private void OnLinkStatusChanged(INodeEndpoint endpoint, LinkStatus status)
 
                 case LinkStatus.Inactive:
                     break;
-
-                default:
-                    break;
             }
         }
 
@@ -893,7 +890,7 @@ private void SetTaskHostEnvironment(IDictionary<string, string> environment)
                         {
                             if (_updateEnvironmentAndLog)
                             {
-                                LogMessageFromResource(MessageImportance.Low, "ModifyingTaskHostEnvironmentVariable", variable, newValue ?? String.Empty, environmentValue ?? String.Empty);
+                                LogMessageFromResource(MessageImportance.Low, "ModifyingTaskHostEnvironmentVariable", variable, newValue, environmentValue ?? String.Empty);
                             }
 
                             updatedEnvironment[variable] = newValue;