Changeset cadaa0

Timestamp:

May 5, 2016, 9:43:09 AM (9 years ago)

Author:

Frederik Heber <heber@…>

Parents:

996ef1

git-author:

Frederik Heber <heber@…> (09/23/14 16:14:18)

git-committer:

Frederik Heber <heber@…> (05/05/16 09:43:09)

Message:

FIX: Setting ActionQueue::_lastchangedaction without heeding whether Action failed is bad.

• causes segfault when notification informs about change and _lastchangeaction is still NULL (as the very first Action failed).
• we now only notify when action succeeded.

File:

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

src/Actions/ActionQueue.cpp

@@ -215 +215 @@
         mtx_queue.unlock();
       }
+      if (lastActionOk) {
+        OBSERVE;
+        NOTIFY(ActionQueued);
+        _lastchangedaction = actionqueue[CurrentAction];
+      }
       // access actionqueue, hence using mutex
       mtx_queue.lock();