X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fobjectfile.cpp;h=f78686e3bd999d2bff52ad5c27b53fe367c6fc88;hb=7c2db9e2b91da953701be233336c5bfa1f3c4af0;hp=8196ef7f2c39ac5d1c2da5d3f4c998debcde7484;hpb=32d6218afe58969f2b18dfa6e262bb9509829abe;p=builder.git

diff --git a/source/objectfile.cpp b/source/objectfile.cpp
index 8196ef7..f78686e 100644
--- a/source/objectfile.cpp
+++ b/source/objectfile.cpp
@@ -1,4 +1,4 @@
-#include <algorithm>
+#include <msp/core/algorithm.h>
 #include <msp/fs/utils.h>
 #include "builder.h"
 #include "component.h"
@@ -28,12 +28,12 @@ FS::Path ObjectFile::generate_target_path(const Component &comp, const FS::Path
 	else
 		rel_src = FS::relative(src, pkg.get_source_directory());
 	string fn;
-	for(FS::Path::Iterator i=rel_src.begin(); i!=rel_src.end(); ++i)
+	for(const string &c: rel_src)
 	{
 		if(!fn.empty())
 			fn += '_';
-		if(*i!=".")
-			fn += *i;
+		if(c!=".")
+			fn += c;
 	}
 	const Architecture &arch = comp.get_package().get_builder().get_current_arch();
 	return temp_dir/comp.get_name()/arch.create_filename<ObjectFile>(FS::basepart(fn));
@@ -52,10 +52,10 @@ void ObjectFile::collect_build_info(BuildInfo &binfo) const
 
 void ObjectFile::find_dependencies()
 {
-	for(Dependencies::iterator i=depends.begin(); i!=depends.end(); ++i)
+	for(Target *d: depends)
 	{
-		(*i)->prepare();
-		find_dependencies(dynamic_cast<FileTarget *>(*i));
+		d->prepare();
+		find_dependencies(dynamic_cast<FileTarget *>(d));
 	}
 }
 
@@ -76,9 +76,9 @@ void ObjectFile::find_dependencies(FileTarget *tgt)
 		/* The target has been displaced by installing it.  Displace any
 		dependencies that come from the same package as well. */
 		const SourcePackage *tpkg = rtgt->get_package();
-		for(Dependencies::const_iterator i=tdeps.begin(); i!=tdeps.end(); ++i)
+		for(Target *d: tdeps)
 		{
-			FileTarget *file = dynamic_cast<FileTarget *>(*i);
+			FileTarget *file = dynamic_cast<FileTarget *>(d);
 			if(file && file->get_package()==tpkg && FS::descendant_depth(file->get_path(), tpkg->get_source_directory())>=0)
 			{
 				const Component *tcomp = file->get_component();
@@ -88,27 +88,25 @@ void ObjectFile::find_dependencies(FileTarget *tgt)
 					deps_to_add.push_back(ddep);
 				else
 				{
-					const Component::OverlayList &overlays = tcomp->get_overlays();
 					string last_dir = FS::basename(FS::dirname(displaced));
-					for(Component::OverlayList::const_iterator j=overlays.begin(); j!=overlays.end(); ++j)
-						if(last_dir==*j)
-						{
-							displaced = displaced.subpath(0, displaced.size()-2)/FS::basename(file->get_path());
-							if((ddep = builder.get_vfs().get_target(displaced)))
-								deps_to_add.push_back(ddep);
-						}
+					if(any_equals(tcomp->get_overlays(), last_dir))
+					{
+						displaced = displaced.subpath(0, displaced.size()-2)/FS::basename(file->get_path());
+						if((ddep = builder.get_vfs().get_target(displaced)))
+							deps_to_add.push_back(ddep);
+					}
 				}
 			}
 			else
-				deps_to_add.push_back(*i);
+				deps_to_add.push_back(d);
 		}
 	}
 
-	for(Dependencies::const_iterator i=deps_to_add.begin(); i!=deps_to_add.end(); ++i)
-		if(find(depends.begin(), depends.end(), *i)==depends.end())
+	for(Target *d: deps_to_add)
+		if(find(depends.begin(), depends.end(), d)==depends.end())
 		{
-			add_dependency(**i);
-			if((*i)->get_real_target()->is_buildable())
-				(*i)->signal_modified.connect(sigc::mem_fun(this, static_cast<void (ObjectFile::*)()>(&ObjectFile::find_dependencies)));
+			add_dependency(*d);
+			if(d->get_real_target()->is_buildable())
+				d->signal_modified.connect(sigc::mem_fun(this, static_cast<void (ObjectFile::*)()>(&ObjectFile::find_dependencies)));
 		}
 }