]> Shamusworld >> Repos - virtualjaguar/blobdiff - src/gui/mainwin.cpp
Fixed DSP/audio options to be unambiguous and consistent.
[virtualjaguar] / src / gui / mainwin.cpp
index af1dab100837dae93376b6338a0c671ccb288e2a..43996880984c307d46bfe1406dc45fe2c75fe83d 100644 (file)
@@ -1,9 +1,9 @@
 //
 // mainwin.cpp - Qt-based GUI for Virtual Jaguar: Main Application Window
-// by James L. Hammons
+// by James Hammons
 // (C) 2009 Underground Software
 //
-// JLH = James L. Hammons <jlhamm@acm.org>
+// JLH = James Hammons <jlhamm@acm.org>
 //
 // Who  When        What
 // ---  ----------  -------------------------------------------------------------
@@ -34,6 +34,7 @@
 #include "mainwin.h"
 
 #include "SDL.h"
+#include "app.h"
 #include "glwidget.h"
 #include "about.h"
 #include "help.h"
 #include "jagstub2bios.h"
 #include "joystick.h"
 
-#ifdef __GCCWIN32__
+// According to SebRmv, this header isn't seen on Arch Linux either... :-/
+//#ifdef __GCCWIN32__
 // Apparently on win32, usleep() is not pulled in by the usual suspects.
 #include <unistd.h>
-#endif
+//#endif
 
 // The way BSNES controls things is by setting a timer with a zero
 // timeout, sleeping if not emulating anything. Seems there has to be a
 // We'll make the VJ core modular so that it doesn't matter what GUI is in
 // use, we can drop it in anywhere and use it as-is.
 
