Changeset 780bfa for src/Actions/ActionHistory.cpp

Timestamp:

Aug 13, 2025, 9:46:30 PM (2 months ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Candidate_v1.7.0, stable

Children:

463bfb

Parents:

bcc29ca

git-author:

Frederik Heber <frederik.heber@…> (08/13/25 08:41:09)

git-committer:

Frederik Heber <frederik.heber@…> (08/13/25 21:46:30)

Message:

FIX: FormatParserStorage did not add parser instance on SetOutput.

• using the InputAction on an empty file would not write that file using the desired parser (judged from suffix) on exit because the respective was set as desired output but not instantiated (ParserPresent).

File:

• src/Actions/ActionHistory.cpp (modified) (1 diff)

src/Actions/ActionHistory.cpp

@@ -100 +100 @@
 void ActionHistory::setMark() {
   HistoryElement * const mark = &(history.back());
-  ASSERT( mark != NULL,
-      "ActionHistory::setMark() - must not push NULL as element");
+  if (mark == NULL) {
+    ELOG(2, "setMark - must not push NULL as element");
+    return;
+  }
   marked.push(mark);
 }
 
 void ActionHistory::unsetMark() {
-  marked.pop();
+  if (!marked.empty()) {
+    marked.pop();
+  } else {
+        ELOG(2, "unsetMark - stack is already empty.");
+  }
 }
 
 void ActionHistory::undoTillMark() {
   if (marked.empty()) {
-    undoLast();
+        ELOG(2, "There is no undo mark, not undoing anything.");
   } else {
     const HistoryElement * const mark = marked.top();