summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJakob Heher <jakob.heher@iaik.tugraz.at>2022-08-10 11:05:07 +0200
committerJakob Heher <jakob.heher@iaik.tugraz.at>2022-08-10 11:05:07 +0200
commit0a1cb4c9b20ca2b95e076961c51e0a4108f38d7e (patch)
treeeaa2b64ae95752bdd94a59278cfe9da09c3393e5
parent1fe888810f558c13752b043e4721c838404a0f7f (diff)
downloadpdf-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.java2
-rw-r--r--pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/config/ConfigurationDataInMemory.java7
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;