Remove current directory library include path in configure not present in cmake.

Add likely/unlikely hinting to all deflate algorithms.

Added CI instances for ARM/AARCH64 without acle/neon.

Fixed configure applying -march flags globally for ARM.
Fixed -mfpu=neon not being applied on arm|arm7 if available.

Remove duplicate Windows GCC CI instance.

Added powerpc instances to CI for configure.

Merge pull request #668 from nmoinvaz/fixes/apply-float-abi-c-flags

Fixed issues with float abi in cmake

Clean up comments about setting C flags.

Modify cmake to apply native flag when using intrinsic flags same as configure.

Apply float abi flag to all files in cmake (same as configure).
Don't override -mfloat-abi if set via -DCMAKE_C_FLAGS in cmake.
Only set -mfloat-abi=softfp if compiler ends in eabi in cmake. (same as configure).

Remove some extra ci configurations.

Apply neonflag only to files using neon instructions with configure.

Fixed integer casting and signed comparison warning in test_gzio.

    example.c(199,57): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data
    example.c:207:32: warning: comparison between signed and unsigned integer expressions

Improve deflatePrime test by wrapping gzip header and footer around deflate stream.

Fixed bad shift operation warning in deflatePrime.
Check that bits value is not greater than bits allowed by value type.

  CID 293475 (#2-4 of 4): Bad bit shift operation (BAD_SHIFT)
  In expression 1UL << put, left shifting by more than 63 bits has undefined behavior.

Fixed whitespace in adler32_avx and adler32_ssse3.

Fixed unsigned integer overflow in adler32_avx and adler32_ssse3 when len is zero.

    adler32_avx.c:108:15: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'size_t' (aka 'unsigned long')

Use local match_len variable in deflate_slow.
Remove match_len from deflate internal_state.

Fixed wrong size being used in calloc in test_deflate_set_header.

    CID 293478 (#1 of 1): Wrong size argument (SIZEOF_MISMATCH)
    suspicious_sizeof: Passing argument 1UL to function calloc that returns a pointer of type zng_gz_header * is suspicious because a multiple of sizeof (zng_gz_header) /*80*/ is expected.

Use calloc to allocate memory and set to zero for memory sanitizer.

Support larger BUFLEN sizes by dynamically allocating buf.

Added ZLIB_REGISTER to disable register keyword for C++.

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

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.