changed macro HEAPMODE into LZ4_HEAPMODE

This macro is susceptible to be triggered from user side
typically through compiler flag (-DLZ4_HEAPMODE=1).
In which case, it makes sense to prefix the macro
since we want to reduce potential side-effect on namespace.
This commit is contained in:
Yann Collet 2017-05-01 22:32:21 -07:00
parent 11bfedb6c3
commit a8dd86d93e
2 changed files with 14 additions and 13 deletions

View File

@ -37,12 +37,12 @@
* Tuning parameters
**************************************/
/*
* HEAPMODE :
* LZ4_HEAPMODE :
* Select how default compression functions will allocate memory for their hash table,
* in memory stack (0:default, fastest), or in memory heap (1:requires malloc()).
*/
#ifndef HEAPMODE
# define HEAPMODE 0
#ifndef LZ4_HEAPMODE
# define LZ4_HEAPMODE 0
#endif
/*
@ -677,7 +677,7 @@ int LZ4_compress_fast_extState(void* state, const char* source, char* dest, int
int LZ4_compress_fast(const char* source, char* dest, int inputSize, int maxOutputSize, int acceleration)
{
#if (HEAPMODE)
#if (LZ4_HEAPMODE)
void* ctxPtr = ALLOCATOR(1, sizeof(LZ4_stream_t)); /* malloc-calloc always properly aligned */
#else
LZ4_stream_t ctx;
@ -686,7 +686,7 @@ int LZ4_compress_fast(const char* source, char* dest, int inputSize, int maxOutp
int const result = LZ4_compress_fast_extState(ctxPtr, source, dest, inputSize, maxOutputSize, acceleration);
#if (HEAPMODE)
#if (LZ4_HEAPMODE)
FREEMEM(ctxPtr);
#endif
return result;
@ -890,7 +890,7 @@ static int LZ4_compress_destSize_extState (LZ4_stream_t* state, const char* src,
int LZ4_compress_destSize(const char* src, char* dst, int* srcSizePtr, int targetDstSize)
{
#if (HEAPMODE)
#if (LZ4_HEAPMODE)
LZ4_stream_t* ctx = (LZ4_stream_t*)ALLOCATOR(1, sizeof(LZ4_stream_t)); /* malloc-calloc always properly aligned */
#else
LZ4_stream_t ctxBody;
@ -899,7 +899,7 @@ int LZ4_compress_destSize(const char* src, char* dst, int* srcSizePtr, int targe
int result = LZ4_compress_destSize_extState(ctx, src, dst, srcSizePtr, targetDstSize);
#if (HEAPMODE)
#if (LZ4_HEAPMODE)
FREEMEM(ctx);
#endif
return result;

View File

@ -49,11 +49,12 @@ typedef struct {
} LZ4HC_optimal_t;
/* price in bits */
/* price in bytes */
FORCE_INLINE size_t LZ4HC_literalsPrice(size_t litlen)
{
size_t price = litlen;
if (litlen >= (size_t)RUN_MASK) price += 1 + (litlen-RUN_MASK)/255;
if (litlen >= (size_t)RUN_MASK)
price += 1 + (litlen-RUN_MASK)/255;
return price;
}
@ -66,7 +67,7 @@ FORCE_INLINE size_t LZ4HC_sequencePrice(size_t litlen, size_t mlen)
price += LZ4HC_literalsPrice(litlen);
if (mlen >= (size_t)(ML_MASK+MINMATCH))
price+= 1+(mlen-(ML_MASK+MINMATCH))/255;
price+= 1 + (mlen-(ML_MASK+MINMATCH))/255;
return price;
}
@ -232,7 +233,7 @@ static int LZ4HC_compress_optimal (
size_t const llen = ip - anchor;
size_t last_pos = 0;
size_t match_num, cur, best_mlen, best_off;
memset(opt, 0, sizeof(LZ4HC_optimal_t));
memset(opt, 0, sizeof(LZ4HC_optimal_t)); /* memset only the first one */
match_num = LZ4HC_BinTree_GetAllMatches(ctx, ip, matchlimit, MINMATCH-1, matches, fullUpdate);
if (!match_num) { ip++; continue; }
@ -279,7 +280,7 @@ static int LZ4HC_compress_optimal (
}
if (price < (size_t)opt[cur].price)
SET_PRICE(cur, 1, 0, litlen, price); /* note : increases last_pos */
SET_PRICE(cur, 1 /*mlen*/, 0 /*off*/, litlen, price); /* note : increases last_pos */
}
if (cur == last_pos || curPtr >= mflimit) break;
@ -331,7 +332,7 @@ encode: /* cur, last_pos, best_mlen, best_off must be set */
opt[cur].off = (int)best_off;
best_mlen = ml;
best_off = offset;
if (ml > cur) break;
if (ml > cur) break; /* can this happen ? */
cur -= ml;
}