-MainWin::MainWin(): running(true), powerButtonOn(false), showUntunedTankCircuit(true),
-       cartridgeLoaded(false), CDActive(false),//, alpineLoadSuccessful(false),
-       pauseForFileSelector(false)
+//MainWin::MainWin(QString filenameToRun): running(true), powerButtonOn(false),
+MainWin::MainWin(bool autoRun): running(true), powerButtonOn(false),
+       showUntunedTankCircuit(true), cartridgeLoaded(false), CDActive(false),
+       //, alpineLoadSuccessful(false),
+//     pauseForFileSelector(false), loadAndGo(false), plzDontKillMyComputer(false)
+       pauseForFileSelector(false), loadAndGo(autoRun), plzDontKillMyComputer(false)
 {
+       for(int i=0; i<8; i++)
+               keyHeld[i] = false;
+
        videoWidget = new GLWidget(this);
        setCentralWidget(videoWidget);
        setWindowIcon(QIcon(":/res/vj-icon.png"));
@@ -185,7 +193,7 @@ MainWin::MainWin(): running(true), powerButtonOn(false), showUntunedTankCircuit(
        useCDAct->setCheckable(true);
        connect(useCDAct, SIGNAL(triggered()), this, SLOT(ToggleCDUsage()));
 
-       frameAdvanceAct = new QAction(QIcon(":/res/generic.png"), tr("&Frame Advance"), this);
+       frameAdvanceAct = new QAction(QIcon(":/res/frame-advance.png"), tr("&Frame Advance"), this);
        frameAdvanceAct->setShortcut(QKeySequence(tr("F7")));
        connect(frameAdvanceAct, SIGNAL(triggered()), this, SLOT(FrameAdvance()));
 
@@ -228,16 +236,6 @@ MainWin::MainWin(): running(true), powerButtonOn(false), showUntunedTankCircuit(
 
        ReadSettings();
 
-       // Set toolbar buttons/menus based on settings read in (sync the UI)...
-       blurAct->setChecked(vjs.glFilter);
-       x1Act->setChecked(zoomLevel == 1);
-       x2Act->setChecked(zoomLevel == 2);
-       x3Act->setChecked(zoomLevel == 3);
-//     running = powerAct->isChecked();
-       ntscAct->setChecked(vjs.hardwareTypeNTSC);
-       palAct->setChecked(!vjs.hardwareTypeNTSC);
-       powerAct->setIcon(vjs.hardwareTypeNTSC ? powerRed : powerGreen);
-
        // Do this in case original size isn't correct (mostly for the first-run case)
        ResizeMainWindow();
 
@@ -258,8 +256,19 @@ MainWin::MainWin(): running(true), powerButtonOn(false), showUntunedTankCircuit(
        WriteLog("Virtual Jaguar %s (Last full build was on %s %s)\n", VJ_RELEASE_VERSION, __DATE__, __TIME__);
        WriteLog("VJ: Initializing jaguar subsystem...\n");
        JaguarInit();
+       memcpy(jagMemSpace + 0xE00000, jaguarBootROM, 0x20000); // Use the stock BIOS
 
-       filePickWin->ScanSoftwareFolder(allowUnknownSoftware);
+       // Check for filename passed in on the command line...
+//     if (!filenameToRun.isEmpty())
+       if (autoRun)
+       {
+//             loadAndGo = true;
+               // Attempt to load/run the file the user passed in...
+//             LoadSoftware(filenameToRun);
+////           memcpy(jagMemSpace + 0xE00000, jaguarBootROM, 0x20000); // Use the stock BIOS
+               // Prevent the file scanner from running...
+               return;
+       }
 
        // Load up the default ROM if in Alpine mode:
        if (vjs.hardwareTypeAlpine)
@@ -279,18 +288,48 @@ MainWin::MainWin(): running(true), powerButtonOn(false), showUntunedTankCircuit(
                // Attempt to load/run the ABS file...
                LoadSoftware(vjs.absROMPath);
                memcpy(jagMemSpace + 0xE00000, jaguarDevBootROM2, 0x20000);     // Use the stub BIOS
+               // Prevent the scanner from running...
+               return;
        }
-       else
-               memcpy(jagMemSpace + 0xE00000, jaguarBootROM, 0x20000); // Otherwise, use the stock BIOS
+//     else
+//             memcpy(jagMemSpace + 0xE00000, jaguarBootROM, 0x20000); // Otherwise, use the stock BIOS
+
+       // Run the scanner if nothing passed in and *not* Alpine mode...
+       // NB: Really need to look into caching the info scanned in here...
+       filePickWin->ScanSoftwareFolder(allowUnknownSoftware);
 }
 
+
+void MainWin::LoadFile(QString file)
+{
+       LoadSoftware(file);
+}
+
+
+void MainWin::SyncUI(void)
+{
+       // Set toolbar buttons/menus based on settings read in (sync the UI)...
+       blurAct->setChecked(vjs.glFilter);
+       x1Act->setChecked(zoomLevel == 1);
+       x2Act->setChecked(zoomLevel == 2);
+       x3Act->setChecked(zoomLevel == 3);
+//     running = powerAct->isChecked();
+       ntscAct->setChecked(vjs.hardwareTypeNTSC);
+       palAct->setChecked(!vjs.hardwareTypeNTSC);
+       powerAct->setIcon(vjs.hardwareTypeNTSC ? powerRed : powerGreen);
+}
+
+
 void MainWin::closeEvent(QCloseEvent * event)
 {
        JaguarDone();
-       WriteSettings();
+// This should only be done by the config dialog
+//     WriteSettings();
+       WriteUISettings();
        event->accept(); // ignore() if can't close for some reason
 }
 
+
 void MainWin::keyPressEvent(QKeyEvent * e)
 {
        // We ignore the Alt key for now, since it causes problems with the GUI
@@ -303,6 +342,7 @@ void MainWin::keyPressEvent(QKeyEvent * e)
        HandleKeys(e, true);
 }
 
+
 void MainWin::keyReleaseEvent(QKeyEvent * e)
 {
        // We ignore the Alt key for now, since it causes problems with the GUI
@@ -315,12 +355,18 @@ void MainWin::keyReleaseEvent(QKeyEvent * e)
        HandleKeys(e, false);
 }
 
+
 void MainWin::HandleKeys(QKeyEvent * e, bool state)
 {
+       enum { P1LEFT = 0, P1RIGHT, P1UP, P1DOWN, P2LEFT, P2RIGHT, P2UP, P2DOWN };
        // We kill bad key combos here, before they can get to the emulator...
        // This also kills the illegal instruction problem that cropped up in Rayman!
        // May want to do this by killing the old one instead of ignoring the new one...
        // Seems to work better that way...
+
+// The problem with this approach is that it causes bad results because it doesn't do
+// any checking of previous states. Need to come up with something better because this
+// causes problems where the keyboard acts as if it were unresponsive. :-P
 #if 0
        if ((e->key() == vjs.p1KeyBindings[BUTTON_L] && joypad_0_buttons[BUTTON_R])
                || (e->key() == vjs.p1KeyBindings[BUTTON_R] && joypad_0_buttons[BUTTON_L])
@@ -328,6 +374,7 @@ void MainWin::HandleKeys(QKeyEvent * e, bool state)
                || (e->key() == vjs.p1KeyBindings[BUTTON_D] && joypad_0_buttons[BUTTON_U]))
                return;
 #else
+#if 0
        if (e->key() == (int)vjs.p1KeyBindings[BUTTON_L] && joypad_0_buttons[BUTTON_R])
                joypad_0_buttons[BUTTON_R] = 0;
        if (e->key() == (int)vjs.p1KeyBindings[BUTTON_R] && joypad_0_buttons[BUTTON_L])
@@ -345,6 +392,33 @@ void MainWin::HandleKeys(QKeyEvent * e, bool state)
                joypad_1_buttons[BUTTON_D] = 0;
        if (e->key() == (int)vjs.p2KeyBindings[BUTTON_D] && joypad_1_buttons[BUTTON_U])
                joypad_1_buttons[BUTTON_U] = 0;
+#else
+//hrm, this still has sticky state problems... Ugh!
+       // First, settle key states...
+       if (e->key() == (int)vjs.p1KeyBindings[BUTTON_L])
+               keyHeld[P1LEFT] = state;
+       else if (e->key() == (int)vjs.p1KeyBindings[BUTTON_R])
+               keyHeld[P1RIGHT] = state;
+       else if (e->key() == (int)vjs.p1KeyBindings[BUTTON_U])
+               keyHeld[P1UP] = state;
+       else if (e->key() == (int)vjs.p1KeyBindings[BUTTON_D])
+               keyHeld[P1DOWN] = state;
+       else if (e->key() == (int)vjs.p2KeyBindings[BUTTON_L])
+               keyHeld[P2LEFT] = state;
+       else if (e->key() == (int)vjs.p2KeyBindings[BUTTON_R])
+               keyHeld[P2RIGHT] = state;
+       else if (e->key() == (int)vjs.p2KeyBindings[BUTTON_U])
+               keyHeld[P2UP] = state;
+       else if (e->key() == (int)vjs.p2KeyBindings[BUTTON_D])
+               keyHeld[P2DOWN] = state;
+
+       // Next, check for conflicts and bail out if there are any...
+       if ((keyHeld[P1LEFT] && keyHeld[P1RIGHT])
+               || (keyHeld[P1UP] && keyHeld[P1DOWN])
+               || (keyHeld[P2LEFT] && keyHeld[P2RIGHT])
+               || (keyHeld[P2UP] && keyHeld[P2DOWN]))
+               return;
+#endif
 #endif
 
        // No bad combos exist, let's stuff the emulator key buffers...!
@@ -362,10 +436,12 @@ void MainWin::HandleKeys(QKeyEvent * e, bool state)
        }
 }
 
+
 void MainWin::Open(void)
 {
 }
 
+
 void MainWin::Configure(void)
 {
        // Call the configuration dialog and update settings
@@ -379,12 +455,14 @@ void MainWin::Configure(void)
        QString before = vjs.ROMPath;
        QString alpineBefore = vjs.alpineROMPath;
        QString absBefore = vjs.absROMPath;
-       bool audioBefore = vjs.audioEnabled;
+//     bool audioBefore = vjs.audioEnabled;
+       bool audioBefore = vjs.DSPEnabled;
        dlg.UpdateVJSettings();
        QString after = vjs.ROMPath;
        QString alpineAfter = vjs.alpineROMPath;
        QString absAfter = vjs.absROMPath;
-       bool audioAfter = vjs.audioEnabled;
+//     bool audioAfter = vjs.audioEnabled;
+       bool audioAfter = vjs.DSPEnabled;
 
        bool allowOld = allowUnknownSoftware;
        //ick.
@@ -421,7 +499,8 @@ void MainWin::Configure(void)
                }
        }
 
-       // If the "Enable audio" checkbox changed, then we have to re-init the DAC...
+       // If the "Enable DSP" checkbox changed, then we have to re-init the DAC,
+       // since it's running in the host audio IRQ...
        if (audioBefore != audioAfter)
        {
                DACDone();
@@ -432,6 +511,7 @@ void MainWin::Configure(void)
        WriteSettings();
 }
 
+
 //
 // Here's the main emulator loop
 //
@@ -442,6 +522,9 @@ void MainWin::Timer(void)
 
        if (showUntunedTankCircuit)
        {
+               // Some machines can't handle this, so we give them the option to disable it. :-)
+               if (!plzDontKillMyComputer)
+               {
                // Random hash & trash
                // We try to simulate an untuned tank circuit here... :-)
                for(uint32_t x=0; x<videoWidget->rasterWidth; x++)
@@ -452,6 +535,7 @@ void MainWin::Timer(void)
                                        = (rand() & 0xFF) << 8 | (rand() & 0xFF) << 16 | (rand() & 0xFF) << 24;
                        }
                }
+               }
        }
        else
        {
@@ -462,6 +546,7 @@ void MainWin::Timer(void)
        videoWidget->updateGL();
 }
 
