Open recent menu. Close #4.

Author: travisgoodspeed

README.md

@@ -42,7 +42,7 @@ These decodings are not yet solved, which might make them an interesting target.
 `master` -- GoodASM REPL now has working autocompletion on Windows.
 Dropping support for x86_64 macOS.  Optional support for Yara-X.  HSL
 color space for bit thresholds, with bit color histograms that are
-easier on the eyes.
+easier on the eyes.  Open recent menu, QSettings.
 
 2025-06-01 -- CLI and GUI features to work with ambiguous/damaged
 bits, useful when part of an implant ROM is poorly exposed.  Solver

maskromtool.cpp

@@ -74,6 +74,11 @@ MaskRomTool::MaskRomTool(QWidget *parent, bool opengl)
     : QMainWindow(parent)
     , ui(new Ui::MaskRomTool) {
 
+    //Set our Qt configuration, needed for settings.
+    QCoreApplication::setOrganizationName("TravisGoodspeed");
+    QCoreApplication::setOrganizationDomain("kk4vcz.com");
+    QCoreApplication::setApplicationName("MaskRomTool");
+
     //Setup caches
     QPixmapCache::setCacheLimit(0x100000); //1GB
     if(verbose) qDebug()<<"Cache limit is "<<QPixmapCache::cacheLimit()<<"kb";
@@ -117,6 +122,9 @@ MaskRomTool::MaskRomTool(QWidget *parent, bool opengl)
     //Set up the second view.
     second.view->setScene(scene);
 
+    //Build a history of past entries.
+    buildFileHistoryList();
+
     //Enable OpenGL without antialiasing, now that it's stable.
     if(opengl)
         enableOpenGL(0);
@@ -1370,6 +1378,61 @@ void MaskRomTool::getAlignSkipCountThreshold(uint32_t &count){
     count=alignSkipThreshold;
 }
 
+//Records a file in settings for the history.
+void MaskRomTool::recordFileHistory(QFileInfo file){
+    /* What we do here is build a stack of the old filenames,
+     * cull the new filename from that stack, and then ensure
+     * that the new filename comes first in the new listing.
+     */
+    QSettings settings;
+    QString path=file.absoluteFilePath();
+    QStack<QString> filenames;  //Ordered list of filenames.
+
+    //Build the new list.
+    filenames.append(path);     //New file always comes first.
+    for(int i=0; i<10; i++){
+        QString k="filename"+QString::number(i);
+        QVariant v=settings.value(k);
+        if(v.isValid()){
+            if(path!=v.toString())
+                filenames.append(v.toString());
+        }
+    }
+
+    //Write back the new list.
+    int i=0;
+    foreach(QString f, filenames){
+        QString k="filename"+QString::number(i);
+        settings.setValue(k, f);
+        i++;
+    }
+}
+
+//Builds the history menu entries.
+void MaskRomTool::buildFileHistoryList(){
+    QMenu* recents=ui->menuOpen_Recent;
+    recents->clear();
+
+    QSettings settings;
+    for(int i=0; i<10; i++){
+        QString k="filename"+QString::number(i);
+        QVariant v=settings.value(k);
+        if(v.isValid()){
+            QAction *action=new QAction(recents);
+            action->setText(v.toString());
+            recents->addAction(action);
+            QObject::connect(action, SIGNAL(triggered(bool)),
+                             this, SLOT(openHistory()));
+        }
+    }
+}
+
+//Opens whichever entry in the history was selected.
+void MaskRomTool::openHistory(){
+    QAction *action = (QAction*) this->sender();
+    fileOpen(action->text());
+}
+
 //Opens *either* an image or a JSON description of lines.
 void MaskRomTool::fileOpen(QString filename){
     //On macOS, each window likes to have a file attached.
@@ -1392,8 +1455,12 @@ void MaskRomTool::fileOpen(QString filename){
 
 
         //We load the image as a background, so it's not really a part of the scene.
+        QFileInfo info(imagefilename);
+        recordFileHistory(info);
+        buildFileHistoryList();
         if(verbose)
-            qDebug()<<"Loading background image: "<<imagefilename;
+            qDebug()<<"Loading background image: "<<info.absoluteFilePath();
+
         background=QImage(imagefilename);
         backgroundpixmap = scene->addPixmap(QPixmap(imagefilename));
         backgroundpixmap->setZValue(-10);

maskromtool.h

@@ -44,6 +44,10 @@ class MaskRomTool : public QMainWindow{
     ~MaskRomTool();
     void closeEvent(QCloseEvent *event);
 
+    //Records a file in settings for the history.
+    void recordFileHistory(QFileInfo file);
+    void buildFileHistoryList();
+
     void enableVerbose(unsigned int level=1);
     void enableOpenGL(unsigned int antialiasing=0);
 
@@ -254,6 +258,7 @@ class MaskRomTool : public QMainWindow{
 
 private slots:
     void on_openButton_triggered();
+    void openHistory();              //Called by any of the file history entries.
     void on_saveButton_triggered();
 
     void on_alignconstrainButton_triggered();
@@ -269,7 +274,7 @@ private slots:
     void on_asciiButton_triggered();
     void on_actionHexView_triggered();
 
-    //Import from a file.  (Loose definition.
+    //Import from a file.  (Loose definition.)
     void on_importDiff_triggered();
 
     //Export to a file.
@@ -306,13 +311,9 @@ private slots:
     void on_selectioncolorButton_triggered();
     void on_crosshaircolorButton_triggered();
     void on_exportSolverSetBytes_triggered();
-
     void on_actionHexDamage_triggered();
-
     void on_actionAsciiDamage_triggered();
-
     void on_exportAsciiDamage_triggered();
-
     void on_exportDamageBytes_triggered();
 
 private:

maskromtool.ui

@@ -43,7 +43,7 @@
      <x>0</x>
      <y>0</y>
      <width>1095</width>
-     <height>24</height>
+     <height>20</height>
     </rect>
    </property>
    <widget class="QMenu" name="menuFile">
@@ -72,11 +72,17 @@
      </property>
      <addaction name="importDiff"/>
     </widget>
+    <widget class="QMenu" name="menuOpen_Recent">
+     <property name="title">
+      <string>Open Recent</string>
+     </property>
+    </widget>
     <addaction name="openButton"/>
     <addaction name="saveButton"/>
     <addaction name="menuImport"/>
     <addaction name="menuExport"/>
     <addaction name="actionPrint"/>
+    <addaction name="menuOpen_Recent"/>
    </widget>
    <widget class="QMenu" name="menuView">
     <property name="title">