flip CanShowMessageLabel around into an error exit

tomysshadow · tomysshadow · commit b2c9f078353d · 2023-05-08T05:04:08.000-06:00
diff --git a/FlashpointSecurePlayer/FlashpointSecurePlayerGUI.cs b/FlashpointSecurePlayer/FlashpointSecurePlayerGUI.cs
@@ -109,13 +109,13 @@ private bool ShowOutput(string text) {
                 return false;
             }
 
-            if (CanShowMessageLabel(text)) {
-                messageLabel.Text = text;
-                return true;
+            if (!CanShowMessageLabel(text)) {
+                MessageBox.Show(text, Properties.Resources.FlashpointSecurePlayer, MessageBoxButtons.OK, MessageBoxIcon.None);
+                return false;
             }
 
-            MessageBox.Show(text, Properties.Resources.FlashpointSecurePlayer, MessageBoxButtons.OK, MessageBoxIcon.None);
-            return false;
+            messageLabel.Text = text;
+            return true;
         }
 
         private bool ShowError(string text) {
@@ -125,13 +125,13 @@ private bool ShowError(string text) {
                 return false;
             }
 
-            if (CanShowMessageLabel(text)) {
-                messageLabel.Text = text;
-                return true;
+            if (!CanShowMessageLabel(text)) {
+                MessageBox.Show(text, Properties.Resources.FlashpointSecurePlayer, MessageBoxButtons.OK, MessageBoxIcon.Error);
+                return false;
             }
 
-            MessageBox.Show(text, Properties.Resources.FlashpointSecurePlayer, MessageBoxButtons.OK, MessageBoxIcon.Error);
-            return false;
+            messageLabel.Text = text;
+            return true;
         }
 
         private void ShowNoGameSelected() {

Original file line number	Diff line number	Diff line change
`@@ -109,13 +109,13 @@ private bool ShowOutput(string text) {`
`109`	`109`	`return false;`
`110`	`110`	`}`
`111`	`111`
`112`		`- if (CanShowMessageLabel(text)) {`
`113`		`- messageLabel.Text = text;`
`114`		`- return true;`
	`112`	`+ if (!CanShowMessageLabel(text)) {`
	`113`	`+ MessageBox.Show(text, Properties.Resources.FlashpointSecurePlayer, MessageBoxButtons.OK, MessageBoxIcon.None);`
	`114`	`+ return false;`
`115`	`115`	`}`
`116`	`116`
`117`		`- MessageBox.Show(text, Properties.Resources.FlashpointSecurePlayer, MessageBoxButtons.OK, MessageBoxIcon.None);`
`118`		`- return false;`
	`117`	`+ messageLabel.Text = text;`
	`118`	`+ return true;`
`119`	`119`	`}`
`120`	`120`
`121`	`121`	`private bool ShowError(string text) {`
`@@ -125,13 +125,13 @@ private bool ShowError(string text) {`
`125`	`125`	`return false;`
`126`	`126`	`}`
`127`	`127`
`128`		`- if (CanShowMessageLabel(text)) {`
`129`		`- messageLabel.Text = text;`
`130`		`- return true;`
	`128`	`+ if (!CanShowMessageLabel(text)) {`
	`129`	`+ MessageBox.Show(text, Properties.Resources.FlashpointSecurePlayer, MessageBoxButtons.OK, MessageBoxIcon.Error);`
	`130`	`+ return false;`
`131`	`131`	`}`
`132`	`132`
`133`		`- MessageBox.Show(text, Properties.Resources.FlashpointSecurePlayer, MessageBoxButtons.OK, MessageBoxIcon.Error);`
`134`		`- return false;`
	`133`	`+ messageLabel.Text = text;`
	`134`	`+ return true;`
`135`	`135`	`}`
`136`	`136`
`137`	`137`	`private void ShowNoGameSelected() {`