Don't write end of last block when returning finish_started.

Fixed avail_out == 0 conditional not returning need_more in deflate_quick.
Fixed ending block when returning need_more caused problems with inflate.

So instead of ending the block each time the function returns to finish the last block, we check upon start to see if it is the last block and if the last block has been started, and if not it will close the previous block and start the last block.

Rename from memchunk to chunkset.

Rename add_intrinsics_option to set_intrinsics_option.

Clean up header includes for Win32 and ARM.
Fixed MSVC compiler warning in storechunk for ARM Neon.
Fixed arm_neon.h include for memchunk for ARM Neon.

Only calculate inflate chunk size once and store it for future use for performance.

Split memcopy by architecture.
Use uint8_t[8] struct on big-endian machines for speed.

Apply compiler flags only to source files that require them in CMake.

Add POWER8_VSX_SLIDEHASH macro

The slide_hash optimization for POWER currently depends on POWER8 macro
to be enabled. Switch this to use POWER8_VSX_SLIDEHASH instead to better
match the naming scheme used by other archs.

cmake: Use consistent arch names for POWER targets

The arch detection code used by cmake build exports ppc arch names in an
abbreviated way, e.g. ppc64le instead of powerpc64le. Some parts of
CMakeLists.txt are using the longer form instead, which will not
properly match the desired subarch as expected. Switch to using the
longer form everywhere for consistency with 'configure' script and with
arch names obtained when running under QEMU.

Do not set s->prev_length in deflate_medium, it is previously set in lm_init already.

Don't set bflush to zero since it is assigned before used.

Don't need to set s->match_length to zero in deflate_huff.

Use local variable to store match_len in deflate_rle.

Only set current_match to literal if hash_head == 0.

Fixed whitespace in deflate_quick.

Use local variable to store match_len in deflate_fast.

Move check for hash_head != 0 directly after quick_insert_string.

Separate defines on individual lines to prevent merge conflicts.

Add adler32_len_64 for length < 64

Add adler32_len_64 to adler32_p.h to allow reuse by other adler32
implementations that may need it.

Move DO* macro definitions to adler32_p.h

Add new generic definitions of DO* macros used by adler32 algorithms to
adler32_p.h to allow reuse by other adler32 implementations.

Add tests for Adler32

Adler32 vector optimization for Power.

This commit implements a Power (POWER8+) vector optimization for Adler32
checksum using VSX (vector) instructions. The VSX adler32 checksum is up
to 10x fast than the adler32 baseline code.

Author: Rogerio Alves <rcardoso@linux.ibm.com>

Determine whether platform supports dll declspec

Clang contains __has_declspec_attribute for MSVC compatibility. This can be
used to determine whether __declspec is available. Use this to determine if
dllimport/dllexport should be used when exporting the zlib API.

Fixed deflate_quick missing from Win32 ARM makefile.

Fix storechunk for MSVC ARM.

Fixed missing compare258 object compilation in Win32 ARM makefiles.

Don't explicitly enable neon flag on MSVC ARM64 builds.

Removed old references to fill_window in NMake ARM makefiles.

Added CI configurations for MSVC ARM and ARM64.

Enable NEON-optimized slide_hash on aarch64 if NEON-optimizations are enabled.

Fixed casting warnings

Add cmake build dir to gitignore

+ ignore build.* or build-* top level dirs

Do not use word xor (iso646 legacy)

Co-authored-by: pps83 <pps83@users.noreply.github.com>

test/pkgcheck.sh: verify cmake and configure install identical bits (excluding .a and .so for now)

Also add workflow to run pkgcheck.sh on one system.

cmake: generate same zconf.h as configure

Also make comment in zconf.h less configure-specific

cmake: generate same .pc file contents as configure

cmake: install .pc file to same location as configure

deflate_slow: Suppress a static analysis warning triggered by recent refactoring c459b4f5.

Move check to start block out of main loop for performance reasons.

Fixed deflate_quick to not emit a block when there is no available input. Pigz requires no blocks to be emitted in certain instances when calling deflate with Z_BLOCK.
Fixed end block not being emitted between calls to deflate_quick causing invalid stored block lengths in certain instances.

Updated features with new Adler32 implementations.

Remove unnecessary adler32_neon.h.

Added regression test for deflateEnd returning -3 when using deflate quick. #382

Makefile: fail minigzip test if minigzip aborts

Pipeline exit status ignores all but the last
process in the pipeline, so break up the pipeline
so we can see minigzip's exit status.

Also fix ancient quoting bug; intention was likely to use
shell $$ variable to uniquify temp file name.

minigzip.c: bring back default bahavior, use distinguishable error code on bad cmdline

Fixes #635

Remove Borland C++ compiler references

There are a few guards checking for Borland C++ 5.0 or greater which was released
in 1996. While there is still a descendent of this compiler in Embarcadero C++ Builder
its value for __BORLANDC__ is greater than 0x500 so it is safe to remove these guards.

infcover.c: assert that we avoid zero-length allocations; fixes #593

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.