Changes

Begin splitting files up (#76)
* rename file
 most of it is MMPatchLoader so that's what it'll be

* Remove corrupt #region
 It starts in one class and ends in another, I can't tell where it's
really supposed to go

* Move addon to its own file

* Put cats in a box

* Can has namespace

* Promote business cat to manager

* Unnecessary now

* Old stuff
 (commit: e29c9cb)

Change debug C# version to default
 (commit: 1191841)

VS, why u do dis?
 (commit: d2fd007)

Add ImmutableStack class
 (commit: 536ff0c)

Add PatchContext struct
 (commit: abba87e)

Use ImmutableStack and PatchContext in MM
 (commit: 05342e2)

Remove unused code
Apparently had to do with texture replacer corruption, but not called 
anywhere
 (commit: 8685da9)

Implement IEnumerable<T>
 (commit: 038db23)

Add Depth property
 (commit: 946194e)

Use immutability in CheckNeeds
 (commit: 475aa66)

Forgot a using directive
 (commit: 27f788f)

Ged rid of Win64 specific code
Doesn't matter anymore
 (commit: 04df683)

Needs to be included in the project too
 (commit: 340d113)

Add logging interface
 (commit: b1a8863)

Extract progress into its own object
 (commit: ad61311)

Use logger and progress
Make some things static that no longer depend on the patch loader's 
state
 (commit: 76c998c)

Remove blocking option
It's no longer used
 (commit: 55fc4e6)

Use inline variable declaration
 (commit: bf640f4)

Make log messages consistent
 (commit: f751ed4)

Make deletes and copies count toward patch count
 (commit: f03b578)

Make names more accurate
These are called before the patch is applied
 (commit: 51b6f75)

Simplify null check
 (commit: 8179c44)

move main project to its own directory
Allows more to be added
 (commit: 1a12a25)

Better output dir for debug
 (commit: f02f418)

Do not copy local
 (commit: 95362b5)

Add test project
 (commit: c4ba1b9)

Add MM, Assembly-CSharp, UnityEngine refs
 (commit: e8159e1)

Add console runner
Will be needed eventually
 (commit: 563a942)

Yo dawg, I heard you like tests
 (commit: a7f901e)

Add TestConfigNode class
Makes testing with ConfigNodes by simplifying creating them
 (commit: be37f18)

Reference TestUtils
 (commit: 278dad8)

Don't reference a specific version of System
 (commit: 66e6612)

Add test for ImmutableStack
 (commit: df9c8ec)

Add test for GetPath
 (commit: 3e71009)

Add NSubstitute
 (commit: 6e8285c)

Add tests for ModLogger
 (commit: 7398d4d)

Fix an error
 (commit: a5a095f)

Add UrlBuilder
Hackily creates UrlDir, UrlFile, UrlConfig for testing purposes
 (commit: 564b226)

Progress shouldn't depend on deleted subnodes
The number of needs unsatisfied nodes it should be counting is the 
number of root nodes that have been removed, not subnodes as well
 (commit: f190d17)

These should use actual URLs
Since all the calls were just using .url anyway
 (commit: fca9717)

These too
 (commit: 28e4ce7)

Minor logging tweak
 (commit: 8d8463d)

Add tests for PatchProgress
 (commit: d37f58b)

Replace DeepCopy with ConfigNode.CreateCopy
It does 100% the same thing (and is recursive)
 (commit: ca12c39)

Inline out variable declarations
Yay C#7
 (commit: 2b82489)

Obey naming conventions
 (commit: 47ce015)

Pull Command and ParseCommand out of MMPatchLoader
Would be nice if enums allowed static methods
 (commit: a9a990d)

Extract ShallowCopy
"this" is the node you're copying to so that the extension method is 
only modifying "its" node
 (commit: b43f79b)

Don't create duplicates in UrlBuilder
 (commit: 534eee2)

Add ArrayEnumerator
Enumerates arrays in a garbage-free way
 (commit: 24f2143)

PatchList
list of patches, 'nuff said
 (commit: 7138bbc)

Add PatchExtractor
Extracts patches from the game database and sorts them
 (commit: 9a55a65)

Add SafeUrl extension method for UrlConfig
Makes sure logging doesn't mess up, and fixes the weird quirk where a 
node with a name value ends up displaying that instead of its actual 
name
 (commit: 90027c2)

Use SafeUrl in logging
 (commit: 504354d)

Remove unused
Doesn't really have any benefit
 (commit: 331ab18)

Log when BEFORE or AFTER patch deleted
This is pretty much equivalent to unsatisfied NEEDS, so it should be 
noted as such.  Also log on an unsatisfied FOR, although this shouldn't 
happen (make it a warning)
 (commit: 3f8adee)

Fix case issues
Mods may not be lowercase to begin with, need to handle this
 (commit: 50346dc)

Extract IsBracketBalanced
 (commit: 99bc0bf)

Remove bracket unbalanced nodes when sorting
 (commit: e52a646)

Unused method
 (commit: 8a81310)

Bring back DeepCopy
Apparently KSP's default implementation fails on badly formed nodes
 (commit: 5a468be)

Fix bad region
 (commit: 2e5854d)

Make sure badly formed mod passes are an error
 (commit: 9bd8253)

That's a bug
 (commit: b010243)

Add some explanatory comments
 (commit: cacc840)

Unnecessary using directives
 (commit: c00735d)

Use sorted patches when applying
Improves performance somewhat
Verified that sorting patches takes almost no time even for a fairly 
large number of patches
 (commit: 8d71ac6)

Remove now-unnecessary try-catch
There's already one around it and we no longer care about removing 
patches from the database at this stage
 (commit: aa990db)

Replace big if with guard clause
Reduces indentation.  Insert nodes shouldn't exist here anyway
 (commit: 3ca1801)

Simplify this
It no longer has to look in actual passes here, so we can just use the 
name we want it to display.
It does change the way it displays in the loading screen but that seems 
fine.
 (commit: ec7c67c)

Invalid command = error on the patch extractor
This seems like the right place to check it
 (commit: 8098313)

Extract RemoveWS
 (commit: 180c2ee)

Fix logging
 (commit: 20a8afb)

Extract PrettyPrint
 (commit: 35d89f8)

Get rid of unnecesary using directives
 (commit: b52a80e)

Don't run PrePatchInit if cache is being used
Mod list is not necessary
 (commit: 45a4c0c)

Eliminate mods instance variable
 (commit: d397a80)

Use method param rather than instance var
Makes things easier to disentagle
 (commit: 12780a6)

Eliminate Update
Status will be updated when necessary anyway
 (commit: b6b628c)

Eliminate redundant logging
MMPatchLoader logs this info itself
 (commit: 7a40730)

Keep track of progress fraction independently
 (commit: 7e714a4)

Make StatusUpdate less general
If cache is used, status only needs to be set once, no need to check it 
every time
 (commit: 9236be1)

Move this
What I get for trying to make a bunch of changes and then split them 
into small commits
 (commit: 8696143)

Eliminate Progress instance variable
Make it local, inject where needed
 (commit: 58d52d9)

Make more methods static
All their instance variable dependencies have been eliminated
 (commit: 2645d22)

This is no longer necessary
And will probably result in an error anyway
 (commit: 824b077)

Move exception handling outside of PrettyConfig
Callers really shouldn't be trying to print the result if it resulted in 
an exception anyway
 (commit: ac0a149)

Tweak test
This isn't the case it was trying to test
 (commit: eb18b60)

Allow adding a ConfigNode.Value in initializer
Not useful yet but maybe at some point
 (commit: 4fdfb89)

Extract PrettyConfig (for UrlConfig)
 (commit: 70eca81)

Add one more test
 (commit: 55308af)

Make CheckNeeds static
Can now be extracted
 (commit: 1a7d1e3)

This can already be static
 (commit: bc451c5)

Remove unnecessary Using
 (commit: 60cb30a)

Improve url and node printing
* Handle null name explicitly
* Include url when printing a UrlConfig
 (commit: a035b67)

Extract CheckNeeds
Equality vs sameness mostly not tested for now, need to determine 
desired behavior
 (commit: 59a9220)

Ensure that final string printed to the screen is the actual status
 (commit: 784e7a8)

Fix up mod list logging
* Use a string builder
* Print assemblies in a nicer format (table)
 (commit: 13850b6)

Use Path.Combine
It's more concise then concatenating with the separator char
 (commit: cd3a468)

Unnecessary now
 (commit: 901b5a5)

Improve assembly list
* Get rid of unused code
* Include KSPAssembly version
 (commit: a5b1547)

Accidentally removed
 (commit: 9cf0011)

Move tracking number of patches
from mod list to sorting patches
 (commit: fde3fbc)

Put progress counts in their own object
Allows the same counts to be used with a different logger.  Also remove 
unused setter for NeedsUnsatisfiedRootCount
 (commit: f09622f)

Move exception handling out of FIleSHA
Callers should be aware of exceptions anyway
 (commit: ce43104)

Extract FileSHA
Interacts with the file system so difficult to test unfortunately
 (commit: 2dc9f91)

Fix unassigned variable
 (commit: 6f26939)

Make this extractable
 (commit: f3352db)

Extract GenerateModList
Unfortunately interacts with AssemblyLoader and the file system so not 
really testable
 (commit: e40426a)

Add MessageQueue
 (commit: 7a2e18d)

Add QueueLogger and supporting classes
Allows logging to a queue
 (commit: 7d0c587)

Don't keep track of non-root needs unsatisfied
Isn't used anywhere
 (commit: d1975db)

Add FatalErrorHandler
Allows us to display a message to the user and quit when an 
unrecoverable error occurs.
Can't really be tested unfortunately.
 (commit: 78d4635)

Add background task support
Allows a background task to be run and monitored, including if it exits 
due to an exception
 (commit: 172e2f3)

Begin creating Progress namespace
 (commit: 7f90887)

Finish creating Progress namespace
 (commit: b0f72c2)

Unnecessary directives
 (commit: 3dbfbb1)

Add needs test for and/or and capitalization
 (commit: 92ae91f)

Separate out progress counter
Make it so that all the values can be incremented but not otherwise 
messed with.
Allow a new progress tracker to be initialized that shares a counter 
with another but uses a different logger
 (commit: 6596b47)

Ensure Counter behaves like an int
 (commit: 9a05e1b)

More unnecessary using
 (commit: 9736800)

Add test for ! (not) in :NEEDS
 (commit: a1af725)

More unnecessary using directives
 (commit: 3a19ff8)

Extract application of patches to its own thread
Allows it to not be bound by logging which can be slow
 (commit: e9c341a)

Test and fix PatchProgress.ProgressFraction
Patches are now only counted after needs are checked, so this shouldn't 
consider needs unsatisfied nodes
 (commit: b0e02e0)

Tweak
 (commit: ba24af3)

Only convert to array once per pass
This is expensive
 (commit: 5fe79fd)

Make node matching its own method
Saves a level of indentation
 (commit: 20e6192)

Loop only applies to edit patches
Saves another indentation level.  Also remove MM_PATCH_LOOP {} after 
done
 (commit: 244af1a)

Don't convert to an array at all
It's not necessary.  Also don't use switch - makes things cleaner.  It's 
only 3 cases anyway
 (commit: cdeb5f7)

Ensure that user gets updates during long passes
The patcher can potentially generate log messages faster than the main 
thread can log them, causing frames that are noticeably long with no 
updates.  This ensures that yields still happen then.
Verified that this does not meaningfully affect performance.  Previous 
tests suggest that the time wasted by waiting until the next frame is 
relatively small.
 (commit: 30eeb6f)

Without switch, i is valid here
 (commit: 7e42de5)

Ensure time between each check of the log queue
This prevents the queue from being locked too often, slowing down the 
patching thread
 (commit: dc1a167)

Convert to an array initially
Apparently it saves a bit of time, and this won't be changed while 
patches run
 (commit: ec5310b)

Having an actual array here no longer necessary
 (commit: 7cc6c51)

Apparently Linq slows things down
I guess it matters at scale
 (commit: bd82d53)

Improve access of name a bit
Looks like GetValue("name") has a bit of overhead, instead we can check 
if the UrlConfig's type == name
 (commit: ce3893f)

Move loop out of loop
This is all a bit loopy
 (commit: 203ad88)

case should match filename
matters on some filesystems
 (commit: 73f7b39)

v3.0.0 for KSP 1.3.1
 (commit: 2caefbd)