Changeset 10873 for trunk/FACT++/src/StateMachineImp.cc

Timestamp:

05/27/11 16:30:07 (13 years ago)

Author:

tbretz

Message:

Replaced a self-done loop by std::find; changed some comments.

File:

• trunk/FACT++/src/StateMachineImp.cc (modified) (2 diffs)

trunk/FACT++/src/StateMachineImp.cc

@@ -316 +316 @@
         // FIXME: Is this thread safe? (Yes, because the data is copied)
         // But two handlers could be called at the same time. Do we
-        // need to lock the handlers?
+        // need to lock the handlers? (Dim + console)
+        // FIXME: Is copying of the data necessary?
         const Event event(evt, ptr, siz);
         HandleEvent(event);
@@ -456 +457 @@
 {
     // Find the event from the list of commands and queue it
-    for (vector<EventImp*>::const_iterator c=fListOfEvents.begin(); c!=fListOfEvents.end(); c++)
-        if (*c == cmd)
-            return true;
-
-    return false;
+    return find(fListOfEvents.begin(), fListOfEvents.end(), cmd)!=fListOfEvents.end();
 }