diff options
| author |   Doug Goldstein <cardoe@cardoe.com> | 2016-10-19 08:27:23 -0500 |
|---|---|---|
| committer | Doug Goldstein <cardoe@cardoe.com> | 2016-10-19 08:27:23 -0500 |
| commit | ffd8dba0c75a07de37877e288a1e5edbb7f42f50 (patch) | |
| tree | 5c54241665890b0faf442e74b2fe56e8dc4fb539 /src | |
| parent | bump the version to 0.1.1 (diff) | |
| download | cargo-ebuild-ffd8dba0c75a07de37877e288a1e5edbb7f42f50.tar.gz cargo-ebuild-ffd8dba0c75a07de37877e288a1e5edbb7f42f50.tar.bz2 cargo-ebuild-ffd8dba0c75a07de37877e288a1e5edbb7f42f50.zip | |
cleanups from using clippy
Fixed some code smells with the help of clippy
Diffstat (limited to 'src')
| -rw-r--r-- | src/main.rs | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/main.rs b/src/main.rs index ed8fc66..7872b69 100644 --- a/src/main.rs +++ b/src/main.rs @@ -57,17 +57,17 @@ fn real_main(options: Options, config: &Config) -> CliResult<Option<()>> { // package description let desc = metadata.description .as_ref() - .map(|d| d.clone()) - .unwrap_or(String::from(package.name())); + .cloned() + .unwrap_or_else(|| String::from(package.name())); // package homepage let homepage = metadata.homepage .as_ref() - .map(|h| h.clone()) + .cloned() .unwrap_or(metadata.repository .as_ref() - .map(|h| h.clone()) - .unwrap_or(String::from(""))); + .cloned() + .unwrap_or_else(|| String::from(""))); // build up the ebuild path let ebuild_path = PathBuf::from(format!("{}-{}.ebuild", package.name(), package.version())); |