[xbuild] Log default msg for TargetSkipped if @reason is null or empty.

diff --git a/mcs/class/Microsoft.Build.Engine/Microsoft.Build.BuildEngine/TargetBatchingImpl.cs b/mcs/class/Microsoft.Build.Engine/Microsoft.Build.BuildEngine/TargetBatchingImpl.cs
index 54aad29da105d31c182d8d74da138b5acb28bda7..b5bc4599fdaf567084c61605456686714338a5c4 100644 (file)
--- a/mcs/class/Microsoft.Build.Engine/Microsoft.Build.BuildEngine/TargetBatchingImpl.cs
+++ b/mcs/class/Microsoft.Build.Engine/Microsoft.Build.BuildEngine/TargetBatchingImpl.cs
@@ -241,8 +241,10 @@ namespace Microsoft.Build.BuildEngine {
                void LogTargetSkipped (Target target, string reason)
                {
                        BuildMessageEventArgs bmea;
-                       bmea = new BuildMessageEventArgs (reason ?? String.Format ("Skipping target \"{0}\" because its outputs are up-to-date.", target.Name),
-                               null, "MSBuild", MessageImportance.Normal);
+                       bmea = new BuildMessageEventArgs (String.IsNullOrEmpty (reason)
+                                                               ? String.Format ("Skipping target \"{0}\" because its outputs are up-to-date.", target.Name)
+                                                               : reason,
+                                                       null, "MSBuild", MessageImportance.Normal);
                        target.Engine.EventSource.FireMessageRaised (this, bmea);
                }