ning / compress

High-performance, streaming/chunking Java LZF codec, compatible with standard C LZF package
Other
254 stars 40 forks source link

Unsafe-based decompressor of 0.9.7 fails on 2 sample files from 'maxcomp' data set #24

Closed cowtowncoder closed 11 years ago

cowtowncoder commented 11 years ago

Looks like latest version of "Unsafe" (aka optimal) decompressor has a small problem with specific test data files, found from test files from "maximum compression" data sets, as per:

https://github.com/ning/jvm-compressor-benchmark/wiki

and specifically, "vcfiu.hlp" and "ohs.doc". Since sizes are not affected, I am guessing it must be a specific overlapping back reference. Need to add a unit test to reproduce, issue a critical fix.

cowtowncoder commented 11 years ago

Indeed, looks like overlapping copy is problematic. Need to figure out efficient way to fix.