Return doesn't matter here anymore

This commit is contained in:
Matt Nadareski
2020-10-07 16:19:40 -07:00
parent f3031fe5f9
commit 82aa1807a9

View File

@@ -239,12 +239,11 @@ namespace SabreTools.Library.Logging
/// </summary> /// </summary>
/// <param name="output">String to be written log</param> /// <param name="output">String to be written log</param>
/// <param name="loglevel">Severity of the information being logged</param> /// <param name="loglevel">Severity of the information being logged</param>
/// <returns>True if the output could be written, false otherwise</returns> private static void Log(string output, LogLevel loglevel)
public static bool Log(string output, LogLevel loglevel)
{ {
// If the log level is less than the filter level, we skip it but claim we didn't // If the log level is less than the filter level, we skip it but claim we didn't
if (loglevel < LowestLogLevel) if (loglevel < LowestLogLevel)
return true; return;
// USER and ERROR writes to console // USER and ERROR writes to console
if (loglevel == LogLevel.USER || loglevel == LogLevel.ERROR) if (loglevel == LogLevel.USER || loglevel == LogLevel.ERROR)
@@ -265,11 +264,11 @@ namespace SabreTools.Library.Logging
Console.WriteLine(ex); Console.WriteLine(ex);
Console.WriteLine("Could not write to log file!"); Console.WriteLine("Could not write to log file!");
if (ThrowOnError) throw ex; if (ThrowOnError) throw ex;
return false; return;
} }
} }
return true; return;
} }
#endregion #endregion