+
 void MainWin::TogglePowerState(void)
 {
        powerButtonOn = !powerButtonOn;
@@ -506,6 +591,7 @@ void MainWin::TogglePowerState(void)
        }
 }
 
+
 void MainWin::ToggleRunState(void)
 {
        running = !running;
@@ -524,55 +610,67 @@ void MainWin::ToggleRunState(void)
        }
 }
 
+
 void MainWin::SetZoom100(void)
 {
        zoomLevel = 1;
        ResizeMainWindow();
 }
 
+
 void MainWin::SetZoom200(void)
 {
        zoomLevel = 2;
        ResizeMainWindow();
 }
 
+
 void MainWin::SetZoom300(void)
 {
        zoomLevel = 3;
        ResizeMainWindow();
 }
 
+
 void MainWin::SetNTSC(void)
 {
        powerAct->setIcon(powerRed);
        timer->setInterval(16);
        vjs.hardwareTypeNTSC = true;
        ResizeMainWindow();
+       WriteSettings();
 }
 
+
 void MainWin::SetPAL(void)
 {
        powerAct->setIcon(powerGreen);
        timer->setInterval(20);
        vjs.hardwareTypeNTSC = false;
        ResizeMainWindow();
+       WriteSettings();
 }
 
+
 void MainWin::ToggleBlur(void)
 {
        vjs.glFilter = !vjs.glFilter;
+       WriteSettings();
 }
 
