fix needless save confirmation 2

katahiromz · Mar 26, 2020 · 8d0bca1 · 8d0bca1
1 parent d4e2373
commit 8d0bca1
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/src/RisohEditor.cpp b/src/RisohEditor.cpp
@@ -3748,7 +3748,6 @@ void MMainWnd::OnCompile(HWND hwnd)
 
         // select the entry
         SelectTV(entry, FALSE);
-        DoSetFileModified(TRUE);
     }
     else
     {
@@ -5910,7 +5909,8 @@ BOOL MMainWnd::CompileStringTable(MStringA& strOutput, const MIdOrString& name,
     // recalculate the splitter
     PostMessageDx(WM_SIZE);
 
-    DoSetFileModified(TRUE);
+    if (bOK)
+        DoSetFileModified(TRUE);
     return bOK;
 }
 
@@ -6046,16 +6046,16 @@ BOOL MMainWnd::CompileMessageTable(MStringA& strOutput, const MIdOrString& name,
     // recalculate the splitter
     PostMessageDx(WM_SIZE);
 
-    DoSetFileModified(TRUE);
+    if (bOK)
+        DoSetFileModified(TRUE);
+
     return bOK;
 }
 
 // compile a resource item source
 BOOL MMainWnd::CompileParts(MStringA& strOutput, const MIdOrString& type, const MIdOrString& name,
                             WORD lang, const MStringW& strWide, BOOL bReopen)
 {
-    DoSetFileModified(TRUE);
-
     if (type == RT_STRING)
     {
         return CompileStringTable(strOutput, name, lang, strWide);
@@ -6130,6 +6130,8 @@ BOOL MMainWnd::CompileParts(MStringA& strOutput, const MIdOrString& type, const
         // select the added entry
         SelectTV(entry, FALSE);
 
+        DoSetFileModified(TRUE);
+
         return TRUE;    // success
     }
 
@@ -6271,6 +6273,8 @@ BOOL MMainWnd::CompileParts(MStringA& strOutput, const MIdOrString& type, const
         {
             PostMessageDx(MYWM_REOPENRAD);
         }
+
+        DoSetFileModified(TRUE);
     }
 
     // recalculate the splitter
@@ -11953,8 +11957,6 @@ void MMainWnd::DoAddRes(HWND hwnd, MAddResDlg& dialog)
             SelectTV(ET_MESSAGE, dialog.m_type, (WORD)0, BAD_LANG, FALSE);
         else
             SelectTV(ET_LANG, dialog, FALSE);
-
-        DoSetFileModified(TRUE);
    }
 }