]> git.ipfire.org Git - thirdparty/zstd.git/commitdiff
No longer pass srcSize to minTableLog 3285/head
authorDanielle Rozenblit <drozenblit@fb.com>
Mon, 17 Oct 2022 15:03:44 +0000 (08:03 -0700)
committerDanielle Rozenblit <drozenblit@fb.com>
Mon, 17 Oct 2022 15:03:44 +0000 (08:03 -0700)
lib/common/huf.h
lib/compress/huf_compress.c

index d79acd1d53ae737aca0e3dd012cb5986970c183f..595b2f6db5d2cfb9a5ddde0c07fbb0626e2ef158 100644 (file)
@@ -190,7 +190,7 @@ typedef enum {
  *  For example, it's possible to compress several blocks using the same 'CTable',
  *  or to save and regenerate 'CTable' using external methods.
  */
-unsigned HUF_minTableLog(size_t srcSize, unsigned symbolCardinality);
+unsigned HUF_minTableLog(unsigned symbolCardinality);
 unsigned HUF_cardinality(const unsigned* count, unsigned maxSymbolValue);
 unsigned HUF_optimalTableLog(unsigned maxTableLog, size_t srcSize, unsigned maxSymbolValue, void* workSpace,
  size_t wkspSize, HUF_CElt* table, const unsigned* count, HUF_depth_mode depthMode); /* table is used as scratch space for building and testing tables, not a return value */
index 149e098607b6e302c7b06a63f729ce71cdadcc36..e94b398da87d6e8d193c7172774eb91126aff973 100644 (file)
@@ -1247,16 +1247,16 @@ unsigned HUF_cardinality(const unsigned* count, unsigned maxSymbolValue)
     return cardinality;
 }
 
-unsigned HUF_minTableLog(size_t srcSize, unsigned symbolCardinality)
+unsigned HUF_minTableLog(unsigned symbolCardinality)
 {
     U32 minBitsSymbols = ZSTD_highbit32(symbolCardinality) + 1;
-    assert(srcSize > 1); /* Not supported, RLE should be used instead */
     return minBitsSymbols;
 }
 
 unsigned HUF_optimalTableLog(unsigned maxTableLog, size_t srcSize, unsigned maxSymbolValue, void* workSpace, size_t wkspSize, HUF_CElt* table, const unsigned* count, HUF_depth_mode depthMode)
 {
     unsigned optLog = FSE_optimalTableLog_internal(maxTableLog, srcSize, maxSymbolValue, 1);
+    assert(srcSize > 1); /* Not supported, RLE should be used instead */
 
     if (depthMode == HUF_depth_optimal) { /** Test valid depths and return optimal **/
         BYTE* dst = (BYTE*)workSpace + sizeof(HUF_WriteCTableWksp);
@@ -1268,7 +1268,7 @@ unsigned HUF_optimalTableLog(unsigned maxTableLog, size_t srcSize, unsigned maxS
 
         if (wkspSize < sizeof(HUF_buildCTable_wksp_tables)) return optLog;
 
-        for (huffLog = HUF_minTableLog(srcSize, symbolCardinality); huffLog <= maxTableLog; huffLog++) {
+        for (huffLog = HUF_minTableLog(symbolCardinality); huffLog <= maxTableLog; huffLog++) {
             maxBits = HUF_buildCTable_wksp(table, count,
                                             maxSymbolValue, huffLog,
                                             workSpace, wkspSize);