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

Timestamp:

04/07/11 08:42:34 (14 years ago)

Author:

tbretz

Message:

AddTransitions and AddConfiguration now return a reference; AddStateName now accepts also an explanatory string; Added GetStateDesc to retrieve this string

File:

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

trunk/FACT++/src/StateMachineImp.cc

@@ -472 +472 @@
 //!    received commands is properly extracted. No check is done.
 //
-EventImp *StateMachineImp::AddTransition(int targetstate, const char *name, const char *states, const char *fmt)
+EventImp &StateMachineImp::AddTransition(int targetstate, const char *name, const char *states, const char *fmt)
 {
     EventImp *evt = CreateEvent(targetstate, name, fmt);
@@ -483 +483 @@
     fListOfEvents.push_back(evt);
 
-    return evt;
+    return *evt;
 }
 
@@ -507 +507 @@
 //!    by this command.
 //
-EventImp *StateMachineImp::AddTransition(int targetstate, const char *name, int s1, int s2, int s3, int s4, int s5)
+EventImp &StateMachineImp::AddTransition(int targetstate, const char *name, int s1, int s2, int s3, int s4, int s5)
 {
     return AddTransition(targetstate, name, Form("%d %d %d %d %d", s1, s2, s3, s4, s5).c_str(), "");
@@ -539 +539 @@
 //!    by this command.
 //
-EventImp *StateMachineImp::AddTransition(int targetstate, const char *name, const char *fmt, int s1, int s2, int s3, int s4, int s5)
+EventImp &StateMachineImp::AddTransition(int targetstate, const char *name, const char *fmt, int s1, int s2, int s3, int s4, int s5)
 {
     return AddTransition(targetstate, name, Form("%d %d %d %d %d", s1, s2, s3, s4, s5).c_str(), fmt);
@@ -566 +566 @@
 //!    received commands is properly extracted. No check is done.
 //!
-EventImp *StateMachineImp::AddConfiguration(const char *name, const char *states, const char *fmt)
+EventImp &StateMachineImp::AddConfiguration(const char *name, const char *states, const char *fmt)
 {
     return AddTransition(-1, name, states, fmt);
@@ -587 +587 @@
 //!    by this command.
 //
-EventImp *StateMachineImp::AddConfiguration(const char *name, int s1, int s2, int s3, int s4, int s5)
+EventImp &StateMachineImp::AddConfiguration(const char *name, int s1, int s2, int s3, int s4, int s5)
 {
     return AddTransition(-1, name, s1, s2, s3, s4, s5);
@@ -614 +614 @@
 //!    by this command.
 //
-EventImp *StateMachineImp::AddConfiguration(const char *name, const char *fmt, int s1, int s2, int s3, int s4, int s5)
+EventImp &StateMachineImp::AddConfiguration(const char *name, const char *fmt, int s1, int s2, int s3, int s4, int s5)
 {
     return AddTransition(-1, name, fmt, s1, s2, s3, s4, s5);
@@ -634 +634 @@
 //! @todo  FIX THE DOCU
 //
-void StateMachineImp::AddStateName(const int state, const std::string &name)
-{
-    if (fStateNames[state].empty())
-        fStateNames[state] = name;
+void StateMachineImp::AddStateName(const int state, const std::string &name, const std::string &doc)
+{
+    if (fStateNames[state].first.empty())
+        fStateNames[state] = make_pair(name, doc);
 }
 
@@ -652 +652 @@
 const string StateMachineImp::GetStateName(int state) /*const*/
 {
-    const string &str = fStateNames[state];
+    const string &str = fStateNames[state].first;
     return str.empty() ? Form("%d", state) : str;
+}
+
+const string StateMachineImp::GetStateDoc(int state) /*const*/
+{
+    return fStateNames[state].second;
 }
 
@@ -673 +678 @@
 const string StateMachineImp::GetStateDescription(int state) /*const*/
 {
-    const string &str = fStateNames[state];
+    const string &str = GetStateName(state);
     return str.empty() ? Form("%d", state) : (str+Form("[%d]", state));
 }