From 897f5e9834ed4e8b4e34380b12a6b9e7b911af90 Mon Sep 17 00:00:00 2001 From: Yann Collet Date: Tue, 7 Nov 2017 17:37:31 -0800 Subject: 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. --- lib/lz4hc.c | 2 -- lib/lz4opt.h | 1 - 2 files changed, 3 deletions(-) diff --git a/lib/lz4hc.c b/lib/lz4hc.c index 3498391..5e2dd2a 100644 --- 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 --- 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); -- cgit v0.12