From 593121c03acfd8cae0a1741c52c7bf1b49b6d005 Mon Sep 17 00:00:00 2001 From: Matt Nadareski Date: Mon, 6 Jan 2025 10:36:49 -0500 Subject: [PATCH] Add static logging to FileTypeTool --- SabreTools.FileTypes/FileTypeTool.cs | 26 +++++++++++++++++--------- 1 file changed, 17 insertions(+), 9 deletions(-) diff --git a/SabreTools.FileTypes/FileTypeTool.cs b/SabreTools.FileTypes/FileTypeTool.cs index ed30bf5a..6bc0fc2e 100644 --- a/SabreTools.FileTypes/FileTypeTool.cs +++ b/SabreTools.FileTypes/FileTypeTool.cs @@ -1,9 +1,11 @@ +using System; using System.IO; using SabreTools.FileTypes.Aaru; using SabreTools.FileTypes.Archives; using SabreTools.FileTypes.CHD; using SabreTools.Hashing; using SabreTools.IO.Extensions; +using SabreTools.IO.Logging; using SabreTools.Matching; using SabreTools.Skippers; using static SabreTools.FileTypes.Constants; @@ -12,6 +14,15 @@ namespace SabreTools.FileTypes { public static class FileTypeTool { + #region Logging + + /// + /// Logging object + /// + private static readonly Logger _logger = new(); + + #endregion + #region File Info /// @@ -52,10 +63,9 @@ namespace SabreTools.FileTypes return baseFile; } - catch + catch (Exception ex) { - // Exceptions are currently not logged - // TODO: Log exceptions + _logger.Error(ex); return new BaseFile(); } } @@ -96,10 +106,9 @@ namespace SabreTools.FileTypes input.SeekIfPossible(); return baseFile; } - catch + catch (Exception ex) { - // Exceptions are currently not logged - // TODO: Log exceptions + _logger.Error(ex); return new BaseFile(); } } @@ -245,10 +254,9 @@ namespace SabreTools.FileTypes using Stream stream = File.Open(input, FileMode.Open, FileAccess.Read, FileShare.ReadWrite); magic = stream.ReadBytes(8); } - catch + catch (Exception ex) { - // Exceptions are currently not logged - // TODO: Log exceptions + _logger.Error(ex); return null; }