whoughton / lz77 Goto Github PK

As per your request

I used some existing source as a reference, but I do not know where from at this stage, if anyone notices it, please let me know.

You are adapting Olle Törnström's 2009 code found here: https://github.com/olle/lz77-kit/blob/master/src/main/js/lz77.js, which is in turn based on Diogo Kollross's 2006 JS implementation here: https://web.archive.org/web/20091026234748/http://geocities.com/diogok_br/lz77/.

The jshint params can be moved into the package.json file.

Repro:

var lz = require('lz77');

var settings = {
    minStringLength: 6
};

var to_compress = "can't read my, can't read my, no he can't read my poker face";
console.log(to_compress);
var compressed = lz.compress(to_compress, settings);
console.log(compressed);
var decompressed = lz.decompress(compressed, settings);
console.log(decompressed);
console.assert(to_compress === decompressed);

Outputs:

can't read my, can't read my, no he can't read my poker face
can't read my, ca` "'no he` '( poker face
can't read my, can't read my, o he, can't read mpoker face

And trips the assertion. This also happens when setting it to below the default value of 5.

lz77 version = 1.1.0
node version = v6.5.0

Repro:

var lz = require('lz77');

var settings = {
    minStringLength: 5
};

var to_compress = 'hello hello baby you called I can\'t hear a thing';
console.log(to_compress);
var compressed = lz.compress(to_compress, settings);
console.log(compressed);
console.assert(to_compress === compressed);

minStringLength is set to 5 (the default), and there's a repeating substring of length 6 (hello ), so it seems like that should get replaced with a reference?

Correct and unify license information in package.json