+
 void MainWin::ShowAboutWin(void)
 {
        aboutWin->show();
 }
 
+
 void MainWin::ShowHelpWin(void)
 {
        helpWin->show();
 }
 
+
 void MainWin::InsertCart(void)
 {
        // If the emulator is running, we pause it here and unpause it later
@@ -586,16 +684,21 @@ void MainWin::InsertCart(void)
        filePickWin->show();
 }
 
+
 void MainWin::Unpause(void)
 {
        // Here we unpause the emulator if it was paused when we went into the file selector
        if (pauseForFileSelector)
        {
                pauseForFileSelector = false;
-               ToggleRunState();
+
+               // Some nutter might have unpaused while in the file selector, so check for that
+               if (!running)
+                       ToggleRunState();
        }
 }
 
+
 void MainWin::LoadSoftware(QString file)
 {
        running = false;                                                        // Prevent bad things(TM) from happening...
@@ -615,7 +718,7 @@ void MainWin::LoadSoftware(QString file)
        powerButtonOn = false;
        TogglePowerState();
 
-       if (!vjs.hardwareTypeAlpine)
+       if (!vjs.hardwareTypeAlpine && !loadAndGo)
        {
                QString newTitle = QString("Virtual Jaguar " VJ_RELEASE_VERSION " - Now playing: %1")
                        .arg(filePickWin->GetSelectedPrettyName());
@@ -623,6 +726,7 @@ void MainWin::LoadSoftware(QString file)
        }
 }
 
