General refactor

[xnm3YQNfEx]

Fix usage of Box<> and unwrap Look into if there is a better way of managing the 'readbuffer' using core rust types

• Needs to support variable length reads rather than just a single byte

Fix ownership and references Possibly remove some of the unused huffman coding for canonical codes, or include it in compression logic. Can't use for decompression, as it modifies paths in the tree, invalidating it. Move compression related code into a compression module Change the "HuffmanTable" module to something else. Maybe remove some of the unused code like the struct, and from_codes().

• Can also keep it for debug or testing, and move the encode, decode, compress, and other functions into a different module namespace.

[xnm3YQNfEx]

Performed a first pass of a refactor Got rid of the custom readbuffer type in favor of using an iterator for the data Vec. Removed some unused code Moved all compression code into a compression module, with functions to take in Vec and return one rather than taking in file paths. Got rid of the HuffmanTable all together, just have it as an empty module with some of the higher level functions like encode and decode.

Still need to:

• investigate an issue where compression is giving different results than before, specifically with extra data at the end (still seems to decompress this different data without issue afterwards)
• Add tests for compression and decompression