From 3b21d0b77124a26ef97922220c29e17e91d178f5 Mon Sep 17 00:00:00 2001 From: Dan <46821332+nsadeveloper789@users.noreply.github.com> Date: Mon, 15 Apr 2024 16:33:32 -0400 Subject: [PATCH] GP-4485: Re-configure default Debugger.tool for Trace RMI --- .../TraceRmiConnectionManagerPlugin.java | 2 +- .../TraceRmiLauncherServicePlugin.java | 2 +- .../debug/gui/model/DebuggerModelPlugin.java | 2 +- .../debug/gui/model/ObjectsTablePanel.java | 2 +- .../gui/objects/DebuggerObjectsPlugin.java | 2 +- .../main/resources/defaultTools/Debugger.tool | 1122 ++++------------- 6 files changed, 264 insertions(+), 868 deletions(-) diff --git a/Ghidra/Debug/Debugger-rmi-trace/src/main/java/ghidra/app/plugin/core/debug/gui/tracermi/connection/TraceRmiConnectionManagerPlugin.java b/Ghidra/Debug/Debugger-rmi-trace/src/main/java/ghidra/app/plugin/core/debug/gui/tracermi/connection/TraceRmiConnectionManagerPlugin.java index 81640171bfb..ef77ed3021f 100644 --- a/Ghidra/Debug/Debugger-rmi-trace/src/main/java/ghidra/app/plugin/core/debug/gui/tracermi/connection/TraceRmiConnectionManagerPlugin.java +++ b/Ghidra/Debug/Debugger-rmi-trace/src/main/java/ghidra/app/plugin/core/debug/gui/tracermi/connection/TraceRmiConnectionManagerPlugin.java @@ -31,7 +31,7 @@ """, category = PluginCategoryNames.DEBUGGER, packageName = DebuggerPluginPackage.NAME, - status = PluginStatus.STABLE, + status = PluginStatus.RELEASED, eventsConsumed = { TraceActivatedPluginEvent.class, TraceInactiveCoordinatesPluginEvent.class, diff --git a/Ghidra/Debug/Debugger-rmi-trace/src/main/java/ghidra/app/plugin/core/debug/gui/tracermi/launcher/TraceRmiLauncherServicePlugin.java b/Ghidra/Debug/Debugger-rmi-trace/src/main/java/ghidra/app/plugin/core/debug/gui/tracermi/launcher/TraceRmiLauncherServicePlugin.java index ebd93ba0083..3f15ebe1cb0 100644 --- a/Ghidra/Debug/Debugger-rmi-trace/src/main/java/ghidra/app/plugin/core/debug/gui/tracermi/launcher/TraceRmiLauncherServicePlugin.java +++ b/Ghidra/Debug/Debugger-rmi-trace/src/main/java/ghidra/app/plugin/core/debug/gui/tracermi/launcher/TraceRmiLauncherServicePlugin.java @@ -58,7 +58,7 @@ """, category = PluginCategoryNames.DEBUGGER, packageName = DebuggerPluginPackage.NAME, - status = PluginStatus.UNSTABLE, + status = PluginStatus.RELEASED, eventsConsumed = { ProgramActivatedPluginEvent.class, ProgramClosedPluginEvent.class, diff --git a/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/model/DebuggerModelPlugin.java b/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/model/DebuggerModelPlugin.java index d23f124983b..e08b8abe38b 100644 --- a/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/model/DebuggerModelPlugin.java +++ b/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/model/DebuggerModelPlugin.java @@ -34,7 +34,7 @@ description = "GUI to browse objects recorded to the trace", category = PluginCategoryNames.DEBUGGER, packageName = DebuggerPluginPackage.NAME, - status = PluginStatus.STABLE, + status = PluginStatus.RELEASED, eventsConsumed = { TraceActivatedPluginEvent.class, TraceClosedPluginEvent.class, diff --git a/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/model/ObjectsTablePanel.java b/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/model/ObjectsTablePanel.java index 55a447497a6..f884775a311 100644 --- a/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/model/ObjectsTablePanel.java +++ b/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/model/ObjectsTablePanel.java @@ -99,7 +99,7 @@ protected String computePreferenceKey() { if (rootSchema == null) { return DEFAULT_PREF_KEY; } - return rootSchema + ":" + schemas + return rootSchema.getName() + ":" + schemas .stream() .map(s -> s.getName().toString()) .collect(Collectors.joining(",")) + diff --git a/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/objects/DebuggerObjectsPlugin.java b/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/objects/DebuggerObjectsPlugin.java index 3df7b586565..46a3a8f1363 100644 --- a/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/objects/DebuggerObjectsPlugin.java +++ b/Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/gui/objects/DebuggerObjectsPlugin.java @@ -63,7 +63,7 @@ public class DebuggerObjectsPlugin extends AbstractDebuggerPlugin implements ObjectUpdateService, CollectionChangeListener { - static String TITLE_PROVIDER_TARGETS = "Debugger Objects"; + static final String TITLE_PROVIDER_TARGETS = "Debugger Objects"; @AutoServiceConsumed protected DebuggerInterpreterService interpreterService; diff --git a/Ghidra/Debug/Debugger/src/main/resources/defaultTools/Debugger.tool b/Ghidra/Debug/Debugger/src/main/resources/defaultTools/Debugger.tool index 5b4175d6f53..e9273dd3051 100644 --- a/Ghidra/Debug/Debugger/src/main/resources/defaultTools/Debugger.tool +++ b/Ghidra/Debug/Debugger/src/main/resources/defaultTools/Debugger.tool @@ -1,151 +1,175 @@ - + - - - - + - - + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + - + - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - + - + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - - + - - - + - + - + - + - + - + - + - + - + - + - + - + - + - + @@ -155,315 +179,118 @@ - - - - - - + - - - - - - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + - - - - - - + + + + - + + - + - - - - - + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + - - - - + + + - + - - - - + + + + + + - + - - - - - - - - - - - - - - - - - - + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + - - - - - + + + + + + - + - + - - - - - + + + + + + @@ -471,12 +298,13 @@ - - - - - - + + + + + + + @@ -484,587 +312,155 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + - + - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + - + - - - - + + + + - + - - - - - + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + - + - - - - - - + + + + + + + + - + + + - - - - - - - - - - - - - - - - - + - - - - - - + + + + + - + - - - - - - - - - - - - - - - - - - - + + + + + + + + + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - +