I had previously been under the assumption that closing always guaranteed a flush. I don't know of any issues that were being caused, but I should push this before i forget.
* Figure out how to access OverlayAddress in wrapper or reader (ideally the latter) for a non-PE reader/wrapper
* Code works
* Remove TODOs
* First round of fixes.
* use constants
* remove comment
* Attempt 1
* Use private helper class
* Remove comment
* Add missing summaries
* Don't use create pattern
* Remove removed variable summary
* Reduce what needs to be passed in
* Further reduce what needs to be passed in
* These didn't get tabbed for some reason
* First round of fixes.
* Second round of changes
* Add a few comments.
* WIP
* WIP 2
* Todo: you're missing a read somehow and getting misaligned by two bytes? maybe properly implementing the buffer will magically fix it
* continued blocks my behated
* Pre-major-testing
* Forgot to add summaries
* Attempt to properly roll back. The state i wanted to roll back to wasn't in a commit before.
* Figured out the issue with the rolled back commit, this has to be a while loop because of 0 byte files. Reimplemented clean code.
* Comment so I don't forget why it's like this
* Skip unsupported compression types before opening filestream.
* Reenable non-start cab skipping
* TODO so i don't forget
* TODO so I don't forget
* iterate on continued block correctly.
* Handle incomplete extraction better
* Remove TODOs to ready for PR comments
* Missed one
* Some minor fixes before rewriting everything
* Next round of fixes.
* Next set of fixes
* Fix debug output
* Better fix
* initial
* Fixed
* Move function to proper location, clear memory better
* work so far
* more
* more work
* No more per file seeking
* Finished work for now.
* First round of fixes.
* Return if array is empty too
