From 6350eddd4689568ff8dcfe431e1bf81579fb0556 Mon Sep 17 00:00:00 2001 From: Alexander Karatarakis Date: Tue, 18 Apr 2017 15:31:10 -0700 Subject: Find group_by_plan_type outside of print_plan() --- toolsrc/src/commands_install.cpp | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'toolsrc/src/commands_install.cpp') diff --git a/toolsrc/src/commands_install.cpp b/toolsrc/src/commands_install.cpp index cb9a8f61b..dc1a69167 100644 --- a/toolsrc/src/commands_install.cpp +++ b/toolsrc/src/commands_install.cpp @@ -169,13 +169,10 @@ namespace vcpkg::Commands::Install return SortedVector(std::move(installed_files)); } - static void print_plan(const std::vector& plan) + static void print_plan(const std::map>& group_by_plan_type) { static constexpr std::array order = { InstallPlanType::ALREADY_INSTALLED, InstallPlanType::BUILD_AND_INSTALL, InstallPlanType::INSTALL }; - std::map> group_by_plan_type; - Util::group_by(plan, &group_by_plan_type, [](const InstallPlanAction& p) { return p.plan_type; }); - for (const InstallPlanType plan_type : order) { auto it = group_by_plan_type.find(plan_type); @@ -286,7 +283,9 @@ namespace vcpkg::Commands::Install const std::string specs_string = Strings::join(",", install_plan, [](const InstallPlanAction& plan) { return plan.spec.to_string(); }); Metrics::track_property("installplan", specs_string); - print_plan(install_plan); + std::map> group_by_plan_type; + Util::group_by(install_plan, &group_by_plan_type, [](const InstallPlanAction& p) { return p.plan_type; }); + print_plan(group_by_plan_type); const bool has_non_user_requested_packages = Util::find_if(install_plan, [](const InstallPlanAction& package)-> bool { -- cgit v1.2.3