From f51c16c54f7bf0f929b966f1a3ef0ca4de0426b9 Mon Sep 17 00:00:00 2001
From: stephena <stephena@8b62c5a3-ac7e-4cc8-8f21-d9a121418aba>
Date: Tue, 5 Oct 2010 00:15:12 +0000
Subject: [PATCH] Shortened some options in the right-mouse click menu in the
 disassembly view.  Instead of showing each separate item (in related items),
 simply show a 'toggle' option instead.

git-svn-id: svn://svn.code.sf.net/p/stella/code/trunk@2142 8b62c5a3-ac7e-4cc8-8f21-d9a121418aba
---
 src/debugger/gui/RomListWidget.cxx |  8 +++---
 src/debugger/gui/RomWidget.cxx     | 39 ++++++++++++------------------
 2 files changed, 19 insertions(+), 28 deletions(-)

diff --git a/src/debugger/gui/RomListWidget.cxx b/src/debugger/gui/RomListWidget.cxx
index e680d6eb8..99a0c9613 100644
--- a/src/debugger/gui/RomListWidget.cxx
+++ b/src/debugger/gui/RomListWidget.cxx
@@ -64,11 +64,9 @@ RomListWidget::RomListWidget(GuiObject* boss, const GUI::Font& font,
   l.push_back("Save ROM", "saverom");
   l.push_back("Set PC", "setpc");
   l.push_back("RunTo PC", "runtopc");
-  l.push_back("------------------", "");
-  l.push_back("Show PC addresses", "showaddr");
-  l.push_back("Hide PC addresses", "hideaddr");
-  l.push_back("Show GFX as binary", "gfxbin");
-  l.push_back("Show GFX as hex", "gfxhex");
+  l.push_back("---------------------", "");
+  l.push_back("Toggle PC addresses", "pcaddr");
+  l.push_back("Toggle GFX binary/hex", "gfx");
   l.push_back("Re-disassemble", "disasm");
   myMenu = new ContextMenu(this, font, l);
 
diff --git a/src/debugger/gui/RomWidget.cxx b/src/debugger/gui/RomWidget.cxx
index 2404e6b86..8f31d2893 100644
--- a/src/debugger/gui/RomWidget.cxx
+++ b/src/debugger/gui/RomWidget.cxx
@@ -175,35 +175,28 @@ void RomWidget::handleCommand(CommandSender* sender, int cmd, int data, int id)
         runtoPC(myRomList->getSelected());
       else if(rmb == "disasm")
         invalidate();
-      else if(rmb == "showaddr")
+      else if(rmb == "pcaddr")
       {
-        if(DiStella::settings.show_addresses)
-          return;
-        DiStella::settings.show_addresses = true;
-        instance().settings().setString("showaddr", "true");
+        DiStella::settings.show_addresses = !DiStella::settings.show_addresses;
+        instance().settings().setString("showaddr",
+            DiStella::settings.show_addresses ? "true" : "false");
         invalidate();
       }
-      else if(rmb == "hideaddr")
+      else if(rmb == "gfx")
       {
-        if(!DiStella::settings.show_addresses)
-          return;
-        DiStella::settings.show_addresses = false;
-        instance().settings().setString("showaddr", "false");
+        if(DiStella::settings.gfx_format == kBASE_16)
+        {
+          DiStella::settings.gfx_format = kBASE_2;
+          instance().settings().setString("gfxformat", "2");
+        }
+        else
+        {
+          DiStella::settings.gfx_format = kBASE_16;
+          instance().settings().setString("gfxformat", "16");
+        }
         invalidate();
       }
-      else if(rmb == "gfxbin")
-      {
-        DiStella::settings.gfx_format = kBASE_2;
-        instance().settings().setString("gfxformat", "2");
-        invalidate();
-      }
-      else if(rmb == "gfxhex")
-      {
-        DiStella::settings.gfx_format = kBASE_16;
-        instance().settings().setString("gfxformat", "16");
-        invalidate();
-      }
-      break;
+      break;  // kCMenuItemSelectedCmd
     }
 
     case kDGItemDataChangedCmd: