Fix dfltcc_deflate.c build

bi_windup and send_bits were moved to tree_emit.h, so include it.

Fix DFLTCC detection

On some Z machines ARCH is determined to be s390, not s390x, which
prevents DFLTCC support from being built. In general, it is safe to
build zlib-ng with DFLTCC support on any SystemZ machine, because its
usage is guarded by STFLE.

Add toolchain files for cross-compiling MinGW on Linux.

Remove return value from insert_string, since it is always ignored and
quick_insert_string is being used instead.

Now that the check is out of the loop, it is also safe to remove it
and unconditionally return head.

Minor optimization of insert_string template.

Store hash_mask in local variable for insert_string loop.

Move zero check for insert_string count to fill_window since it is the only place where count is ever passed as zero.

Fix if statements in cmake failing because variables are unquoted.
Exposed by #729, but does not fix the real problems there.

Fix testsuite warnings on Windows, using PRIu64

zlib-ng already counts on inttypes.h and stdint.h, so use those to avoid
printf-related warnings by casting integer fields whose size may vary to
uint64_t and printing them that way.

zlib.h: Use unsigned long for totals, for compatibility with zlib on 64-bit Windows

Stock zlib uses unsigned long for total_in and total_out. On 64-bit
Windows, that isn't the same as size_t.

Harmonize the CPU architecture preprocessor definitions.

Prevent unaligned double word access on ARMv7 in put_uint64

By implementing a (UNALIGNED_OK && !UNALIGNED64_OK) codepath.

Prevent unaligned double word accesses on ARMv7 in chunkset_c

Requires some mild trickery to walk the edge between readability and
doing what it needs to.

Correct some typos in README.md

Jus some minor typos :)

Honor CFLAGS when linking

It's the GCC driver, honor CFLAGS to make stuff consistent.
(Allows one to actually get an LTO build w/ debug info, for
instance).

Remove NO_DIVIDE from adler32.

Optimize inflate_fast for a 0.8% speedup

When inflate_fast checks for extra length bits, it first checks if the
number of extra length bits (in op) is non-zero. However, if the number
of extra length bits is 0, the `bits < op` check will be false, BITS(op)
will be 0, and DROPBITS(op) will do nothing. So, drop the conditional,
for a speedup of about 0.8%.

This makes the handling of extra length bits match the handling of extra
dist bits, which already lacks the extra conditional.

Don't let the compiler switch to 64bit vectors behind our back.

On ARMv7, they have more stringent alignment requirements,
which aren't accounted for.

Fix inconsistent wildcards in CHOST canonicalization script

Should prevent mangling actually sane four-part CHOSTs,
while not regressing #666

Update clang static analyzer to version 12

Style cleanup.

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

Fix signature of crc32_combine(), crc32_combine64() and crc32_z() in compat mode.

Cast wsize as early as possible in slide_hash for AVX2 and SSE2.

Fixed possible loss of data warning in h2b in infcover on Windows.

    infcover.c(247,35): warning C4267: '=': conversion from 'size_t' to 'unsigned int', possible loss of data

Fixed many possible loss of data warnings where insert_string and quick_insert_string function used on Windows.

Fixed possible loss of data warning in LONGEST_MATCH.

    match_tpl.h(69,64): warning C4244: '=': conversion from 'unsigned int' to 'Pos', possible loss of data

Fixed conversion from unsigned int to short warning in slide_hash_sse2 and slide_hash_avx2.

    slide_sse.c(20,51): warning C4244: 'function': conversion from 'unsigned int' to 'short', possible loss of data
    slide_avx.c(21,54): warning C4244: 'function': conversion from 'unsigned int' to 'short', possible loss of data

Fixed warning about indexing literal string in GCC macOS

    infcover.c:378:56: warning: adding 'int' to a string does not append to the string [-Wstring-plus-int]
    infcover.c:378:56: note: use array indexing to silence this warning

Removed unnecessary header guards in adler32 source files.

Fixed warning about no newline at the end of adler32_ssse3.c.

    adler32_ssse3.c:123:7: warning: no newline at end of file [-Wnewline-eof]

Remove ARM crc instruction set from NEONFLAG. The crc instruction set is not used for neon source files. It may have been necessary back when the NEONFLAG was applied to all source files in CMake. Configure script does not apply crc instruction set when setting neon flags.

Fixed extra symbols added to ABI when zlib-compat specified.

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.