Merge bb3dd98 into 8cae0b0

Cargo.toml

@@ -30,7 +30,7 @@ semver = "0.1"
 toml = "0.1"
 curl = "0.2.11"
 quick-error = "0.1.3"
-clippy = {version = "0.0.19", optional = true}
+clippy = {version = "0.0.21", optional = true}
 
 [dev-dependencies]
 assert_cli = "0.1.0"

src/bin/add/args.rs

@@ -56,7 +56,7 @@ impl Args {
             parse_path(path.clone())
         } else {
             get_latest_version(&self.arg_crate)
-                .map(|v| toml::Value::String(v))
+                .map(toml::Value::String)
                 .map_err(From::from)
         };
 

src/bin/list/tree.rs

@@ -91,7 +91,7 @@ fn get_packages(lock_file: &toml::Table) -> Result<Packages, Box<Error>> {
 }
 
 fn list_deps_helper(pkgs: &Packages,
-                    deps: &Dependencies,
+                    deps: &[Dependency],
                     levels: &mut Vec<bool>)
                     -> Result<String, Box<Error>> {
     let mut output = String::new();
@@ -126,7 +126,7 @@ fn list_deps_helper(pkgs: &Packages,
     Ok(output)
 }
 
-fn list_deps(pkgs: &Packages, deps: &Dependencies) -> Result<String, Box<Error>> {
+fn list_deps(pkgs: &Packages, deps: &[Dependency]) -> Result<String, Box<Error>> {
     list_deps_helper(pkgs, deps, &mut vec![])
 }
 

src/manifest.rs

@@ -180,6 +180,7 @@ impl Manifest {
     ///     assert!(manifest.remove_from_table("dependencies", &dep.0).is_err());
     /// # }
     /// ```
+    #[cfg_attr(feature = "dev", allow(toplevel_ref_arg))]
     pub fn remove_from_table(&mut self, table: &str, name: &str) -> Result<(), ManifestError> {
         let ref mut manifest = self.data;
         let entry = manifest.entry(String::from(table));