Changeset 14002 for trunk

Timestamp:

05/31/12 16:45:52 (12 years ago)

Author:

tbretz

Message:

Replaced some 'const char*' with a const-refernce to a std::string

Location:

trunk/FACT++/src

Files:

• Event.h (modified) (2 diffs)
• StateMachine.cc (modified) (1 diff)
• StateMachine.h (modified) (1 diff)
• StateMachineDim.cc (modified) (1 diff)
• StateMachineDim.h (modified) (1 diff)
• StateMachineImp.cc (modified) (2 diffs)
• StateMachineImp.h (modified) (2 diffs)

trunk/FACT++/src/Event.h

@@ -18 +18 @@
 public:
     /// Constructs an event as a combination of an EventImp and a DimCommand
-    Event(const std::string &name, const char *fmt="");
+    Event(const std::string &name, const std::string &fmt="");
     /// Copy constructor
     Event(const EventImp &imp);
-    Event(const EventImp &imp, const char *name, size_t siz);
+    Event(const EventImp &imp, const char *ptr, size_t siz);
 
     void SetDescription(const std::string &str) { fDescription=str; }
@@ -53 +53 @@
     void SetText(const char *txt) { SetData(txt, strlen(txt)+1); }
     void SetString(const std::string &str) { SetData(str.c_str(), str.length()+1); }
-
 };
 

trunk/FACT++/src/StateMachine.cc

@@ -37 +37 @@
 //! For parameter description see StateMachineImp.
 //!
-EventImp *StateMachine::CreateEvent(const char *name, const char *fmt)
+EventImp *StateMachine::CreateEvent(const string &name, const string &fmt)
 {
-    return new Event((GetName()+'/'+name).c_str(), fmt);
+    return new Event(GetName()+'/'+name, fmt);
 }
 

trunk/FACT++/src/StateMachine.h

@@ -7 +7 @@
 {
 protected:
-    EventImp *CreateEvent(const char *name, const char *fmt);
+    EventImp *CreateEvent(const std::string &name, const std::string &fmt);
 
 public:

trunk/FACT++/src/StateMachineDim.cc

@@ -82 +82 @@
 //! For parameter description see StateMachineImp.
 //!
-EventImp *StateMachineDim::CreateEvent(const char *name, const char *fmt)
+EventImp *StateMachineDim::CreateEvent(const string &name, const string &fmt)
 {
     return new EventDim(GetName()+"/"+name, fmt, this);
 }
 
-EventImp *StateMachineDim::CreateService(const char *name)
+EventImp *StateMachineDim::CreateService(const string &name)
 {
     return new ServiceDim(name, this);

trunk/FACT++/src/StateMachineDim.h

@@ -68 +68 @@
     void infoHandler();          /// Overwritten DimInfo::infoHandler
 
-    EventImp *CreateEvent(const char *name, const char *fmt);
-    EventImp *CreateService(const char *name);
+    EventImp *CreateEvent(const std::string &name, const std::string &fmt);
+    EventImp *CreateService(const std::string &name);
 
 protected:

trunk/FACT++/src/StateMachineImp.cc

@@ -604 +604 @@
 }
 
-EventImp *StateMachineImp::CreateService(const char *)
+EventImp *StateMachineImp::CreateService(const string &)
 {
     return new EventImp();
@@ -611 +611 @@
 // --------------------------------------------------------------------------
 //
-EventImp &StateMachineImp::Subscribe(const char *name)
+EventImp &StateMachineImp::Subscribe(const string &name)
 {
     EventImp *evt = CreateService(name);

trunk/FACT++/src/StateMachineImp.h

@@ -54 +54 @@
     bool HandleNewState(int newstate, const EventImp *evt, const char *txt);
 
-    virtual EventImp *CreateEvent(const char *name, const char *fmt) = 0;
-    virtual EventImp *CreateService(const char *);
+    virtual EventImp *CreateEvent(const std::string &name, const std::string &fmt) = 0;
+    virtual EventImp *CreateService(const std::string &);
 
     /// Is called continously to execute actions in the current state
@@ -100 +100 @@
     const std::string &GetName() const { return fName; }
 
-    EventImp &Subscribe(const char *name);
+    EventImp &Subscribe(const std::string &name);
 
     /// return the current state of the machine