+
 void MainWin::ToggleCDUsage(void)
 {
        CDActive = !CDActive;
@@ -645,6 +749,7 @@ void MainWin::ToggleCDUsage(void)
 #endif
 }
 
+
 void MainWin::FrameAdvance(void)
 {
 //printf("Frame Advance...\n");
@@ -653,6 +758,7 @@ void MainWin::FrameAdvance(void)
        videoWidget->updateGL();
 }
 
+
 void MainWin::ResizeMainWindow(void)
 {
        videoWidget->setFixedSize(zoomLevel * 320, zoomLevel * (vjs.hardwareTypeNTSC ? 240 : 256));
@@ -666,6 +772,9 @@ void MainWin::ResizeMainWindow(void)
        }
 }
 
+
+#warning "!!! Need to check the window geometry to see if the positions are legal !!!"
+// i.e., someone could drag it to another screen, close it, then disconnect that screen
 void MainWin::ReadSettings(void)
 {
        QSettings settings("Underground Software", "Virtual Jaguar");
@@ -706,6 +815,7 @@ WriteLog("   EEPROMPath = \"%s\"\n", vjs.EEPROMPath);
 WriteLog("      ROMPath = \"%s\"\n", vjs.ROMPath);
 WriteLog("AlpineROMPath = \"%s\"\n", vjs.alpineROMPath);
 WriteLog("   absROMPath = \"%s\"\n", vjs.absROMPath);
+WriteLog("Pipelined DSP = %s\n", (vjs.usePipelinedDSP ? "ON" : "off"));
 
        // Keybindings in order of U, D, L, R, C, B, A, Op, Pa, 0-9, #, *
        vjs.p1KeyBindings[BUTTON_U] = settings.value("p1k_up", Qt::Key_S).toInt();
@@ -753,6 +863,7 @@ WriteLog("   absROMPath = \"%s\"\n", vjs.absROMPath);
        vjs.p2KeyBindings[BUTTON_s] = settings.value("p2k_star", Qt::Key_Asterisk).toInt();
 }
 
+
 void MainWin::WriteSettings(void)
 {
        QSettings settings("Underground Software", "Virtual Jaguar");
@@ -828,3 +939,14 @@ void MainWin::WriteSettings(void)
        settings.setValue("p2k_pound", vjs.p2KeyBindings[BUTTON_d]);
        settings.setValue("p2k_star", vjs.p2KeyBindings[BUTTON_s]);
 }
+
+
+void MainWin::WriteUISettings(void)
+{
+       QSettings settings("Underground Software", "Virtual Jaguar");
+       settings.setValue("pos", pos());
+       settings.setValue("size", size());
+       settings.setValue("cartLoadPos", filePickWin->pos());
+
+       settings.setValue("zoom", zoomLevel);
+}