From 51bc553e73eaa6db59dd2c8ff959884a982c869c Mon Sep 17 00:00:00 2001 From: Nathan Crawford Date: Sun, 3 Jun 2018 13:38:47 -0400 Subject: [PATCH] Attempt to stop crash related to settings file when closing --- embroideryReader/frmMain.cs | 13 ++++++++++++- embroideryReader/frmSettingsDialog.cs | 3 +++ 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/embroideryReader/frmMain.cs b/embroideryReader/frmMain.cs index 2a023df..1d3529f 100644 --- a/embroideryReader/frmMain.cs +++ b/embroideryReader/frmMain.cs @@ -479,7 +479,18 @@ namespace embroideryReader settings.windowHeight = this.Height; } } - settings.save(); + try + { + settings.save(); + } + catch (System.IO.IOException /*ioex*/) + { + // Ignore IO exceptions when saving settings file, to deal with crashes caused by + // closing many windows at once. + + // This isn't a great way to handle the problem, but it's better than getting + // the same automated error report over and over. + } } private string CurrentVersion() diff --git a/embroideryReader/frmSettingsDialog.cs b/embroideryReader/frmSettingsDialog.cs index bccc244..2351e06 100644 --- a/embroideryReader/frmSettingsDialog.cs +++ b/embroideryReader/frmSettingsDialog.cs @@ -135,6 +135,9 @@ namespace embroideryReader // Save translation setting settings.translation = availableTranslations[cmbLanguage.SelectedIndex].Item2; + + // Save settings file + settings.save(); } public Translation setTranslation