Making sure the EnsureLoad method is not optimized away

StanleyGoldman · jcansdale · StanleyGoldman · commit 46fe8631eaee · 2019-06-03T11:25:15.000-04:00
co-authored-by: Jamie Cansdale &lt;jcansdale@gmail.com&gt;
diff --git a/src/GitHub.VisualStudio.16/CompositionServices.cs b/src/GitHub.VisualStudio.16/CompositionServices.cs
@@ -12,6 +12,7 @@
 using GitHub.Settings;
 using GitHub.VisualStudio.Settings;
 using GitHub.VisualStudio.Views.Dialog.Clone;
+using Microsoft;
 using Microsoft.VisualStudio.Shell;
 using Microsoft.VisualStudio.Threading;
 using Rothko;
@@ -128,6 +129,7 @@ static CompositionContainer CreateCompositionContainer(ExportProvider defaultExp
         static void EnsureLoaded(Type type)
         {
             // Ensure the containing assembly has been loaded
+            Assumes.NotNull(type);
         }
 
         static LoginManager CreateLoginManager(CompositionContainer compositionContainer)

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,7 @@`
`12`	`12`	`using GitHub.Settings;`
`13`	`13`	`using GitHub.VisualStudio.Settings;`
`14`	`14`	`using GitHub.VisualStudio.Views.Dialog.Clone;`
	`15`	`+using Microsoft;`
`15`	`16`	`using Microsoft.VisualStudio.Shell;`
`16`	`17`	`using Microsoft.VisualStudio.Threading;`
`17`	`18`	`using Rothko;`
`@@ -128,6 +129,7 @@ static CompositionContainer CreateCompositionContainer(ExportProvider defaultExp`
`128`	`129`	`static void EnsureLoaded(Type type)`
`129`	`130`	`{`
`130`	`131`	`// Ensure the containing assembly has been loaded`
	`132`	`+ Assumes.NotNull(type);`
`131`	`133`	`}`
`132`	`134`
`133`	`135`	`static LoginManager CreateLoginManager(CompositionContainer compositionContainer)`