diff options
author | Melody Horn <melody@boringcactus.com> | 2021-03-27 17:10:04 -0600 |
---|---|---|
committer | Melody Horn <melody@boringcactus.com> | 2021-03-27 17:10:04 -0600 |
commit | 574d29445199e12e4d1474b3cafe0fa5605a84cf (patch) | |
tree | 325d04d750594572346474856eccc0d32934e781 /src/args.rs | |
parent | 3cb8fdd3e0bad44bfe65a467230b6e828be47af8 (diff) | |
download | makers-574d29445199e12e4d1474b3cafe0fa5605a84cf.tar.gz makers-574d29445199e12e4d1474b3cafe0fa5605a84cf.zip |
appease clippy in makers::args
Diffstat (limited to 'src/args.rs')
-rw-r--r-- | src/args.rs | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/args.rs b/src/args.rs index ae58ce2..f96cba4 100644 --- a/src/args.rs +++ b/src/args.rs @@ -7,6 +7,7 @@ use structopt::StructOpt; #[derive(StructOpt, Debug, PartialEq, Eq, Clone)] #[structopt(author, about)] +#[allow(clippy::struct_excessive_bools)] pub(crate) struct Args { /// Cause environment variables, including those with null values, to override macro /// assignments within makefiles. @@ -127,7 +128,7 @@ impl Args { fn from_given_args_and_given_env( mut args: impl Iterator<Item = OsString>, env_makeflags: String, - ) -> Args { + ) -> Self { // POSIX spec says "Any options specified in the MAKEFLAGS environment variable // shall be evaluated before any options specified on the make utility command // line." @@ -149,16 +150,16 @@ impl Args { let env_makeflags = env_makeflags.split_whitespace().map(OsString::from); // per the structopt docs, the first argument will be used as the binary name, // so we need to make sure it goes in before MAKEFLAGS - let arg0 = args.next().unwrap_or_else(|| env!("CARGO_PKG_NAME").into()); + let arg_0 = args.next().unwrap_or_else(|| env!("CARGO_PKG_NAME").into()); - let args = iter::once(arg0) + let args = iter::once(arg_0) .chain(env_makeflags.into_iter()) .chain(args); - Args::from_iter(args) + Self::from_iter(args) } - pub(crate) fn from_env_and_args() -> Args { + pub(crate) fn from_env_and_args() -> Self { let env_makeflags = env::var("MAKEFLAGS").unwrap_or_default(); let args = env::args_os(); Self::from_given_args_and_given_env(args, env_makeflags) |