X-Git-Url: https://ruin.nu/git/?a=blobdiff_plain;f=planner.cpp;h=6d70ae40df2114d32e6cf4f3ab611155e96b6aa6;hb=7b9821d42d980e5788bbc65563973eddd6e4c40d;hp=a15028e29a3175e8fa316e2e6e6a37e2dc41194f;hpb=09f6dfa65b66103c27dc831271d961de9ab0109e;p=popboot.git

diff --git a/planner.cpp b/planner.cpp
index a15028e..6d70ae4 100644
--- a/planner.cpp
+++ b/planner.cpp
@@ -11,8 +11,8 @@ Planner::Planner(std::vector<Action> actions, literals init, literals goal){
 	Node* finish = new Node(Action("finish",goal,literals()));
 
 	for(vector<Action>::iterator action = actions.begin(); action != actions.end(); ++action){
-		literals effects = action->effects();
-		for (literals::iterator effect = effects.begin(); effect != effects.end(); ++effect){
+		const literals& effects = action->effects();
+		for (literals::const_iterator effect = effects.begin(); effect != effects.end(); ++effect){
 			cerr << "Adding effect: " << *effect << endl;
 			_actions[*effect] = *action;
 		}
@@ -20,15 +20,24 @@ Planner::Planner(std::vector<Action> actions, literals init, literals goal){
 	makePlan(finish);
 }
 
+Planner::~Planner(){
+	for (hash_map<string,Node*>::iterator node = _addedNodes.begin(); node != _addedNodes.end(); ++node){
+		cerr << "Deleting node " << node->second->action().executable() << endl;
+		delete node->second;
+	}
+}
+
 
 void Planner::makePlan(Node* node){
-	literals preconds = node->action().preconditions();
+	cerr << "Fetching preconditions for action: " << node->action().executable() << ".. ";
+	const literals& preconds = node->action().preconditions();
+	cerr << "done" << endl;
 
 	if (preconds.size() == 0){
 		cerr << "Found no preconds" << endl;
 		_start->addChild(node);
 	}else{
-		for (literals::iterator precond = preconds.begin(); precond != preconds.end(); ++precond){
+		for (literals::const_iterator precond = preconds.begin(); precond != preconds.end(); ++precond){
 			cerr << "Looking for: " << *precond << endl;
 			hash_map<string,Node*>::iterator addedNode = _addedNodes.find(*precond);
 			if(addedNode != _addedNodes.end()){
@@ -51,9 +60,9 @@ void Planner::makePlan(Node* node){
 }
 
 void Planner::addNode(Node* node){
-	literals effects = node->action().effects();
+	const literals& effects = node->action().effects();
 
-	for (literals::iterator effect = effects.begin(); effect != effects.end(); ++effect){
+	for (literals::const_iterator effect = effects.begin(); effect != effects.end(); ++effect){
 		cout << "Adding node for effect: " << *effect << endl;
 		_addedNodes[*effect] = node;
 	}