Use standard int types in zlib-ng api.

Fixed variable set but not used static analysis warning in example.

  example.c:84:14: warning: variable ‘read’ set but not used [-Wunused-but-set-variable]
     int err, read;
  example.c:198:5: warning: Value stored to 'read' is never read [deadcode.DeadStores]
    read = PREFIX(gzfread)(uncompr, uncomprLen, 1, file);

Fixed clang static analysis warning value stored is never read in infcover.

  infcover.c:466:5: warning: Value stored to 'buf' is never read [deadcode.DeadStores]
    buf += len;

Disable unaligned access of > 4 bytes on 32-bit arm

Fixed defined WIN32 in switchlevels should be _WIN32.

Fixed deflate_quick algorithm not being used due to old check.

Added new lines in Alder32 AVX2 and SSSE3 to improve readability.

Zero entire s1 and s2 for possibly better performance.

Add UNLIKELY to first 3 branches in adler32 variants.

Added Adler32 SSSE3 and AVX2 implementations to functable.

Co-authored-by: Brian Bockelman <bockelman@gmail.com>
Co-authored-by: Mika T. Lindqvist <postmaster@raasu.org>

Remove unnecessary commented-out printf statements.

(cherry picked from commit e0284cc3ef470c433bc0831d71fba038c3251944)

Rewrite adler32 to have a SSE and AVX variant.

On a 2.3GHz Haswell, I get the following timings when doing
an adler32 call against 1907MB of data:

Adler32 base time: 970682 usec
Adler32 vectorized time: 403403 usec
Adler32 AVX time: 264453 usec

(cherry picked from commit 905c306d54ec37a9efd56b6561ce4ab843be3a1c)

Move Tracev flush statement into flush_pending.

Move Tracevv statements when emitting literal to zng_tr_tally_lit.

Move check for match length in deflate_quick to check_match.

Fixed whitespace formatting in bi_flush.

Removed example64 and minigzip64 since large file support is now natively supported.

For gzseek, gzoffset, gzopen, adler32_combine, crc32_combine and crc32_combine_gen, export 32-bit and 64-bit versions for zlib-compatible api and only 64-bit version (without 64 suffix) for zlib-ng native api.

Fixed wrong 64-bit casting in deflatePrime potentially causing bits to be lost.

    Arithmetic overflow: Using operator '<<' on a 4 byte value and then casting the result to a 8 byte value.
       Cast the value to the wider type before calling operator '<<' to avoid overflow (io.2).
    Arithmetic overflow: 32-bit value is shifted, then cast to 64-bit value.
       Results might not be an expected value.

Cast distance code to uint64_t before shifting in zng_emit_dist.

Fixed arithmetic overflow when emitting deflate header bits.

    Arithmetic overflow: 32-bit value is shifted, then cast to 64-bit value.
      Results might not be an expected value.

Clean up windows defines, use _WIN32

 + replaced WIN32 with _WIN32
 + removed unused WINDOWS/_WINDOWS defines
 + no need to test for __MINGW__, as _WIN32 is also defined

Fix specification of warnings for intel compilers

Make deflate_quick algorithm available to all architectures. #205

Added GitHub Actions workflow to test against libpng.

Add a workflow for clang and gcc static analysis

Run compression level tests without open mode.
Remove duplicate minigzip cmake tests.

Added help message to switchlevels.

Added help message to minigzip.

Add optimized slide_hash for POWER processors

This commit introduces a new slide_hash function that
uses VSX vector instructions to slide 8 hash elements at a time,
instead of just one as the standard code does.

Preparation for POWER optimizations

Add the scaffolding for future optimizations for POWER processors. Now
the build is capable of correctly detecting multiple processor
sub-architectures (ppc, ppc64 and ppc64le) and also if features
needed for the optimizations are available during build and runtime.

With these changes, adding a new optimized function for POWER should be
as simple as adding a new file under arch/power/, appending build
instructions to the build files and editing functable.c accordingly.

The UNALIGNED_OK flag is now also added by default for powerpc64le
targets.

Use char* instead of caddr_t.

Fixed multi-line assembly macro in UPDATE_HASH for MSVC when using ClangCl.
https://docs.microsoft.com/en-us/cpp/assembler/inline/defining-asm-blocks-as-c-macros

    insert_string_tpl.h(44,5): error : cannot use more than one symbol in memory operand
    insert_string_sse.c(28,13): message : expanded from macro 'UPDATE_HASH'

Remove toofar checks now that we always hash 4 bytes, and the minimum
match length is thus also 4. (MIN_MATCH is still 3 for other reasons)

Remove IPos typedef which also helps to reduce casting warnings.

Add friendly name to checkout action in GitHub Actions workflows.

Remove verbose out from pip and choco during cmake workflow.

Fixed casting warnings in trees.c with count being int and freq being uint16_t.
Changed to use uint16_t for all counting variable types.

  trees.c(431,45): warning C4244: '+=': conversion from 'int' to 'uint16_t', possible loss of data
  trees.c(431,45): warning C4244: '+=': conversion from 'int' to 'uint16_t', possible loss of data

Fixed casting warnings in calls to put_short and put_short_msb.

  deflate.c(845,32): warning C4244: 'function': conversion from 'unsigned int' to 'uint16_t', possible loss of data
  deflate.c(894,50): warning C4244: 'function': conversion from 'unsigned int' to 'uint16_t', possible loss of data

Reuse wsize variable in slide_hash_avx2.

