From 48b0a2601fbd52ee98b65d23970f22256aade17d Mon Sep 17 00:00:00 2001 From: Lars Op den Kamp Date: Fri, 13 Dec 2013 11:51:41 +0100 Subject: [PATCH] still locking up when disposing, because the log callback is trying to write to the disposing ui. disconnect from a background thread instead when disposing --- src/LibCecTray/ui/CECTray.cs | 24 ++++++++++++++++++------ 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/src/LibCecTray/ui/CECTray.cs b/src/LibCecTray/ui/CECTray.cs index 197ce32..021eaa3 100644 --- a/src/LibCecTray/ui/CECTray.cs +++ b/src/LibCecTray/ui/CECTray.cs @@ -41,6 +41,7 @@ using LibCECTray.settings; using Microsoft.Win32; using System.Security.Permissions; using System.Runtime.InteropServices; +using System.Threading; namespace LibCECTray.ui { @@ -57,6 +58,21 @@ namespace LibCECTray.ui XBMC } + class AsyncDisconnect + { + public AsyncDisconnect(CECController controller) + { + _controller = controller; + } + + public void Process() + { + _controller.Close(); + } + + private CECController _controller; + } + /// /// Main LibCecTray GUI /// @@ -174,11 +190,11 @@ namespace LibCECTray.ui protected override void Dispose(bool disposing) { Hide(); - SuppressLogUpdates = true; if (disposing) { Controller.CECActions.SuppressUpdates = true; - Controller.Close(); + AsyncDisconnect dc = new AsyncDisconnect(Controller); + (new Thread(dc.Process)).Start(); } if (disposing && (components != null)) { @@ -312,9 +328,6 @@ namespace LibCECTray.ui delegate void UpdateLogCallback(); private void UpdateLog() { - if (SuppressLogUpdates) - return; - if (tbLog.InvokeRequired) { UpdateLogCallback d = UpdateLog; @@ -572,7 +585,6 @@ namespace LibCECTray.ui #endregion #region Class members - private bool SuppressLogUpdates = false; private ConfigTab _selectedTab = ConfigTab.Configuration; private string _log = string.Empty; private CECController _controller; -- 2.34.1