Просмотр исходного кода

Fix debugger plugin not cleaning up all event listeners

Michael Ragazzon 2 лет назад
Родитель
Сommit
85045314ff
2 измененных файлов с 22 добавлено и 17 удалено
  1. 20 17
      Source/Debugger/DebuggerPlugin.cpp
  2. 2 0
      Source/Debugger/DebuggerPlugin.h

+ 20 - 17
Source/Debugger/DebuggerPlugin.cpp

@@ -127,20 +127,7 @@ bool DebuggerPlugin::SetContext(Context* context)
 	// Attach the info element to the new context.
 	if (info_element)
 	{
-		if (debug_context)
-		{
-			debug_context->RemoveEventListener("click", info_element, true);
-			debug_context->RemoveEventListener("mouseover", info_element, true);
-			debug_context->RemoveEventListener("mouseout", info_element, true);
-		}
-
-		if (context)
-		{
-			context->AddEventListener("click", info_element, true);
-			context->AddEventListener("mouseover", info_element, true);
-			context->AddEventListener("mouseout", info_element, true);
-		}
-
+		SetupInfoListeners(context);
 		info_element->Reset();
 	}
 
@@ -373,15 +360,31 @@ bool DebuggerPlugin::LoadLogElement()
 	return true;
 }
 
-void DebuggerPlugin::ReleaseElements()
+void DebuggerPlugin::SetupInfoListeners(Rml::Context* new_context)
 {
-	// Erase event listeners to prevent crashes.
-	if (debug_context && info_element)
+	RMLUI_ASSERT(info_element);
+
+	if (debug_context)
 	{
 		debug_context->RemoveEventListener("click", info_element, true);
 		debug_context->RemoveEventListener("mouseover", info_element, true);
+		debug_context->RemoveEventListener("mouseout", info_element, true);
 	}
 
+	if (new_context)
+	{
+		new_context->AddEventListener("click", info_element, true);
+		new_context->AddEventListener("mouseover", info_element, true);
+		new_context->AddEventListener("mouseout", info_element, true);
+	}
+}
+
+void DebuggerPlugin::ReleaseElements()
+{
+	// Erase event listeners to prevent crashes.
+	if (info_element)
+		SetupInfoListeners(nullptr);
+
 	if (host_context)
 	{
 		if (menu_element)

+ 2 - 0
Source/Debugger/DebuggerPlugin.h

@@ -101,6 +101,8 @@ private:
 	bool LoadInfoElement();
 	bool LoadLogElement();
 
+	void SetupInfoListeners(Rml::Context* new_context);
+
 	// Release all loaded elements
 	void ReleaseElements();