message: Cleanup Packages to packages
diff: @@ -110,22 +110,22 @@ impl Packages {
  }
  pub fn to_package_id_specs(&self, ws: &Workspace<'_>) -> CargoResult<Vec<PackageIdSpec>> {
  - let specs = match *self {
  + let specs = match self {
  Packages::All => ws
  .members()
  .map(Package::package_id)
  .map(PackageIdSpec::from_package_id)
  .collect(),
  - Packages::OptOut(ref opt_out) => ws
  + Packages::OptOut(opt_out) => ws
  .members()
  + .filter(|pkg| !opt_out.iter().any(|name| pkg.name().as_str() == name))
  .map(Package::package_id)
  .map(PackageIdSpec::from_package_id)
  - .filter(|p| opt_out.iter().position(|x| *x == p.name()).is_none())
  .collect(),
  - Packages::Packages(ref packages) if packages.is_empty() => {
  + Packages::Packages(packages) if packages.is_empty() => {
  vec![PackageIdSpec::from_package_id(ws.current()?.package_id())]
  }
  - Packages::Packages(ref packages) => packages
  + Packages::Packages(packages) => packages
  .iter()
  .map(|p| PackageIdSpec::parse(p))
  .collect::<CargoResult<Vec<_>>>()?,
  @@ -152,11 +152,11 @@ impl Packages {
  let packages: Vec<_> = match self {
  Packages::Default => ws.default_members().collect(),
  Packages::All => ws.members().collect(),
  - Packages::OptOut(ref opt_out) => ws
  + Packages::OptOut(opt_out) => ws
  .members()
  .filter(|pkg| !opt_out.iter().any(|name| pkg.name().as_str() == name))
  .collect(),
  - Packages::Packages(ref pkgs) => pkgs
  + Packages::Packages(packages) => packages
  .iter()
  .map(|name| {
  ws.members()
  