fixed a bug with a long rep

inikep · inikep · commit 5e618eda2c23 · 2016-01-04T19:39:42.000+01:00
diff --git a/lib/lz5hc.c b/lib/lz5hc.c
@@ -742,7 +742,7 @@ static int LZ5HC_compress_optimal_price (
         if (mlen >= MINMATCH)
         {
             LZ5_LOG_PARSER("%d: start try REP rep=%d mlen=%d\n", (int)((char*)ip-source), rep, mlen);
-            if (mlen > sufficient_len)
+            if (mlen > sufficient_len || mlen >= LZ5_OPT_NUM)
             {
                 best_mlen = mlen; best_off = 0; cur = 0; last_pos = 1;
                 goto encode;
@@ -927,6 +927,7 @@ static int LZ5HC_compress_optimal_price (
 
             LZ5HC_Insert(ctx, inr);
             match_num = LZ5HC_GetAllMatches(ctx, inr, ip, matchlimit, best_mlen, matches);
+            LZ5_LOG_PARSER("%d: LZ5HC_GetAllMatches match_num=%d\n", (int)((char*)inr-source), match_num);
 
             if (match_num > 0 && matches[match_num-1].len > sufficient_len)
             {

Original file line number	Diff line number	Diff line change
`@@ -742,7 +742,7 @@ static int LZ5HC_compress_optimal_price (`
`742`	`742`	`if (mlen >= MINMATCH)`
`743`	`743`	`{`
`744`	`744`	`LZ5_LOG_PARSER("%d: start try REP rep=%d mlen=%d\n", (int)((char*)ip-source), rep, mlen);`
`745`		`- if (mlen > sufficient_len)`
	`745`	`+ if (mlen > sufficient_len \|\| mlen >= LZ5_OPT_NUM)`
`746`	`746`	`{`
`747`	`747`	`best_mlen = mlen; best_off = 0; cur = 0; last_pos = 1;`
`748`	`748`	`goto encode;`
`@@ -927,6 +927,7 @@ static int LZ5HC_compress_optimal_price (`
`927`	`927`
`928`	`928`	`LZ5HC_Insert(ctx, inr);`
`929`	`929`	`match_num = LZ5HC_GetAllMatches(ctx, inr, ip, matchlimit, best_mlen, matches);`
	`930`	`+ LZ5_LOG_PARSER("%d: LZ5HC_GetAllMatches match_num=%d\n", (int)((char*)inr-source), match_num);`
`930`	`931`
`931`	`932`	`if (match_num > 0 && matches[match_num-1].len > sufficient_len)`
`932`	`933`	`{`