[Globals] Add global variables to be used everywhere

This commit is contained in:
Matt Nadareski
2017-03-01 21:26:27 -08:00
parent 9c29fb0dbe
commit 89fd9f2237
22 changed files with 672 additions and 780 deletions

View File

@@ -72,7 +72,7 @@ namespace SabreTools.Helper.Skippers
SourceFile = Path.GetFileNameWithoutExtension(filename);
Logger logger = new Logger();
XmlReader xtr = FileTools.GetXmlTextReader(filename, logger);
XmlReader xtr = FileTools.GetXmlTextReader(filename);
if (xtr == null)
{
@@ -338,16 +338,16 @@ namespace SabreTools.Helper.Skippers
/// <param name="skipperName">Name of the skipper to be used, blank to find a matching skipper</param>
/// <param name="logger">Logger object for file and console output</param>
/// <returns>The SkipperRule that matched the file</returns>
public static SkipperRule GetMatchingRule(string input, string skipperName, Logger logger)
public static SkipperRule GetMatchingRule(string input, string skipperName)
{
// If the file doesn't exist, return a blank skipper rule
if (!File.Exists(input))
{
logger.Error("The file '" + input + "' does not exist so it cannot be tested");
Globals.Logger.Error("The file '" + input + "' does not exist so it cannot be tested");
return new SkipperRule();
}
return GetMatchingRule(File.OpenRead(input), skipperName, logger);
return GetMatchingRule(File.OpenRead(input), skipperName);
}
/// <summary>
@@ -358,7 +358,7 @@ namespace SabreTools.Helper.Skippers
/// <param name="logger">Logger object for file and console output</param>
/// <param name="keepOpen">True if the underlying stream should be kept open, false otherwise</param>
/// <returns>The SkipperRule that matched the file</returns>
public static SkipperRule GetMatchingRule(Stream input, string skipperName, Logger logger, bool keepOpen = false)
public static SkipperRule GetMatchingRule(Stream input, string skipperName, bool keepOpen = false)
{
SkipperRule skipperRule = new SkipperRule();
@@ -369,7 +369,7 @@ namespace SabreTools.Helper.Skippers
}
// Loop through and find a Skipper that has the right name
logger.Verbose("Beginning search for matching header skip rules");
Globals.Logger.Verbose("Beginning search for matching header skip rules");
List<Skipper> tempList = new List<Skipper>();
tempList.AddRange(List);
@@ -519,7 +519,7 @@ namespace SabreTools.Helper.Skippers
input.Dispose();
}
logger.User(" Matching rule found!");
Globals.Logger.User(" Matching rule found!");
return rule;
}
}
@@ -535,7 +535,7 @@ namespace SabreTools.Helper.Skippers
// If we have a blank rule, inform the user
if (skipperRule.Tests == null)
{
logger.Verbose("No matching rule found!");
Globals.Logger.Verbose("No matching rule found!");
}
return skipperRule;

View File

@@ -34,16 +34,15 @@ namespace SabreTools.Helper.Skippers
/// </summary>
/// <param name="input">Input file name</param>
/// <param name="output">Output file name</param>
/// <param name="logger">Logger object for file and console output</param>
/// <returns>True if the file was transformed properly, false otherwise</returns>
public bool TransformFile(string input, string output, Logger logger)
public bool TransformFile(string input, string output)
{
bool success = true;
// If the input file doesn't exist, fail
if (!File.Exists(input))
{
logger.Error("I'm sorry but '" + input + "' doesn't exist!");
Globals.Logger.Error("I'm sorry but '" + input + "' doesn't exist!");
return false;
}
@@ -53,8 +52,8 @@ namespace SabreTools.Helper.Skippers
Directory.CreateDirectory(Path.GetDirectoryName(output));
}
logger.User("Attempting to apply rule to '" + input + "'");
success = TransformStream(File.Open(input, FileMode.Open, FileAccess.Read, FileShare.ReadWrite), File.OpenWrite(output), logger);
Globals.Logger.User("Attempting to apply rule to '" + input + "'");
success = TransformStream(File.Open(input, FileMode.Open, FileAccess.Read, FileShare.ReadWrite), File.OpenWrite(output));
// If the output file has size 0, delete it
if (new FileInfo(output).Length == 0)
@@ -78,11 +77,10 @@ namespace SabreTools.Helper.Skippers
/// </summary>
/// <param name="input">Input stream</param>
/// <param name="output">Output stream</param>
/// <param name="logger">Logger object for file and console output</param>
/// <param name="keepReadOpen">True if the underlying read stream should be kept open, false otherwise</param>
/// <param name="keepWriteOpen">True if the underlying write stream should be kept open, false otherwise</param>
/// <returns>True if the file was transformed properly, false otherwise</returns>
public bool TransformStream(Stream input, Stream output, Logger logger, bool keepReadOpen = false, bool keepWriteOpen = false)
public bool TransformStream(Stream input, Stream output, bool keepReadOpen = false, bool keepWriteOpen = false)
{
bool success = true;
@@ -92,7 +90,7 @@ namespace SabreTools.Helper.Skippers
|| (Operation > HeaderSkipOperation.Byteswap && (extsize % 4) != 0)
|| (Operation > HeaderSkipOperation.Bitswap && (StartOffset == null || StartOffset % 2 == 0)))
{
logger.Error("The stream did not have the correct size to be transformed!");
Globals.Logger.Error("The stream did not have the correct size to be transformed!");
return false;
}
@@ -101,7 +99,7 @@ namespace SabreTools.Helper.Skippers
BinaryReader br = null;
try
{
logger.User("Applying found rule to input stream");
Globals.Logger.User("Applying found rule to input stream");
bw = new BinaryWriter(output);
br = new BinaryReader(input);
@@ -189,7 +187,7 @@ namespace SabreTools.Helper.Skippers
}
catch (Exception ex)
{
logger.Error(ex.ToString());
Globals.Logger.Error(ex.ToString());
return false;
}
finally