X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;ds=sidebyside;f=source%2Ffs%2Fdir.cpp;h=098020328342db2edf93d4b2b425555405ab5f00;hb=44da9fc9afb6b7e49c1558c5572213a1e6f401e8;hp=be471628ceac105de600b341107ff2064a0bc090;hpb=ed4f5d6012b5f638ac21d96bc2ddf12b08a2d217;p=libs%2Fcore.git

diff --git a/source/fs/dir.cpp b/source/fs/dir.cpp
index be47162..0980203 100644
--- a/source/fs/dir.cpp
+++ b/source/fs/dir.cpp
@@ -38,16 +38,15 @@ const Path &get_bin_dir(const string &argv0)
 	{
 		Path exe;
 		if(argv0.find(DIRSEP)==string::npos)
-		{
-			const char *path = getenv("PATH");
-			vector<string> dirs = split(path, ITEMSEP);
-			for(vector<string>::const_iterator i=dirs.begin(); i!=dirs.end(); ++i)
-				if(exists(Path(*i)/argv0))
-				{
-					exe = realpath(Path(*i)/argv0);
-					break;
-				}
-		}
+			if(const char *path = getenv("PATH"))
+			{
+				for(const string &d: split(path, ITEMSEP))
+					if(exists(Path(d)/argv0))
+					{
+						exe = realpath(Path(d)/argv0);
+						break;
+					}
+			}
 
 		if(exe.empty())
 			exe = realpath(argv0);
@@ -70,9 +69,9 @@ not_a_directory::not_a_directory(const Path &p):
 void mkpath(const Path &path, int mode)
 {
 	Path p;
-	for(Path::Iterator i=path.begin(); i!=path.end(); ++i)
+	for(const string &c: path)
 	{
-		p /= *i;
+		p /= c;
 #ifdef _WIN32
 		if(p.size()==1 && p.is_absolute())
 			continue;
@@ -90,10 +89,9 @@ void mkpath(const Path &path, int mode)
 
 void rmpath(const Path &path)
 {
-	list<string> files = list_files(path);
-	for(list<string>::iterator i=files.begin(); i!=files.end(); ++i)
+	for(const string &f: list_files(path))
 	{
-		Path p = path / *i;
+		Path p = path/f;
 		if(is_dir(p))
 			rmpath(p);
 		else
@@ -103,16 +101,16 @@ void rmpath(const Path &path)
 	rmdir(path);
 }
 
-list<string> list_files(const Path &path)
+vector<string> list_files(const Path &path)
 {
 	return list_filtered(path, string());
 }
 
-list<string> list_filtered(const Path &path, const string &filter)
+vector<string> list_filtered(const Path &path, const string &filter)
 {
 	Regex r_filter(filter);
 
-	list<string> result;
+	vector<string> result;
 	DIR *dir = opendir(path.str().c_str());
 	if(!dir)
 		throw system_error("opendir");
@@ -130,11 +128,6 @@ list<string> list_filtered(const Path &path, const string &filter)
 	return result;
 }
 
-Path get_sys_conf_dir(const string &)
-{
-	return get_sys_conf_dir();
-}
-
 Path get_sys_conf_dir()
 {
 	const char *argv0 = Application::get_argv0();
@@ -154,11 +147,6 @@ Path get_sys_conf_dir()
 		return dir;
 }
 
-Path get_sys_data_dir(const string &, const string &)
-{
-	return get_sys_data_dir();
-}
-
 Path get_sys_data_dir()
 {
 	const char *argv0 = Application::get_argv0();
@@ -175,11 +163,6 @@ Path get_sys_data_dir()
 		return dir;
 }
 
-Path get_sys_lib_dir(const string &, const string &)
-{
-	return get_sys_lib_dir();
-}
-
 Path get_sys_lib_dir()
 {
 	const char *argv0 = Application::get_argv0();
@@ -194,11 +177,11 @@ Path get_sys_lib_dir()
 		return dir;
 }
 
-Path path_lookup(const string &name, const list<Path> &paths)
+Path path_lookup(const string &name, const vector<Path> &paths)
 {
-	for(list<Path>::const_iterator i=paths.begin(); i!=paths.end(); ++i)
+	for(const Path &p: paths)
 	{
-		Path full = *i/name;
+		Path full = p/name;
 		if(exists(full))
 			return realpath(full);
 	}
@@ -210,7 +193,7 @@ Path path_lookup(const string &name)
 {
 	const char *path = getenv("PATH");
 	vector<string> dirs = split(path, ITEMSEP);
-	return path_lookup(name, list<Path>(dirs.begin(), dirs.end()));
+	return path_lookup(name, vector<Path>(dirs.begin(), dirs.end()));
 }
 
 } // namespace FS