about summary refs log tree commit diff
diff options
context:
space:
mode:
authorNathan Stocks <nathan.stocks@gmail.com>2017-03-30 22:20:51 -0600
committerNathan Stocks <nathan.stocks@gmail.com>2017-04-02 12:57:08 -0600
commit992a59efc3e3fd976728fc19a1a854afcbb06adf (patch)
tree9d4c94ee381d7c294bf95b56883f75b51db31719
parent584b40578d8ab999031da0855f319a94db06dc47 (diff)
downloadrust-992a59efc3e3fd976728fc19a1a854afcbb06adf.tar.gz
rust-992a59efc3e3fd976728fc19a1a854afcbb06adf.zip
Using an untyped, one-letter variable binding as an argument to a function and then not using it until over 100 lines later is just mean.
-rw-r--r--src/bootstrap/flags.rs4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/bootstrap/flags.rs b/src/bootstrap/flags.rs
index 1a260050a94..64b66d9c6bc 100644
--- a/src/bootstrap/flags.rs
+++ b/src/bootstrap/flags.rs
@@ -89,7 +89,7 @@ impl Flags {
         opts.optopt("j", "jobs", "number of jobs to run in parallel", "JOBS");
         opts.optflag("h", "help", "print this help message");
 
-        let usage = |n, opts: &Options| -> ! {
+        let usage = |exit_code, opts: &Options| -> ! {
             let subcommand_help = format!("\
 Usage: x.py <subcommand> [options] [<paths>...]
 
@@ -191,7 +191,7 @@ Arguments:
                 }
             }
 
-            process::exit(n);
+            process::exit(exit_code);
         };
         if args.len() == 0 {
             println!("a subcommand must be passed");