On 2013-06-04 08:40, Timothee Cour wrote:
A)
there already is std.zlib; why not have:
std.compress.zlib: public import std.zlib
std.compress.lzw: put this new module there instead of in std.compress
std.compress.image.png
std.compress.image.jpg
B)
rename:
std.compress.lzwCompress => std.compress.lzw.compress
std.compress. lzwExpand => std.compress.lzw.uncompress
which is more consistent with compress/uncompress from std.zlib
C)
maybe add a link to
https://en.wikipedia.org/wiki/Lempel%E2%80%93Ziv%E2%80%93Welch or other
source
D)
CircularBuffer belongs somewhere else; maybe std.range or std.container
I agree with all of these.
Perhaps it should be put in the review queue as well.
--
/Jacob Carlborg