claunia/SabreTools
1
0
Fork 0
mirror of https://github.com/claunia/SabreTools.git synced 2025-12-16 19:14:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

Dispose only if there's something to dispose

Browse Source
This commit is contained in:
Matt Nadareski
2016-05-23 12:11:22 -07:00
parent 79d1da0583
commit 5b48934eb3
2 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
DATFromDir/DATFromDir.cs
View File

@@ -456,13 +456,19 @@ namespace SabreTools
catch (InvalidOperationException) catch (InvalidOperationException)
{ {
encounteredErrors = true; encounteredErrors = true;
archive.Dispose(); if (archive != null)
{
archive.Dispose();
}
} }
catch (Exception ex) catch (Exception ex)
{ {
_logger.Error(ex.ToString()); _logger.Error(ex.ToString());
encounteredErrors = true; encounteredErrors = true;
archive.Dispose(); if (archive != null)
{
archive.Dispose();
}
} }
} }

2
SabreHelper/Logger.cs
View File

@@ -11,7 +11,7 @@ namespace SabreTools.Helper
/// - Create log files by date /// - Create log files by date
/// - Clean up log files if there are more than x number per program /// - Clean up log files if there are more than x number per program
/// - Allow for "triggerable" logging done on an interval (async) /// - Allow for "triggerable" logging done on an interval (async)
/// - Log filtering? /// - Log filtering? (#if debug?)
/// </remarks> /// </remarks>
public class Logger public class Logger
{ {