Use default member initializers and constructor delegation

[builder.git] / source / chainedtask.cpp

diff --git a/source/chainedtask.cpp b/source/chainedtask.cpp
index 066dc22c3be1f6aba48a7f68adfefa97dfb67287..4e0decb67a8f2b25aad18d92baa8f1357cd6ba0d 100644 (file)
--- a/source/chainedtask.cpp
+++ b/source/chainedtask.cpp
@@ -1,18 +1,18 @@
+#include <msp/strings/utils.h>
 #include "chainedtask.h"
 
 using namespace std;
+using namespace Msp;
 
-ChainedTask::ChainedTask(Task *t):
-       current(0),
-       final_status(RUNNING)
+ChainedTask::ChainedTask(Task *t)
 {
        add_task(t);
 }
 
 ChainedTask::~ChainedTask()
 {
-       for(vector<Task *>::iterator i=tasks.begin(); i!=tasks.end(); ++i)
-               delete *i;
+       for(Task *t: tasks)
+               delete t;
 }
 
 void ChainedTask::add_task(Task *t)
@@ -23,12 +23,8 @@ void ChainedTask::add_task(Task *t)
 string ChainedTask::get_command() const
 {
        string cmd;
-       for(vector<Task *>::const_iterator i=tasks.begin(); i!=tasks.end(); ++i)
-       {
-               if(i!=tasks.begin())
-                       cmd += '\n';
-               cmd += (*i)->get_command();
-       }
+       for(Task *t: tasks)
+               append(cmd, "\n", t->get_command());
        return cmd;
 }