Don't call cancel.OnCompleted after disposal.

grokys · grokys · commit d7b2353f4cdc · 2016-05-19T11:05:30.000+01:00
`cancel.OnNext` causes `cancel` to get disposed, so don't try calling `OnCompleted` on it afterwards. Fixes #311.
diff --git a/src/GitHub.UI.Reactive/Controls/SimpleViewUserControl.cs b/src/GitHub.UI.Reactive/Controls/SimpleViewUserControl.cs
@@ -56,7 +56,6 @@ protected void NotifyCancel()
                 return;
 
             cancel.OnNext(null);
-            cancel.OnCompleted();
         }
 
         protected void NotifyIsBusy(bool busy)

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,6 @@ protected void NotifyCancel()`
`56`	`56`	`return;`
`57`	`57`
`58`	`58`	`cancel.OnNext(null);`
`59`		`- cancel.OnCompleted();`
`60`	`59`	`}`
`61`	`60`
`62`	`61`	`protected void NotifyIsBusy(bool busy)`