# HG changeset patch # User Santeri Piippo # Date 1366638181 -10800 # Node ID 9cf313447c8f2cfe4965b3a066330ff0f730728c # Parent 73a7edf82ca987942164d6319367d7a636e8fd45 A saving failure is a critical error, not a warning and should be drawn as such diff -r 73a7edf82ca9 -r 9cf313447c8f file.cpp --- a/file.cpp Mon Apr 22 16:30:33 2013 +0300 +++ b/file.cpp Mon Apr 22 16:43:01 2013 +0300 @@ -182,8 +182,12 @@ if (!save ()) { str errormsg = format ("Failed to save %s: %s\nDo you still want to close?", zFileName.chars (), strerror (lastError)); - if (!confirm ("Save Failure", errormsg)) + + if (QMessageBox::critical (g_ForgeWindow, "Save Failure", errormsg, + (QMessageBox::Yes | QMessageBox::No), QMessageBox::No) == QMessageBox::No) + { return false; + } } break; diff -r 73a7edf82ca9 -r 9cf313447c8f gui_actions.cpp --- a/gui_actions.cpp Mon Apr 22 16:30:33 2013 +0300 +++ b/gui_actions.cpp Mon Apr 22 16:43:01 2013 +0300 @@ -72,7 +72,7 @@ setlocale (LC_ALL, "C"); // Tell the user the save failed, and give the option for saving as with it. - QMessageBox dlg (QMessageBox::Warning, "Save Failure", + QMessageBox dlg (QMessageBox::Critical, "Save Failure", format ("Failed to save to %s\nReason: %s", path.chars(), strerror (g_CurrentFile->lastError)), QMessageBox::Close, g_ForgeWindow);