diff options
author | Jakob Heher <jakob.heher@iaik.tugraz.at> | 2022-08-10 11:05:07 +0200 |
---|---|---|
committer | Jakob Heher <jakob.heher@iaik.tugraz.at> | 2022-08-10 11:05:07 +0200 |
commit | 0a1cb4c9b20ca2b95e076961c51e0a4108f38d7e (patch) | |
tree | eaa2b64ae95752bdd94a59278cfe9da09c3393e5 | |
parent | 1fe888810f558c13752b043e4721c838404a0f7f (diff) | |
download | pdf-over-0a1cb4c9b20ca2b95e076961c51e0a4108f38d7e.tar.gz pdf-over-0a1cb4c9b20ca2b95e076961c51e0a4108f38d7e.tar.bz2 pdf-over-0a1cb4c9b20ca2b95e076961c51e0a4108f38d7e.zip |
fix error dialogs in initial config load
-rw-r--r-- | pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachine.java | 2 | ||||
-rw-r--r-- | pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/config/ConfigurationDataInMemory.java | 7 |
2 files changed, 6 insertions, 3 deletions
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachine.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachine.java index f60393a8..28f3a599 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachine.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/StateMachine.java @@ -166,7 +166,7 @@ public class StateMachine implements GUIProvider { this.shell.open(); this.shell.layout(); } catch (Exception e) { - log.warn("Main-Window creation FAILED. Reason: " + e.getMessage()); + log.warn("Main-Window creation FAILED.", e); this.display = null; this.mainWindow = null; this.shell = null; diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/config/ConfigurationDataInMemory.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/config/ConfigurationDataInMemory.java index 9fda1f1f..31b351f8 100644 --- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/config/ConfigurationDataInMemory.java +++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/config/ConfigurationDataInMemory.java @@ -157,8 +157,11 @@ public class ConfigurationDataInMemory { /** Whether to automatically check for updates */ public boolean updateCheck = true; - /** Holds the main window size */ - public Point mainWindowSize = null; + /** Holds the main window size + * + * @IMPORTANT this must always be valid and non-null, even if configuration failed to load for whatever reason (it is used by error handlers!) + */ + public Point mainWindowSize = new Point(Constants.DEFAULT_MAINWINDOW_WIDTH, Constants.DEFAULT_MAINWINDOW_HEIGHT); /** Whether to skip the output state */ public boolean skipFinish = false; |