top | item 20172140

Bzip2 in Rust: porting the randomization table

7 points| kaeso | 6 years ago |people.gnome.org

9 comments

order

Annatar|6 years ago

"See the commit for the corresponding extern declarations in bzlib_private.h. With those functions and the table ported to Rust, we can remove randtable.c. Yay!"

You just made bzip2 artificially depend on Rust. I have no Rust compiler on Solaris 10, which means that with this commit I can no longer compile your code. This goes for any other OS which doesn't have Rust.

steveklabnik|6 years ago

Introducing a Rust dependency isn’t accidental when it’s the point of the whole enterprise.

The author is also keeping the C version around. bzip2 didn’t even have version control before this. The C version will live on for all those who require it.