make symbol loading/unloading work a bit better

fresh-eggs · Jul 22, 2019 · c0d52a2 · c0d52a2
1 parent 25f3229
commit c0d52a2
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 13 deletions.
diff --git a/bsnes/ui-qt/cartridge/cartridge.cpp b/bsnes/ui-qt/cartridge/cartridge.cpp
@@ -109,12 +109,6 @@ bool Cartridge::loadNormal(const char *base) {
   loadMemory(baseName, ".srm", SNES::memory::cartram);
   loadMemory(baseName, ".rtc", SNES::memory::cartrtc);
 
-  #if defined(DEBUGGER)
-    debugger->symbolsCPU->loadFromFile(nall::basename(baseName), ".sym");
-    debugger->symbolsCPU->loadFromFile(nall::basename(baseName), ".cpu.sym");
-    debugger->symbolsSMP->loadFromFile(nall::basename(baseName), ".smp.sym");
-  #endif
-
   fileName = baseName;
   name = notdir(nall::basename(baseName));
 

diff --git a/bsnes/ui-qt/debugger/debugger.cpp b/bsnes/ui-qt/debugger/debugger.cpp
@@ -271,6 +271,16 @@ void Debugger::modifySystemState(unsigned state) {
       SNES::cpuAnalyst.performFullAnalysis();
     }
 
+    symbolsCPU->loadFromString(DEFAULT_SYMBOL_MAP_CPU);
+    symbolsCPU->loadFromFile(nall::basename(symfile), ".sym");
+    symbolsCPU->loadFromFile(nall::basename(symfile), ".cpu.sym");
+    symbolsSMP->loadFromString(DEFAULT_SYMBOL_MAP_SMP);
+    symbolsSMP->loadFromFile(nall::basename(symfile), ".smp.sym");
+    if (SNES::cartridge.has_sa1())
+      symbolsSA1->loadFromFile(nall::basename(symfile), ".sa1.sym");
+    else
+      symbolsSA1->reset();
+
     string data;
     if(config().debugger.saveBreakpoints) {
       breakpointEditor->clear();
@@ -298,8 +308,10 @@ void Debugger::modifySystemState(unsigned state) {
     }
 
     if(config().debugger.saveSymbols) {
-      debugger->symbolsCPU->saveToFile(nall::basename(symfile), ".cpu.sym");
-      debugger->symbolsSMP->saveToFile(nall::basename(symfile), ".smp.sym");
+      symbolsCPU->saveToFile(nall::basename(symfile), ".cpu.sym");
+      symbolsSMP->saveToFile(nall::basename(symfile), ".smp.sym");
+      if (SNES::cartridge.has_sa1())
+        symbolsSA1->saveToFile(nall::basename(symfile), ".sa1.sym");
     }
 
     if(config().debugger.saveBreakpoints) {

diff --git a/bsnes/ui-qt/debugger/debuggerview.cpp b/bsnes/ui-qt/debugger/debuggerview.cpp
@@ -55,17 +55,14 @@ DebuggerView::DebuggerView(RegisterEdit *registers, DisasmProcessor *processor,
     symbolsViewerDialog = new SymbolsView(processor);
     symbolsViewer = new QPushButton("Symbols");
     controlLayout->addWidget(symbolsViewer);
+    connect(symbolsViewer, SIGNAL(clicked(bool)), symbolsViewerDialog, SLOT(show()));
   }
 
   controlLayout->addStretch();
 
-  connect(stepProcessor, SIGNAL(released()), this, SLOT(synchronize()));
+  connect(stepProcessor, SIGNAL(clicked(bool)), this, SLOT(synchronize()));
   connect(traceProcessor, SIGNAL(stateChanged(int)), this, SIGNAL(traceStateChanged(int)));
 
-  if (processor->getSymbols() != NULL) {
-    connect(symbolsViewer, SIGNAL(released()), symbolsViewerDialog, SLOT(show()));
-  }
-
   connect(consoleLayout, SIGNAL(splitterMoved(int,int)), this, SLOT(synchronize()));
 
   synchronize();

diff --git a/bsnes/ui-qt/debugger/disassembler/symbols/symbol_map.cpp b/bsnes/ui-qt/debugger/disassembler/symbols/symbol_map.cpp
@@ -52,6 +52,12 @@ int32_t SymbolMap::getSymbolIndex(uint32_t address) {
   return -1;
 }
 
+// ------------------------------------------------------------------------
+void SymbolMap::reset() {
+  symbols.reset();
+  finishUpdates();
+}
+
 // ------------------------------------------------------------------------
 void SymbolMap::addLocation(uint32_t address, const string &name) {
   addSymbol(address, Symbol::createLocation(address, name));
@@ -186,6 +192,7 @@ void SymbolMap::loadFromString(const string &file) {
     return;
   }
 
+  symbols.reset();
   if (adapter->read(rows, this)) {
     finishUpdates();
   }

diff --git a/bsnes/ui-qt/debugger/disassembler/symbols/symbol_map.moc.hpp b/bsnes/ui-qt/debugger/disassembler/symbols/symbol_map.moc.hpp
@@ -71,6 +71,7 @@ class SymbolMap : public QObject {
 
   typedef nall::linear_vector<Symbols> SymbolsLists;
 
+  void reset();
   void addLocation(uint32_t address, const string &name);
   void addComment(uint32_t address, const string &name);
   void addSymbol(uint32_t address, const Symbol &name);