Bump version to v1.2.0

Cargo.lock

@@ -4,4 +4,4 @@ version = 4
 
 [[package]]
 name = "dbus-prettifier"
-version = "0.1.0"
+version = "1.2.0"

Cargo.toml

@@ -1,6 +1,14 @@
 [package]
 name = "dbus-prettifier"
-version = "0.1.0"
+version = "1.2.0"
 edition = "2024"
+authors = [ "Nicolás A. Ortega Froysa <nicolas@ortegas.org>" ]
+description = "A tool to prettify the output of qdbus."
+readme = "./README.md"
+homepage = "https://code.ortegas.org/nortega/dbus-prettifier"
+repository = "https://code.ortegas.org/nortega/dbus-prettifier"
+license = "Zlib"
+keywords = [ "dbus", "qdbus", "prettifier" ]
+categories = [ "command-line-utilities" ]
 
 [dependencies]

src/main.rs

@@ -23,83 +23,68 @@
  */
 
 use std::env;
-use std::fs;
-use std::io;
-use std::io::Read;
-use std::boxed::Box;
 use std::process;
 
-// Read 1KiB of the file at a time.
-static BLOCK_SIZE:usize = 1024;
+use crate::worker::work;
 
-fn main() -> io::Result<()> {
+mod worker;
+
+fn print_version() {
+	println!("{} v{}", env!("CARGO_PKG_NAME"), env!("CARGO_PKG_VERSION"));
+}
+
+fn print_usage() {
+	println!("Usage: {} <in-file> [out-file]", env!("CARGO_PKG_NAME"));
+}
+
+fn print_help() {
+	print_version();
+	println!("Usage: \n\
+		\t{} <in-file> [out-file]\n\
+		\t{} -h | -v\n\
+		\n\
+		OPTIONS:\n\
+		\tin-file   Input file ('-' for stdin)\n\
+		\tout-file  Output file\n\
+		\t-h        Show this help information\n\
+		\t-v        Show version information",
+		env!("CARGO_PKG_NAME"), env!("CARGO_PKG_NAME"));
+}
+
+fn main() {
 	let args:Vec<String> = env::args().collect();
+	let mut in_path:Option<&str> = None;
+	let mut out_path:Option<&str> = None;
 
 	if args.len() < 2 || args.len() > 3 {
-		eprintln!("Usage: {} <in-file> [out-file]", args[0]);
+		eprintln!("Invalid number of arguments. Use -h for more information.");
+		print_usage();
 		process::exit(1);
 	}
 
-	let mut in_reader:Box<dyn io::Read>;
-	if args[1] == "-" {
-		in_reader = Box::new(io::stdin());
-	} else {
-		let in_file = fs::File::open(&args[1])?;
-		in_reader = Box::new(io::BufReader::new(in_file));
-	}
-
-	let mut out_writer:Box<dyn io::Write>;
-	if args.len() == 3 {
-		let out_file = fs::File::create(&args[2])?;
-		out_writer = Box::new(io::BufWriter::new(out_file));
-	} else {
-		out_writer = Box::new(io::stdout());
-	}
-
-	let mut read_buf:[u8; BLOCK_SIZE] = [0; BLOCK_SIZE];
-	let mut last_ch:char = '\0';
-	let mut tab_num = 0;
-
-	while in_reader.read(&mut read_buf)? > 0 {
-		for i in read_buf {
-			let ch = i as char;
-			match ch {
-				'[' | '{' => {
-					tab_num += 1;
-					out_writer.write(format!("{}\n", ch).as_bytes())?;
-					for _ in 0..tab_num {
-						out_writer.write(b"  ")?;
-					}
-					last_ch = ' ';
-				},
-				']' | '}' => {
-					tab_num -= 1;
-					out_writer.write(b"\n")?;
-					for _ in 0..tab_num {
-						out_writer.write(b"  ")?;
-					}
-					out_writer.write(format!("{}", ch).as_bytes())?;
-					last_ch = ch;
-				},
-				',' => {
-					out_writer.write(format!("{}\n", ch).as_bytes())?;
-					for _ in 0..tab_num {
-						out_writer.write(b"  ")?;
-					}
-					last_ch = ' ';
-				},
-				_ => {
-					let ch = ch;
-					if ch != ' ' || last_ch != ' ' {
-						out_writer.write(format!("{}", ch).as_bytes())?;
-						last_ch = ch;
-					}
-				},
-			}
+	for i in &args[1..args.len()] {
+		if i == "-h" {
+			print_help();
+			return;
+		} else if i == "-v" {
+			print_version();
+			return;
+		} else if i.starts_with("-") && i != "-" {
+			eprintln!("Invalid argument '{}'. Use -h for help information.", i);
+			process::exit(1);
 		}
 	}
 
-	out_writer.flush()?;
+	if args[1] != "-" {
+		in_path = Some(args[1].as_str());
+	}
+	if args.len() == 3 {
+		out_path = Some(args[2].as_str());
+	}
 
-	Ok(())
+	let res = work(in_path, out_path);
+	if let Err(err) = res {
+		eprintln!("{}", err);
+		process::exit(1);
+	}
 }

src/worker.rs

@@ -0,0 +1,102 @@
+/*
+ * Copyright (C) 2025 Nicolás Ortega Froysa <nicolas@ortegas.org> All rights reserved.
+ * Author: Nicolás Ortega Froysa <nicolas@ortegas.org>
+ *
+ * This software is provided 'as-is', without any express or implied
+ * warranty. In no event will the authors be held liable for any damages
+ * arising from the use of this software.
+ *
+ * Permission is granted to anyone to use this software for any purpose,
+ * including commercial applications, and to alter it and redistribute it
+ * freely, subject to the following restrictions:
+ *
+ * 1. The origin of this software must not be misrepresented; you must not
+ *    claim that you wrote the original software. If you use this software
+ *    in a product, an acknowledgment in the product documentation would be
+ *    appreciated but is not required.
+ *
+ * 2. Altered source versions must be plainly marked as such, and must not be
+ *    misrepresented as being the original software.
+ *
+ * 3. This notice may not be removed or altered from any source
+ *    distribution.
+ */
+
+use std::io;
+use std::fs;
+use std::io::Read;
+use std::boxed::Box;
+
+pub fn work(in_path:Option<&str>, out_path:Option<&str>) -> Result<(), String>  {
+	let in_reader:Box<dyn io::Read> = match in_path {
+		Some(path) => {
+			let in_file = fs::File::open(path);
+			if let Err(e) = in_file {
+				return Err(format!("{path}: {e}"));
+			}
+			Box::new(io::BufReader::new(in_file.unwrap()))
+		},
+		None => {
+			Box::new(io::stdin())
+		},
+	};
+
+	let mut out_writer:Box<dyn io::Write> = match out_path {
+		Some(path) => {
+			let out_file = fs::File::create(path);
+			if let Err(e) = out_file {
+				return Err(format!("{path}: {e}"));
+			}
+			Box::new(io::BufWriter::new(out_file.unwrap()))
+		},
+		None => {
+			Box::new(io::stdout())
+		},
+	};
+
+	let mut last_ch = '\0';
+	let mut tab_num = 0;
+
+	for i in in_reader.bytes() {
+		let ch = i.unwrap() as char;
+		match ch {
+			'[' | '{' => {
+				tab_num += 1;
+				out_writer.write(format!("{ch}\n").as_bytes()).unwrap();
+				for _ in 0..tab_num {
+					out_writer.write(b"  ").unwrap();
+				}
+				last_ch = ' ';
+			},
+			']' | '}' => {
+				tab_num -= 1;
+				out_writer.write(b"\n").unwrap();
+				for _ in 0..tab_num {
+					out_writer.write(b"  ").unwrap();
+				}
+				out_writer.write(format!("{ch}").as_bytes()).unwrap();
+				last_ch = ch;
+			},
+			',' => {
+				out_writer.write(format!("{ch}").as_bytes()).unwrap();
+				if last_ch == '}' || last_ch == ']' {
+					out_writer.write(b"\n").unwrap();
+					for _ in 0..tab_num {
+						out_writer.write(b"  ").unwrap();
+					}
+					last_ch = ' ';
+				} else {
+					last_ch = ch;
+				}
+			},
+			_ => {
+				if ch != ' ' || last_ch != ' ' {
+					out_writer.write(format!("{ch}").as_bytes()).unwrap();
+					last_ch = ch;
+				}
+			},
+		}
+	}
+
+	Ok(())
+}