paolobarbolini / bzip2-rs

Pure Rust bzip2 decoder
Apache License 2.0
44 stars 6 forks source link

PositionalMTFEncoder #3

Closed GaurangTandon closed 3 years ago

GaurangTandon commented 3 years ago

Description

Tests

GaurangTandon commented 3 years ago
  1. I'll remove the println!'s on the next commit (forgot earlier :P)
  2. Considering the tight coupling with ptr and in_use, should we move this directly to encoder module itself, instead of keeping it in standalone move_to_front.rs?
  3. Any other comments on performance code or otherwise? (I left a couple of TODOs as I couldn't figure them out yet, if you know, please tell :D)
GaurangTandon commented 3 years ago

For future reference:

MTF generator code
Random string generator