Do a save on shutdown.

This avoids losing up to 10s worth of data.

In passing also don't try running the scoreboard without autosave,
and make gui use a monospaced font to match terminals

scoreboard.sh

@@ -13,4 +13,4 @@ if [ -x /usr/libexec/java_home ]; then
   JAVA="/usr/libexec/java_home -exec java"
 fi
 
-$JAVA -Done-jar.silent=true -Dorg.eclipse.jetty.server.LEVEL=WARN -jar lib/crg-scoreboard.jar "$GUI" "$@"
+exec $JAVA -Done-jar.silent=true -Dorg.eclipse.jetty.server.LEVEL=WARN -jar lib/crg-scoreboard.jar "$GUI" "$@"

src/com/carolinarollergirls/scoreboard/Main.java

@@ -9,6 +9,7 @@
  */
 
 import java.awt.Dimension;
+import java.awt.Font;
 import java.awt.Toolkit;
 import java.io.File;
 import java.io.IOException;
@@ -84,8 +85,16 @@ public void run() {
         });
 
         // Only start auto-saves once everything is loaded in.
-        new AutoSaveJSONState(jsm, autoSaveDir);
+        final AutoSaveJSONState autosaver = new AutoSaveJSONState(jsm, autoSaveDir);
         jetty.start();
+
+        Runtime.getRuntime().addShutdownHook(new Thread() {
+            @Override
+            public void run() {
+                // Save any changes since last regular autosave before we shutdown.
+                autosaver.run();
+          }
+        });
     }
 
     private void setSystemProperties() {
@@ -139,6 +148,7 @@ private void createGui() {
         guiFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
         guiMessages = new JTextArea();
         guiMessages.setEditable(false);
+        guiMessages.setFont(new Font("monospaced", Font.PLAIN, 12));
         guiFrameText = new JLabel("ScoreBoard status: starting...");
         guiFrame.getContentPane().setLayout(new BoxLayout(guiFrame.getContentPane(), BoxLayout.Y_AXIS));
         guiFrame.getContentPane().add(guiFrameText);

src/com/carolinarollergirls/scoreboard/jetty/ScoreBoardSessionManager.java

@@ -46,7 +46,7 @@ protected void addSession(AbstractSession session) {
 
     @Override
     protected void invalidateSessions() {
-      throw new RuntimeException("Not implemented");
+        // Our session objects are created per request, so nothing to cleanup on shutdown.
     }
 
     @Override

src/com/carolinarollergirls/scoreboard/json/AutoSaveJSONState.java

@@ -32,14 +32,14 @@ public AutoSaveJSONState(JSONStateManager jsm, File dir) {
             FileUtils.forceMkdir(dir);
         } catch ( IOException ioE ) {
             Logger.printMessage("WARNING: Unable to create auto-save directory '"+dir+"' : "+ioE.getMessage());
-            return;
+            throw new RuntimeException(ioE);
         }
         backupAutoSavedFiles();
         executor.scheduleAtFixedRate(AutoSaveJSONState.this, INTERVAL_SECONDS, INTERVAL_SECONDS, TimeUnit.SECONDS);
     }
 
     @Override
-    public void run() {
+    public synchronized void run() {
         Histogram.Timer timer = autosaveDuration.startTimer();
         try {
             int n = AUTOSAVE_FILES;