removed the ip++ at the beginning of block

The first byte used to be skipped
to avoid a infinite self-comparison.
This is no longer necessary, since init() ensures that index starts at 64K.

The first byte is also useless to search when each block is independent,
but it's no longer the case when blocks are linked.

Removing the first-byte-skip saves
about 10 bytes / MB on files compressed with -BD4 (linked blocks 64Kb),
which feels correct as each MB has 16 blocks of 64KB.

diff --git a/lib/lz4hc.c b/lib/lz4hc.c
index 3498391..5e2dd2a 100644 (file)
--- a/lib/lz4hc.c
+++ b/lib/lz4hc.c
@@ -429,8 +429,6 @@ static int LZ4HC_compress_hashChain (
     if (limit == limitedDestSize) oend -= LASTLITERALS;                  /* Hack for support limitations LZ4 decompressor */
     if (inputSize < LZ4_minLength) goto _last_literals;                  /* Input too small, no compression (all literals) */
 
-    ip++;
-
     /* Main Loop */
     while (ip < mflimit) {
         ml = LZ4HC_InsertAndFindBestMatch (ctx, ip, matchlimit, (&ref), maxNbAttempts);

diff --git a/lib/lz4opt.h b/lib/lz4opt.h
index dd22b7a..03ab825 100644 (file)
--- a/lib/lz4opt.h
+++ b/lib/lz4opt.h
@@ -127,7 +127,6 @@ static int LZ4HC_compress_optimal (
     /* init */
     DEBUGLOG(5, "LZ4HC_compress_optimal");
     if (sufficient_len >= LZ4_OPT_NUM) sufficient_len = LZ4_OPT_NUM-1;
-    ip++;
 
     /* Main Loop */
     assert(ip - anchor < LZ4_MAX_INPUT_SIZE);