Merge pull request #35 from iancleary/feat/better-error-messaging

Updated error message and color

Author: iancleary

Cargo.lock

@@ -7,7 +7,7 @@ homepage = "https://github.com/iancleary/touchstone"
 license = "MIT"
 name = "touchstone"
 repository = "https://github.com/iancleary/touchstone"
-version = "0.10.1"
+version = "0.10.2"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 

Cargo.toml

@@ -128,19 +128,24 @@ impl Config {
     }
 }
 
-fn print_version() {
+pub fn print_version() {
     println!("touchstone {}", env!("CARGO_PKG_VERSION"));
 }
 
-fn print_help() {
+pub fn print_error(error: &str) {
+    const RED: &str = "\x1b[31m";
+    const RESET: &str = "\x1b[0m";
+    println!("{}Problem parsing arguments: {error}{}", RED, RESET);
+}
+
+pub fn print_help() {
     // ANSI color codes
     const BOLD: &str = "\x1b[1m";
     const CYAN: &str = "\x1b[36m";
     const GREEN: &str = "\x1b[32m";
     const YELLOW: &str = "\x1b[33m";
     const RESET: &str = "\x1b[0m";
 
-    println!();
     println!(
         "📡 Touchstone (s2p, etc.) file parser, plotter, and more - https://github.com/iancleary/touchstone{}",
         RESET

src/cli.rs

@@ -7,7 +7,12 @@ fn main() {
     let args: Vec<String> = env::args().collect();
 
     let _ = cli::Config::run(&args).unwrap_or_else(|err| {
-        println!("Problem parsing arguments: {err}");
+        println!();
+        cli::print_error(err); //print at the top, but might be lost or hard to read
+        println!();
+        cli::print_help();
+        println!();
+        cli::print_error(err); // print error again, for human factors
         process::exit(1);
     });
 }