diff --git a/src/log.rs b/src/log.rs index f36016a..7d4bba1 100644 --- a/src/log.rs +++ b/src/log.rs @@ -1,10 +1,10 @@ //! Einfaches, threadsicheres Logging-Modul. //! //! Merkmale: -//! - Ausgabe in Terminal (stdout/stderr) und zusätzlich in eine Logdatei im temporären Verzeichnis. +//! - Ausgabe auf Standard-Streams (stdout/stderr) und zusätzlich in eine Logdatei im temporären Verzeichnis. //! - Nachrichtenvorlage: `[DD.MM.YYYY HH:MM:SS.mmm][LEVEL][TAG]: ` mit Präfix ([!]/[?]/[i]/[d]). //! - Der aktuell verwendete Schweregradfilter ist statisch (`LOG_LEVEL`) und wird zur Laufzeit nicht geändert. -//! - Terminal-Erkennung und Dateihandle werden lazily initialisiert und zwischengespeichert (`OnceLock`). +//! - Dateihandle wird lazily initialisiert und zwischengespeichert (`OnceLock`). //! //! Hinweise: //! - Bei einem leeren `tag` wird ein leerer Tag-Abschnitt erzeugt. @@ -22,13 +22,13 @@ use std::env; use std::fmt::{Display, Formatter}; -use std::fs::{create_dir_all, File, OpenOptions}; -use std::io::{stdout, IsTerminal, Write}; +use std::fs::{File, OpenOptions, create_dir_all}; +use std::io::Write; use std::sync::{Mutex, OnceLock}; use crate::config; use crate::program; -use time::{macros::format_description, OffsetDateTime}; +use time::{OffsetDateTime, macros::format_description}; /// Schweregrade für Logeinträge in aufsteigender Detailtiefe. /// @@ -71,14 +71,11 @@ impl TryFrom for LogLevel { "warn" => Ok(LogLevel::Warn), "info" => Ok(LogLevel::Info), "debug" => Ok(LogLevel::Debug), - _ => Err(LogLevel::Info) + _ => Err(LogLevel::Info), } } } -/// Zwischenspeicher für die einmalig ermittelte Terminal-Fähigkeit von `stdout`. -static IS_TERMINAL: OnceLock = OnceLock::new(); - /// Globaler Schweregradfilter für Ausgabe. static LOG_LEVEL: OnceLock = OnceLock::new(); @@ -89,7 +86,7 @@ static LOG_FILE: OnceLock>> = OnceLock::new(); /// /// Verhalten: /// - Wenn `log_level` größer als der globale Filter ist, wird nichts ausgegeben. -/// - Bei Terminalausgabe gehen `Error`-Meldungen nach `stderr`, alle anderen nach `stdout`. +/// - `Error`-Meldungen gehen nach `stderr`, alle anderen nach `stdout`. /// - Zusätzlich wird in eine tägliche Logdatei im Temp-Verzeichnis geschrieben (wenn erfolgreich geöffnet). /// /// Parameter: @@ -109,12 +106,10 @@ pub fn log(tag: &str, message: &str, log_level: LogLevel) { if log_level <= get_log_level() { let message: String = format_message(tag, message, &log_level); - if is_terminal() { - if log_level == LogLevel::Error { - eprintln!("{}", message) - } else { - println!("{}", message) - } + if log_level == LogLevel::Error { + eprintln!("{}", message); + } else { + println!("{}", message); } let file_lock = get_or_init_log_file(); @@ -138,17 +133,7 @@ fn get_log_level() -> LogLevel { *LOG_LEVEL.get_or_init(|| { let log_level = &config::get_config().general.log_level; LogLevel::try_from(log_level.to_string()).unwrap_or(LogLevel::Info) - } - ) -} - -/// Ermittelt einmalig, ob `stdout` ein Terminal ist, und cached das Ergebnis. -/// -/// Rückgabe: -/// - `true`, wenn `stdout` ein TTY/Terminal ist. -/// - `false` andernfalls. -fn is_terminal() -> bool { - *IS_TERMINAL.get_or_init(|| stdout().is_terminal()) + }) } /// Formatiert eine Lognachricht mit Zeitstempel, Level, Tag und Präfix. @@ -234,10 +219,7 @@ fn open_log_file() -> std::io::Result { let file_name = format!("log-{}.log", date_str); dir.push(file_name); - OpenOptions::new() - .create(true) - .append(true) - .open(dir) + OpenOptions::new().create(true).append(true).open(dir) } #[cfg(test)]