X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fandroidassetpackagingtool.cpp;h=af8bc95ea422187112d5dce75927d8567803d309;hb=5e00719d0c63e306786ff36df61797cdbc86f3e9;hp=9e0fef75a990f8252d226ceda12dd396fe84fe60;hpb=266697cea92aed098fff31c1ddd4a4bfab3a683f;p=builder.git

diff --git a/source/androidassetpackagingtool.cpp b/source/androidassetpackagingtool.cpp
index 9e0fef7..af8bc95 100644
--- a/source/androidassetpackagingtool.cpp
+++ b/source/androidassetpackagingtool.cpp
@@ -1,3 +1,4 @@
+#include <msp/core/algorithm.h>
 #include <msp/fs/utils.h>
 #include "androidassetpackagingtool.h"
 #include "androidmanifestfile.h"
@@ -25,15 +26,16 @@ AndroidAssetPackagingTool::AndroidAssetPackagingTool(Builder &b, const AndroidSd
 		problems.push_back("Android platform not found");
 }
 
-Target *AndroidAssetPackagingTool::create_target(const list<Target *> &sources, const string &)
+Target *AndroidAssetPackagingTool::create_target(const vector<Target *> &sources, const string &)
 {
 	AndroidManifestFile *manifest = 0;
-	list<FileTarget *> resources;
-	for(list<Target *>::const_iterator i=sources.begin(); i!=sources.end(); ++i)
+	vector<FileTarget *> resources;
+	resources.reserve(sources.size());
+	for(Target *s: sources)
 	{
-		if(AndroidManifestFile *m = dynamic_cast<AndroidManifestFile *>(*i))
+		if(AndroidManifestFile *m = dynamic_cast<AndroidManifestFile *>(s))
 			manifest = m;
-		else if(FileTarget *f = dynamic_cast<FileTarget *>(*i))
+		else if(FileTarget *f = dynamic_cast<FileTarget *>(s))
 			resources.push_back(f);
 	}
 
@@ -61,12 +63,12 @@ Task *AndroidAssetPackagingTool::run(const Target &tgt) const
 	argv.push_back("-F");
 	argv.push_back(FS::relative(res.get_path(), work_dir).str());
 
-	const Target::Dependencies &depends = res.get_dependencies();
-	list<FS::Path> resource_dirs;
-	for(Target::Dependencies::const_iterator i=depends.begin(); i!=depends.end(); ++i)
+	vector<FS::Path> resource_dirs;
+	resource_dirs.reserve(res.get_dependencies().size());
+	for(Target *d: res.get_dependencies())
 	{
-		FileTarget *file = dynamic_cast<FileTarget *>(*i);
-		Target *real = (*i)->get_real_target();
+		FileTarget *file = dynamic_cast<FileTarget *>(d);
+		Target *real = d->get_real_target();
 
 		if(dynamic_cast<AndroidManifestFile *>(real))
 		{
@@ -76,17 +78,17 @@ Task *AndroidAssetPackagingTool::run(const Target &tgt) const
 		else if(real->get_package()==res.get_package())
 		{
 			const FS::Path &path = file->get_path();
-			resource_dirs.push_back(path.subpath(0, path.size()-2));
+			FS::Path res_dir = path.subpath(0, path.size()-2);
+			if(!any_equals(resource_dirs, res_dir))
+				resource_dirs.push_back(res_dir);
 		}
 	}
 
-	set<string> seen_dirs;
-	for(list<FS::Path>::const_iterator i=resource_dirs.begin(); i!=resource_dirs.end(); ++i)
-		if(seen_dirs.insert(i->str()).second)
-		{
-			argv.push_back("-S");
-			argv.push_back(FS::relative(*i, work_dir).str());
-		}
+	for(const FS::Path &d: resource_dirs)
+	{
+		argv.push_back("-S");
+		argv.push_back(FS::relative(d, work_dir).str());
+	}
 
 	return new ExternalTask(argv, work_dir);
 }