Changes

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)