Replace basic for loops with range-based loops or algorithms

[builder.git] / source / androidapplicationcomponent.cpp

diff --git a/source/androidapplicationcomponent.cpp b/source/androidapplicationcomponent.cpp
index 99f14408d8e05ebe6d6c50e9348c3a956433a5c8..26277c8c02450fd8c75c065cc4f1e0a3ec090e0e 100644 (file)
--- a/source/androidapplicationcomponent.cpp
+++ b/source/androidapplicationcomponent.cpp
@@ -21,27 +21,25 @@ void AndroidApplicationComponent::create_targets() const
        Builder &builder = package.get_builder();
        BuildGraph &build_graph = builder.get_build_graph();
 
-       const BuildGraph::TargetMap &targets = build_graph.get_targets();
        list<Target *> contents;
-       for(BuildGraph::TargetMap::const_iterator i=targets.begin(); i!=targets.end(); ++i)
-               if(i->second->get_package()==&package)
-                       if(InstalledFile *inst = dynamic_cast<InstalledFile *>(i->second))
+       for(const auto &kvp: build_graph.get_targets())
+               if(kvp.second->get_package()==&package)
+                       if(InstalledFile *inst = dynamic_cast<InstalledFile *>(kvp.second))
                                contents.push_back(inst->get_real_target());
 
        AndroidManifestFile *manifest = new AndroidManifestFile(builder, *this);
        manifest->set_orientation(orientation);
-       for(set<string>::const_iterator i=permissions.begin(); i!=permissions.end(); ++i)
-               manifest->add_permission(*i);
+       for(const string &p: permissions)
+               manifest->add_permission(p);
 
        list<Target *> resource_sources;
        resource_sources.push_back(manifest);
 
        const Toolchain &toolchain = builder.get_toolchain();
        Tool &copy = toolchain.get_tool("CP");
-       SourceList source_filenames = collect_source_files();
-       for(SourceList::const_iterator i=source_filenames.begin(); i!=source_filenames.end(); ++i)
+       for(const FS::Path &s: collect_source_files())
        {
-               Target *tgt = new AndroidResourceFile(builder, *this, *i);
+               Target *tgt = new AndroidResourceFile(builder, *this, s);
                resource_sources.push_back(copy.create_target(*tgt, "//"));
        }
 
@@ -63,16 +61,16 @@ void AndroidApplicationComponent::create_targets() const
                lib_dir += arch.get_type();
 
        string assets_dir = "//"+name+"/assets";
-       for(list<Target *>::const_iterator i=contents.begin(); i!=contents.end(); ++i)
+       for(Target *t: contents)
        {
                Target *staged = 0;
-               if(SharedLibrary *shlib = dynamic_cast<SharedLibrary *>(*i))
+               if(SharedLibrary *shlib = dynamic_cast<SharedLibrary *>(t))
                {
                        manifest->set_native_library(shlib);
-                       staged = copy.create_target(**i, lib_dir);
+                       staged = copy.create_target(*t, lib_dir);
                }
                else
-                       staged = copy.create_target(**i, assets_dir);
+                       staged = copy.create_target(*t, assets_dir);
                apk_sources.push_back(staged);
        }