changeset 761:641b7693fecc

Cosmetic cleanups.
author Matti Hamalainen <ccr@tnsp.org>
date Mon, 05 Nov 2012 20:04:05 +0200
parents f6a57c0656fa
children 03c5cde1dfbb
files src/xs_config.c src/xs_sidplay2.cc
diffstat 2 files changed, 26 insertions(+), 38 deletions(-) [+]
line wrap: on
line diff
--- a/src/xs_config.c	Mon Nov 05 20:03:14 2012 +0200
+++ b/src/xs_config.c	Mon Nov 05 20:04:05 2012 +0200
@@ -221,17 +221,15 @@
     xs_cfg.sid1Filter.fm = XS_SIDPLAY1_FM;
     xs_cfg.sid1Filter.ft = XS_SIDPLAY1_FT;
 
-#ifdef HAVE_SIDPLAY2
+#if defined(HAVE_SIDPLAY2)
     xs_cfg.playerEngine = XS_ENG_SIDPLAY2;
     xs_cfg.memoryMode = XS_MPU_REAL;
-#else
-#ifdef HAVE_SIDPLAY1
+#elif defined(HAVE_SIDPLAY1)
     xs_cfg.playerEngine = XS_ENG_SIDPLAY1;
     xs_cfg.memoryMode = XS_MPU_BANK_SWITCHING;
 #else
 #error This should not happen! No emulator engines configured in!
 #endif
-#endif
 
     xs_cfg.clockSpeed = XS_CLOCK_PAL;
     xs_cfg.forceSpeed = FALSE;
@@ -239,16 +237,12 @@
     xs_cfg.sid2OptLevel = 0;
     xs_cfg.sid2NFilterPresets = 0;
 
-#ifdef HAVE_RESID_BUILDER
+#if defined(HAVE_RESID_BUILDER)
     xs_cfg.sid2Builder = XS_BLD_RESID;
-#else
-#ifdef HAVE_HARDSID_BUILDER
+#elif defined(HAVE_HARDSID_BUILDER)
     xs_cfg.sid2Builder = XS_BLD_HARDSID;
-#else
-#ifdef HAVE_SIDPLAY2
-#error This should not happen! No supported SIDPlay2 builders configured in!
-#endif
-#endif
+#elif defined(HAVE_SIDPLAY2)
+#  error This should not happen! No supported SIDPlay2 builders configured in!
 #endif
 
     xs_cfg.playMaxTimeEnable = FALSE;
@@ -1249,26 +1243,13 @@
 
     gtk_widget_set_sensitive(LUW("cfg_sidplay2_frame"), isActive);
 
-#ifdef HAVE_SIDPLAY2_DISTORTION
     /* Optimization mode removed from distortion patch */
-    gtk_widget_set_sensitive(LUW("cfg_emu_sp2_opt"), FALSE);
-#else
-    gtk_widget_set_sensitive(LUW("cfg_emu_sp2_opt"), isActive);
-#endif
+    gtk_widget_set_sensitive(LUW("cfg_emu_sp2_opt"), HAVE_SIDPLAY2_FP && isActive);
 
     gtk_widget_set_sensitive(LUW("cfg_chn_autopan"), !isActive);
 
-#ifdef HAVE_RESID_BUILDER
-    gtk_widget_set_sensitive(LUW("cfg_emu_sp2_resid"), isActive);
-#else
-    gtk_widget_set_sensitive(LUW("cfg_emu_sp2_resid"), FALSE);
-#endif
-
-#ifdef HAVE_HARDSID_BUILDER
-    gtk_widget_set_sensitive(LUW("cfg_emu_sp2_hardsid"), isActive);
-#else
-    gtk_widget_set_sensitive(LUW("cfg_emu_sp2_hardsid"), FALSE);
-#endif
+    gtk_widget_set_sensitive(LUW("cfg_emu_sp2_resid"), HAVE_RESID_BUILDER && isActive);
+    gtk_widget_set_sensitive(LUW("cfg_emu_sp2_hardsid"), HAVE_HARDSID_BUILDER && isActive);
 }
 
 
--- a/src/xs_sidplay2.cc	Mon Nov 05 20:03:14 2012 +0200
+++ b/src/xs_sidplay2.cc	Mon Nov 05 20:04:05 2012 +0200
@@ -265,19 +265,26 @@
 
     /* Initialize builder object */
     XSDEBUG("init builder #%i, maxsids=%i\n", xs_cfg.sid2Builder, (engine->currEng->info()).maxsids);
+
 #ifdef HAVE_RESID_BUILDER
     if (xs_cfg.sid2Builder == XS_BLD_RESID) {
+
 #ifdef HAVE_SIDPLAY2_FP
-        ReSIDBuilder *rs = new ReSIDBuilder("ReSID builder");
+        ReSIDBuilder *rs = new ReSIDBuilder("ReSID builder FP!");
         engine->currBuilder = (sidbuilder *) rs;
         if (rs) {
+            rs->create((engine->currEng->info()).maxsids);
+            if (!*rs) {
+                xs_error("reSID->create() failed.\n");
+                return FALSE;
+            }
+            rs->bias(0.0f);
+        }
 #else
-        engine->currBuilder = ReSIDBuilderCreate("");
+        engine->currBuilder = ReSIDBuilderCreate("ReSID builder");
         SidLazyIPtr<IReSIDBuilder> rs(engine->currBuilder);
         if (rs) {
             engine->currConfig.sidEmulation = rs->iaggregate();
-#endif
-            /* Builder object created, initialize it */
             rs->create((engine->currEng->info()).maxsids);
             if (!*rs) {
                 xs_error("reSID->create() failed.\n");
@@ -309,6 +316,7 @@
                 return FALSE;
             }
         }
+#endif
     }
 #endif
 
@@ -377,16 +385,15 @@
     }
     
 
-#if !defined(HAVE_SIDPLAY2_FP)
-    if ((xs_cfg.sid2OptLevel >= 0) && (xs_cfg.sid2OptLevel <= SID2_MAX_OPTIMISATION))
-        engine->currConfig.optimisation = xs_cfg.sid2OptLevel;
-    else {
+#ifndef HAVE_SIDPLAY2_FP
+    if (xs_cfg.sid2OptLevel < 0 || xs_cfg.sid2OptLevel > SID2_MAX_OPTIMISATION)
+    {
         xs_error("Invalid sid2OptLevel=%d, falling back to %d.\n",
             xs_cfg.sid2OptLevel, SID2_DEFAULT_OPTIMISATION);
         
-        xs_cfg.sid2OptLevel =
-        engine->currConfig.optimisation = SID2_DEFAULT_OPTIMISATION;
+        xs_cfg.sid2OptLevel = SID2_DEFAULT_OPTIMISATION;
     }
+    engine->currConfig.optimisation = xs_cfg.sid2OptLevel;
 #endif
 
     if (xs_cfg.mos8580)