Merge pull request #3788 from bclothier/EventInterface

retailcoder · web-flow · commit f92736b3799f · 2018-02-23T09:58:10.000-05:00
Fixes a bug in COM event management (a constant had the wrong value).
diff --git a/Rubberduck.VBEEditor/SafeComWrappers/SafeEventedComWrapper.cs b/Rubberduck.VBEEditor/SafeComWrappers/SafeEventedComWrapper.cs
@@ -9,7 +9,7 @@ public abstract class SafeEventedComWrapper<TSource, TEventInterface> : SafeComW
         where TSource : class
         where TEventInterface : class
     {
-        private const int NotAdvising = -1;
+        private const int NotAdvising = 0;
         private readonly object _lock = new object();
         private IConnectionPoint _icp; // The connection point
         private int _cookie = NotAdvising;     // The cookie for the connection

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ public abstract class SafeEventedComWrapper<TSource, TEventInterface> : SafeComW`
`9`	`9`	`where TSource : class`
`10`	`10`	`where TEventInterface : class`
`11`	`11`	`{`
`12`		`- private const int NotAdvising = -1;`
	`12`	`+ private const int NotAdvising = 0;`
`13`	`13`	`private readonly object _lock = new object();`
`14`	`14`	`private IConnectionPoint _icp; // The connection point`
`15`	`15`	`private int _cookie = NotAdvising; // The cookie for the connection`