Remove forward declarations for stub functions and have one forward declaration for functable instead.

Style cleanup.
* Remove space between function name and opening parenthesis
* Remove space closing parenthesis of cast and variable name

Fix signature of adler32(), adler32_combine(), adler32_combine64() and adler32_z() in compat mode.
* See #700

Change ctest to test all the files in test/data.
Improved minigzip test configurations in ctest.
Added corpora repository to test/data to test for each run.

Allow for CMake to run tests in parallel.
Seperate minigzip and switchlevels set of tests into test-compress.cmake.
Create run-and-compare.cmake for tests that need to run a command and compare output.

CMakeLists.txt, configure: match source order; .so's now identical on more platforms.

Also:
- use same name for .so on CMake and configure in --zlib-compat case
- configure: merge i686 and x86_64 sections, since they were essentially identical
- pkgcheck.sh: verify resulting shared libraries are bit-for-bit identical
- pkgcheck.sh: use diffoscope if present to show differences in shared libraries

Don't install diffoscope in ci, since that slows down successful
runs; we can add it later if failures become common.

Fixes #638 (at least for the systems tested)

configure: move config.sub to tools subdir

cmake: mac: don't use @rpath (since configure doesn't); for #638

configure: canonicalize CHOST.

Fixes https://github.com/zlib-ng/zlib-ng/issues/666
for all the CHOST values currently used in
.github/workflows/*.yml.

Also make ci's shellcheck happy by avoiding backticks in the changed lines.

configure: don't link shared libraries explicitly with -lc

The -lc was introduced by d6231142d2b883a8c3b253fa34992b5cdb4ac2fe :
which said in part
"Changes in 1.2.3.2 (3 September 2006)
...
- Rig configure --shared to build both shared and static [Teredesai, Truta]"

It's not clear why it was needed.  On powerpc, linking -lc explicitly
causes __moddi3 and __stack_chk_fail_local to swap location in the
binary.
If we want configure and cmake to generate identical shared libraries,
either we need to link -lc explicitly in both configure and cmake,
or in neither.   Occam's Razor suggests omitting it from configure
rather than adding it to cmake.

Return proper exit code after flushing end block in deflate_quick when no available output.

Make static libraries produced by cmake and configure identical.

Also make compat mode .pc files identical.

- cmake: don't do runtime feature checks when doing native builds.
- test/pkgcheck.sh: also compare .a, check compat mode, and handle mac better
- CMakeLists.txt: get Version right in compat mode .pc file
- .github/workflows/pkgcheck.sh: test on more systems, including mac

Fixes #681

Fixed warning about implicit declaration of function fileno().

Removed legacy typedef caddr_t from gz_compress_mmap.

Added CI instance and coverage for USE_MMAP define in minigzip.

Default to optimization level 2 in cmake for release mode. Can be overwritten by setting the optimization level with CMAKE_C_FLAGS. MSVC does not have optimization level 3.

Added missing switch for configure script --native to readme.

Added SSE4.2 inline assembly check for configure script.

Fixed chunksize not being initialized when using inflateBack.

Fix long string warning on adler32_test

Since the introduction of test/adler32_test.c there has been a new
warning about too long strings:

 /home/runner/work/zlib-ng/zlib-ng/test/adler32_test.c:178:5: warning: string
length ‘5552’ is greater than the length ‘4095’ ISO C99 compilers are required
to support [-Woverlength-strings]
     "byRKqAu3J", 5552, 0x8b81718f},
     ^~~~~~~~~~~
/home/runner/work/zlib-ng/zlib-ng/test/adler32_test.c:321:5: warning: string
length ‘5552’ is greater than the length ‘4095’ ISO C99 compilers are required
to support [-Woverlength-strings]
     "byRKqAu3J", 5552, 0x7dc51be2},
     ^~~~~~~~~~~

Declaring the long string as a static byte array fixes the issue.

Merge pull request #683 from nmoinvaz/fixes/config-cwd-lib-path

Remove current directory library include path in configure not in cmake

Fixed const warning in deflate prime tests.

     zlib-ng.h:1726:10: note: expected ‘const uint8_t * {aka const unsigned char *}’ but argument is of type ‘const char *’

Fixed signed integer comparison warning in deflatePrime bits check.

    deflate.c:589:49: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]

Fixed ubsan error in deflatePrime when bits is 32.

    deflate.c:602:15: runtime error: shift exponent 32 is too large for 32-bit type 'int32_t' (aka 'int')
    SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /src/zlib-ng/deflate.c:602:15

Add options to install minigzip and minideflate.

Added GitHub Actions workflow to create releases when commits are tagged.

configure: fix typo (.o in shared library)

Linking .o into a shared library could be bad.
In this case it seems to have been harmless, but it should be fixed anyway.

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;