From 2f8fd706db7aec790680d144035a0a5a5fc1897a Mon Sep 17 00:00:00 2001 From: baldurk Date: Wed, 14 Sep 2016 12:06:48 +0200 Subject: [PATCH] Change where remote lookup counter is inc/decremented for better results --- renderdocui/Windows/Dialogs/RemoteManager.cs | 21 +++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/renderdocui/Windows/Dialogs/RemoteManager.cs b/renderdocui/Windows/Dialogs/RemoteManager.cs index 2325b21dc..af44f3725 100644 --- a/renderdocui/Windows/Dialogs/RemoteManager.cs +++ b/renderdocui/Windows/Dialogs/RemoteManager.cs @@ -139,10 +139,17 @@ namespace renderdocui.Windows.Dialogs hosts.Nodes.Add(node); refreshOne.Enabled = refreshAll.Enabled = false; - UpdateLookupsStatus(); + + { + lookupMutex.WaitOne(); + lookupsInProgress++; + lookupMutex.ReleaseMutex(); + } Thread th = Helpers.NewThread(new ParameterizedThreadStart(LookupHostConnections)); th.Start(node); + + UpdateLookupsStatus(); } private void UpdateLookupsStatus() @@ -170,12 +177,6 @@ namespace renderdocui.Windows.Dialogs // queries it for the API, target (usually executable name) and if any user is already connected private static void LookupHostConnections(object o) { - { - lookupMutex.WaitOne(); - lookupsInProgress++; - lookupMutex.ReleaseMutex(); - } - TreelistView.Node node = o as TreelistView.Node; Control p = node.OwnerView; @@ -505,6 +506,12 @@ namespace renderdocui.Windows.Dialogs // try to run refreshOne.Enabled = refreshAll.Enabled = false; + { + lookupMutex.WaitOne(); + lookupsInProgress++; + lookupMutex.ReleaseMutex(); + } + Thread th = Helpers.NewThread(new ParameterizedThreadStart(RunRemoteServer)); th.Start(node);