Let the verbosity options actually do something

Author: msuchane

src/cmd_line.rs

@@ -5,8 +5,6 @@ use clap::{
     ArgMatches,
 };
 
-use log::info;
-
 /// Define the command-line arguments and return them as the `clap::ArgMatches` struct.
 pub fn get_args() -> ArgMatches<'static> {
     // Define command-line options
@@ -120,9 +118,5 @@ pub fn get_args() -> ArgMatches<'static> {
         )
         .get_matches();
 
-    if matches.is_present("detect-directory") {
-        info!("The `--detect-directory` (`-D`) option is now enabled by default.");
-    }
-
     matches
 }

src/logging.rs

@@ -2,16 +2,26 @@ use simplelog::{ColorChoice, ConfigBuilder, LevelFilter, TerminalMode, TermLogge
 
 /// This function initializes the `simplelog` logging system, which plugs into the `log`
 /// infrastructure. The function returns nothing. It only affects the global state when it runs.
-pub fn initialize_logger() {
+pub fn initialize_logger(verbose: bool, quiet: bool) {
+    // Set the verbosity level based on the command-line options.
+    // Our `clap` configuration ensures that `verbose` and `quiet` can never be both true.
+    let verbosity = if verbose {
+        LevelFilter::Debug
+    } else if quiet {
+        LevelFilter::Warn
+    } else {
+        // The default verbosity level is Info because newdoc displays the include statement
+        // at that level.
+        LevelFilter::Info
+    };
+
     let config = ConfigBuilder::new()
         // Display a time stamp only for the debug and more verbose levels.
         .set_time_level(LevelFilter::Debug)
         .build();
 
     TermLogger::init(
-        // The default verbosity level is Info because newdoc displays the include statement
-        // at that level.
-        LevelFilter::Info,
+        verbosity,
         config,
         // Mixed mode prints errors to stderr and info to stdout. Not sure about the other levels.
         TerminalMode::Mixed,

src/main.rs

@@ -1,3 +1,5 @@
+use log::info;
+
 mod cmd_line;
 mod logging;
 mod module;
@@ -17,9 +19,17 @@ pub struct Options {
 }
 
 fn main() {
-    logging::initialize_logger();
-
+    // Parse the command-line options
     let cmdline_args = cmd_line::get_args();
+    // Determine the configured verbosity level
+    let verbose = cmdline_args.is_present("verbose");
+    let quiet = cmdline_args.is_present("quiet");
+    // Initialize the logging system based on the set verbosity
+    logging::initialize_logger(verbose, quiet);
+
+    if cmdline_args.is_present("detect-directory") {
+        info!("The `--detect-directory` (`-D`) option is now enabled by default.");
+    }
 
     // Set current options based on the command-line options
     let options = Options {