Merge pull request #311 from jamesc-skyward/ExceptionFix

madskristensen · web-flow · commit 2ef185333386 · 2022-03-21T09:16:58.000-07:00
Fix exception being thrown when the project being observed is the one…
diff --git a/src/toolkit/Community.VisualStudio.Toolkit.Shared/Build/Build.cs b/src/toolkit/Community.VisualStudio.Toolkit.Shared/Build/Build.cs
@@ -173,10 +173,9 @@ public int UpdateProjectCfg_Done(IVsHierarchy pHierProj, IVsCfg pCfgProj, IVsCfg
                     {
                         _result.SetCanceled();
                     }
-
                     // We are observing the build of a specific project. If the project
                     // that finished is the one we are observing, then we can set the result.
-                    if (pHierProj == _hierarchy)
+                    else if (pHierProj == _hierarchy)
                     {
                         _result.SetResult(fSuccess != 0);
                     }

Original file line number	Diff line number	Diff line change
`@@ -173,10 +173,9 @@ public int UpdateProjectCfg_Done(IVsHierarchy pHierProj, IVsCfg pCfgProj, IVsCfg`
`173`	`173`	`{`
`174`	`174`	`_result.SetCanceled();`
`175`	`175`	`}`
`176`		`-`
`177`	`176`	`// We are observing the build of a specific project. If the project`
`178`	`177`	`// that finished is the one we are observing, then we can set the result.`
`179`		`- if (pHierProj == _hierarchy)`
	`178`	`+ else if (pHierProj == _hierarchy)`
`180`	`179`	`{`
`181`	`180`	`_result.SetResult(fSuccess != 0);`
`182`	`181`	`}`