No reason to throw here - it's the lambda that needs the UI thread

GrahamTheCoder · GrahamTheCoder · commit 7e02130a8857 · 2019-01-21T12:03:22.000Z
diff --git a/Vsix/BlockingOleMenuCommand.cs b/Vsix/BlockingOleMenuCommand.cs
@@ -11,8 +11,9 @@ internal class BlockingOleMenuCommand
 
         private static EventHandler WaitFor(Func<object, EventArgs, Task> task)
         {
-            ThreadHelper.ThrowIfNotOnUIThread();
-            return (o, a) => ThreadHelper.JoinableTaskFactory.Run(() => task(o,a));
+            return (o, a) => {
+                ThreadHelper.JoinableTaskFactory.Run(() => task(o, a));
+            };
         }
 
         public BlockingOleMenuCommand(Func<object, EventArgs, Task> invokeHandler, CommandID id)

Original file line number	Diff line number	Diff line change
`@@ -11,8 +11,9 @@ internal class BlockingOleMenuCommand`
`11`	`11`
`12`	`12`	`private static EventHandler WaitFor(Func<object, EventArgs, Task> task)`
`13`	`13`	`{`
`14`		`- ThreadHelper.ThrowIfNotOnUIThread();`
`15`		`- return (o, a) => ThreadHelper.JoinableTaskFactory.Run(() => task(o,a));`
	`14`	`+ return (o, a) => {`
	`15`	`+ ThreadHelper.JoinableTaskFactory.Run(() => task(o, a));`
	`16`	`+ };`
`16`	`17`	`}`
`17`	`18`
`18`	`19`	`public BlockingOleMenuCommand(Func<object, EventArgs, Task> invokeHandler, CommandID id)`