message: [commands_remove.cpp] Naming scheme and missing const
diff: @@ -18,7 +18,7 @@ namespace vcpkg::Commands::Remove
  {
  auto& fs = paths.get_filesystem();
  auto spghs = status_db->find_all(spec.name(), spec.triplet());
  - auto core_pkg = **status_db->find(spec.name(), spec.triplet(), Strings::EMPTY);
  + const auto core_pkg = **status_db->find(spec.name(), spec.triplet(), Strings::EMPTY);
  for (auto&& spgh : spghs)
  {
  @@ -31,7 +31,7 @@ namespace vcpkg::Commands::Remove
  auto maybe_lines = fs.read_lines(paths.listfile_path(core_pkg.package));
  - if (auto lines = maybe_lines.get())
  + if (const auto lines = maybe_lines.get())
  {
  std::vector<fs::path> dirs_touched;
  for (auto&& suffix : *lines)
  @@ -42,7 +42,7 @@ namespace vcpkg::Commands::Remove
  auto target = paths.installed / suffix;
  - auto status = fs.status(target, ec);
  + const auto status = fs.status(target, ec);
  if (ec)
  {
  System::println(System::Color::error, "failed: %s", ec.message());
  @@ -72,7 +72,7 @@ namespace vcpkg::Commands::Remove
  }
  auto b = dirs_touched.rbegin();
  - auto e = dirs_touched.rend();
  + const auto e = dirs_touched.rend();
  for (; b != e; ++b)
  {
  if (fs.is_empty(*b))
  @@ -100,11 +100,11 @@ namespace vcpkg::Commands::Remove
  static void print_plan(const std::map<RemovePlanType, std::vector<const RemovePlanAction*>>& group_by_plan_type)
  {
  - static constexpr std::array<RemovePlanType, 2> order = {RemovePlanType::NOT_INSTALLED, RemovePlanType::REMOVE};
  + static constexpr std::array<RemovePlanType, 2> ORDER = {RemovePlanType::NOT_INSTALLED, RemovePlanType::REMOVE};
  - for (const RemovePlanType plan_type : order)
  + for (const RemovePlanType plan_type : ORDER)
  {
  - auto it = group_by_plan_type.find(plan_type);
  + const auto it = group_by_plan_type.find(plan_type);
  if (it == group_by_plan_type.cend())
  {
  continue;
  @@ -206,8 +206,8 @@ namespace vcpkg::Commands::Remove
  Checks::exit_fail(VCPKG_LINE_INFO);
  }
  const Purge purge = to_purge(purge_was_passed || !no_purge_was_passed);
  - const bool isRecursive = options.find(OPTION_RECURSE) != options.cend();
  - const bool dryRun = options.find(OPTION_DRY_RUN) != options.cend();
  + const bool is_recursive = options.find(OPTION_RECURSE) != options.cend();
  + const bool dry_run = options.find(OPTION_DRY_RUN) != options.cend();
  const std::vector<RemovePlanAction> remove_plan = Dependencies::create_remove_plan(specs, status_db);
  Checks::check_exit(VCPKG_LINE_INFO, !remove_plan.empty(), "Remove plan cannot be empty");
  @@ -226,7 +226,7 @@ namespace vcpkg::Commands::Remove
  System::println(System::Color::warning,
  "Additional packages (*) need to be removed to complete this operation.");
  - if (!isRecursive)
  + if (!is_recursive)
  {
  System::println(System::Color::warning,
  "If you are sure you want to remove them, run the command with the --recurse option");
  @@ -234,7 +234,7 @@ namespace vcpkg::Commands::Remove
  }
  }
  - if (dryRun)
  + if (dry_run)
  {
  Checks::exit_success(VCPKG_LINE_INFO);
  }
  