Merge pull request #140 from spudwebb/fixlistener

Oren Novotny · web-flow · commit 0af235b9511b · 2019-08-15T13:34:24.000-07:00
Fix ResolverListener so that ServiceLost is called only after N successive failed pings
diff --git a/Zeroconf/ZeroConfResolver.Listener.cs b/Zeroconf/ZeroConfResolver.Listener.cs
@@ -98,8 +98,9 @@ void OnResolved(IReadOnlyList<IZeroconfHost> hosts)
                             {
                                 ServiceFound?.Invoke(this, host);
                                 newHosts.Add(keyValue);
-                                if (toRemove.ContainsKey(keyValue)) toRemove.Remove(keyValue);
                             }
+                            if (toRemove.ContainsKey(keyValue))
+                                toRemove.Remove(keyValue);
                         }
                     }
 

Original file line number	Diff line number	Diff line change
`@@ -98,8 +98,9 @@ void OnResolved(IReadOnlyList<IZeroconfHost> hosts)`
`98`	`98`	`{`
`99`	`99`	`ServiceFound?.Invoke(this, host);`
`100`	`100`	`newHosts.Add(keyValue);`
`101`		`- if (toRemove.ContainsKey(keyValue)) toRemove.Remove(keyValue);`
`102`	`101`	`}`
	`102`	`+ if (toRemove.ContainsKey(keyValue))`
	`103`	`+ toRemove.Remove(keyValue);`
`103`	`104`	`}`
`104`	`105`	`}`
`105`	`106`