From 928f45adf24f5ba011d9a2c4448d131a0afa5955 Mon Sep 17 00:00:00 2001 From: Baptiste Wicht Date: Sun, 4 Sep 2016 11:09:13 +0200 Subject: [PATCH] Refactorings --- kernel/src/path.cpp | 22 +++++----------------- 1 file changed, 5 insertions(+), 17 deletions(-) diff --git a/kernel/src/path.cpp b/kernel/src/path.cpp index 13a2defc..7739b54c 100644 --- a/kernel/src/path.cpp +++ b/kernel/src/path.cpp @@ -21,9 +21,7 @@ path::path(const std::string& path){ auto parts = std::split(path, '/'); names.reserve(names.size() + parts.size()); - for(auto& part : parts){ - names.push_back(part); - } + std::copy(parts.begin(), parts.end(), std::back_inserter(names)); } path::path(const path& base_path, const std::string& p){ @@ -32,13 +30,8 @@ path::path(const path& base_path, const std::string& p){ auto parts = std::split(p, '/'); names.reserve(names.size() + base_path.size() + parts.size()); - for(auto& part : base_path){ - names.push_back(part); - } - - for(auto& part : parts){ - names.push_back(part); - } + std::copy(base_path.begin(), base_path.end(), std::back_inserter(names)); + std::copy(parts.begin(), parts.end(), std::back_inserter(names)); } path::path(const path& base_path, const path& p){ @@ -46,13 +39,8 @@ path::path(const path& base_path, const path& p){ names.reserve(names.size() + base_path.size() + p.size()); - for(auto& part : base_path){ - names.push_back(part); - } - - for(auto& part : p){ - names.push_back(part); - } + std::copy(base_path.begin(), base_path.end(), std::back_inserter(names)); + std::copy(p.begin(), p.end(), std::back_inserter(names)); } // TODO Ideally, the last / should not be used