More simplification

Paul van Brenk · Paul van Brenk · commit 6c59bfa76f38 · 2017-08-24T16:03:36.000-07:00
diff --git a/Nodejs/Product/Nodejs/Project/NodejsProjectNode.cs b/Nodejs/Product/Nodejs/Project/NodejsProjectNode.cs
@@ -585,16 +585,13 @@ public async Task CheckForLongPaths(string npmArguments = null)
                 this._isCheckingForLongPaths = true;
 
                 var longPaths = await Task.Factory.StartNew(() =>
-                    GetLongSubPaths(this.ProjectHome)
-                    .Concat(GetLongSubPaths(this._intermediateOutputPath))
-                    .Any());
-                if (!longPaths)
+                    GetLongSubPaths(this.ProjectHome).Any() || GetLongSubPaths(this._intermediateOutputPath).Any());
+
+                if (longPaths)
                 {
-                    return;
+                    Utilities.ShowMessageBox(
+                      this.Site, Resources.LongPathWarningText, SR.ProductName, OLEMSGICON.OLEMSGICON_WARNING, OLEMSGBUTTON.OLEMSGBUTTON_OK, OLEMSGDEFBUTTON.OLEMSGDEFBUTTON_FIRST);
                 }
-
-                Utilities.ShowMessageBox(
-                  this.Site, Resources.LongPathWarningText, SR.ProductName, OLEMSGICON.OLEMSGICON_WARNING, OLEMSGBUTTON.OLEMSGBUTTON_OK, OLEMSGDEFBUTTON.OLEMSGDEFBUTTON_FIRST);
             }
             finally
             {

Original file line number	Diff line number	Diff line change
`@@ -585,16 +585,13 @@ public async Task CheckForLongPaths(string npmArguments = null)`
`585`	`585`	`this._isCheckingForLongPaths = true;`
`586`	`586`
`587`	`587`	`var longPaths = await Task.Factory.StartNew(() =>`
`588`		`- GetLongSubPaths(this.ProjectHome)`
`589`		`- .Concat(GetLongSubPaths(this._intermediateOutputPath))`
`590`		`- .Any());`
`591`		`- if (!longPaths)`
	`588`	`+ GetLongSubPaths(this.ProjectHome).Any() \|\| GetLongSubPaths(this._intermediateOutputPath).Any());`
	`589`	`+`
	`590`	`+ if (longPaths)`
`592`	`591`	`{`
`593`		`- return;`
	`592`	`+ Utilities.ShowMessageBox(`
	`593`	`+ this.Site, Resources.LongPathWarningText, SR.ProductName, OLEMSGICON.OLEMSGICON_WARNING, OLEMSGBUTTON.OLEMSGBUTTON_OK, OLEMSGDEFBUTTON.OLEMSGDEFBUTTON_FIRST);`
`594`	`594`	`}`
`595`		`-`
`596`		`- Utilities.ShowMessageBox(`
`597`		`- this.Site, Resources.LongPathWarningText, SR.ProductName, OLEMSGICON.OLEMSGICON_WARNING, OLEMSGBUTTON.OLEMSGBUTTON_OK, OLEMSGDEFBUTTON.OLEMSGDEFBUTTON_FIRST);`
`598`	`595`	`}`
`599`	`596`	`finally`
`600`	`597`	`{`