Fixed casting warnings about copying len into pending buf, replaced with calls to put_short.

  deflate.c(1413,45): warning C4244: '=': conversion from 'unsigned int' to 'unsigned char', possible loss of data
  deflate.c(1414,50): warning C4244: '=': conversion from 'unsigned int' to 'unsigned char', possible loss of data
  deflate.c(1415,46): warning C4244: '=': conversion from 'unsigned int' to 'unsigned char', possible loss of data
  deflate.c(1416,51): warning C4244: '=': conversion from 'unsigned int' to 'unsigned char', possible loss of data

Fixed dist casting warnings in zng_tr_tally_dist.

    deflate_p.h(42,37): warning C4244: '=': conversion from 'unsigned int' to 'unsigned char', possible loss of data
    deflate_p.h(43,42): warning C4244: '=': conversion from 'unsigned int' to 'unsigned char', possible loss of data

Rename github actions fuzz workflow to be consistent with the rest of the workflows.

Fixed warning about bi_flush being defined but not used. #592

    trees_emit.h:75:13: warning: ‘bi_flush’ defined but not used

Separate makefile object list on each line for less merge conflicts in the future.

Fixed intrinsics spelling typo in CMake comments.

Fixed signedness warning in calls to __cpuid and cpuidex.

x86.c(29,22): warning C4057: 'function': 'int *' differs in indirection to slightly different base types from 'unsigned int [4]'
x86.c(43,24): warning C4057: 'function': 'int *' differs in indirection to slightly different base types from 'unsigned int [4]'

Remove duplicate zlib license from source comments.

Fixed unused variable compiler warning in checksum_fuzzer.

checksum_fuzzer.c:23:14: warning: variable ‘combine1’ set but not used [-Wunused-but-set-variable]
     uint32_t combine1, combine2;

Move contents tables to INDEX.md.

Added GitHub Actions CI NMake badge to README.md.

Fixed const correctness when assigning stream next_in.

Fixed no tab formatting in CVE-2003-0107.c.

Fixed casting warning from bi_reverse in maketrees.

maketrees.c(93,58): warning C4244: '=': conversion from 'unsigned int' to 'uint16_
t', possible loss of data

Group check_include_file together in CMakeLists.txt.

Fixed whitespace and tabbing in cmake files.

Change quick_insert_string memory access to be similar to insert_string.

Remove extra lines between functions and their comments.

Fixed casting warnings and non-standard initializer warning in example project.

Fixed target not being linked properly to zlib system libraries.

Fixed wrong cmake variable used in zlib dual link ci instance.

Added ability to set window bits for switchlevels.
Initialize deflateInit with first level which is necessary for deflate_quick testing where initial window size is set to 8k.

Use a constant to load adler-32 initial hash value.

Cache chocolatey packages on Windows for cmake GitHub Actions workflow.

Fixed missing window_padding added to window alloc size in deflateCopy.

Fixed zero length stored block left open when using Z_SYNC_FLUSH. Moved toggling of block_open back to deflate_quick since zero length stored block doesn't emit end of block code.

Check distance in deflate_quick using MAX_DIST(s).
Added check_match to validate matches when using deflate_quick in debug mode.

Fixed formatting for readability in switchlevels.

Set binary mode for stdin and stdout in switchlevels.

Rename match_p.h to match_tpl.h since it has been converted to a template header file.

Added features list to README.md.

Remove support for DYNAMIC_CRC_TABLE which was broken.

Rename archdetect files to detect-arch to match detect-sanitizer.

Move compiler sanitizer checking to separate cmake file.

Fixed compiler warning about inline already being defined when including zutil.h in C++ app.

Simplify generic hash function using knuth's multiplicative hash.

Use 64-bit bit buffer when emitting codes.

Unroll more in compare258_c for performance improvement.
Unify length count variable across all compare256 variants.
Early return without break for possible performance improvements.

Remove additional byte comparison check in compare258 SSE4 and AVX2 functions.

Split compare258 into compare256 for longest_match and compare258 for deflate_quick.

Converted compare258 to static and convert longest_match to template.

Added support for AVX2 intrinsics to compare258.

Combine longest_match implementations and use compare258 functable stub.

Changed deflate_quick to use compare258 functable stub.

Removed fallback for __builtin_ctzl since it is no longer used.

Modified longest_match variants to use associated compare258 functions.

Modified deflate_quick to use compare258_unaligned_sse4.

Abstracted out architecture specific implementations of 258 byte comparison to compare258.

Remove unused cmake-prefix variable.

Always use the GHA latest images and remove OS version number from CI names.

Change two NOT_TWEAK_COMPILER to check for MIN-MATCH!=3 instead.

Remove several NOT_TWEAK_COMPILER checks and their legacy code.

Added additional gzio tests for improved code coverage.

Added CI test for dual linking zlib-ng with system zlib.

Split tree emitting code into its own source header to be included by both trees.c and deflate_quick.c so that their functions can be statically linked for performance reasons.

Unify emitting of literals and match dist/lengths. Removed deflate quick static tables, allowing for 32k window.

Fix ARM64 build with Visual C++.

Added CMake and CI configuration for INFLATE_STRICT and INFLATE_ALLOW_INVALID_DISTANCE_TOOFAR_ARRR.

Standardize fill_window implementations and abstract out slide_hash_neon for ARM.

Allow deflate_quick() to continue processing input data after flushing pending buffer.