Fixed deference null return value in gz_uncompress (CID 301524)

Remove unused F_OPEN preprocessor macro.

Remove gcov-7 install since warnings on gcov-8 no longer occur in Windows CI.

Remove chunkmemset_3 and chunkmemset_6 on ARM/AArch64 as they need 3 chunks...
* Don't unroll distances smaller than chunk size.

Add missing z_const to fuzzers.

Fixed match_byte casting warning in compare256_unaligned_64_static.

  compare258.c(158,36): warning C4244: 'return': conversion from 'uint64_t' to 'uint32_t', possible loss of data

Move gzclose() to gzlib.c. Modern linkers should be smart enough to link it in only if it is used.

Optionally include gzguts.h with WITH_GZFILEOP flag.

Added ubuntu mingw cross-compile CI instances to cmake GHA.

Fixed buf resource leak in gz_compress when USE_MMAP is defined. (CID 298965)

Allocate gzlib/gzread/gzwrite structs and in/out buffers using zng_alloc
instead of malloc, this also enforces data alignment.

Increase zng_alloc alignment to 64-bytes for non-MacOS platforms.

Improve malloc.h handling in zutil.h

Simplify zng_calloc and zng_cfree.
Make new static functions zng_alloc and zng_free available to other parts of the code.
Always request aligned allocations, even if UNALIGNED_OK is set.

Make it possible to disable UNALIGNED[64]_OK in cmake.
Enable testing with -O3 and unaligned reads disabled.

Remove some of the references to the unreleased zlib 1.2.12

Add CI testing of NOT_TWEAK_COMPILER

Improve coverage data accuracy

Enable WITH_GZFILEOP by default in nmake Makefiles.

Add support for ZLIB_COMPÅT in nmake Makefiles for ARM/AArch64.

Fixed description for INSTALL_UTILS in README.md.

Various small changes to README.md

Reorder "Features" block in README.md

Added AVX support to chunkset functions.

Update CI configs to account for gzfileops being on by default.

Enable gzfileops by default when compiling using Cmake or configure.

Remove FORCE_STATIC and FORCE_STORED

Rename variable to correctly indicate register usage,
YMM (256bit) instead of ZMM (512bit) register usage.

test/abicheck.md: document care and feeding of abicheck.sh

test/abicheck.sh: new script to verify abi compatibility with older versions

Verifies that zlib-ng's ABI has not changed since the reference commit
(indicated by variables ABI_URL and ABI_COMMIT in the script).
If --zlib-compat is given, the reference commit is zlib's 1.2.11;
otherwise, it's zlib-ng's 1d2504ddc489 (for now).

Ignores new symbols entirely, as they probably don't break backwards compatibility.

Ignores warnings listed in test/abi/ignore, currently, just those related to internal_state or z_stream*.

If --refresh is given, actually checks out the reference commit,
builds it, and stores its ABI description into an .abi file;
otherwise just uses the .abi file saved in git for that CHOST.
(--refresh_if is similar, but only does the above if the file
is not present.)

Fixes https://github.com/zlib-ng/zlib-ng/issues/699

Known issues:
- pkgcheck.yml skips -m32 abicheck failures loudly until #705 is fixed
- although abicheck.sh supports -m32 (if in both CFLAGS and LDFLAGS), it doesn't yet support -32 in CONFIGURE_ARGS.
- only includes a few abi definitions; the rest can be added in a followon commit (see --refresh).

Clean up if blocks in deflate.c

longest_match optimization

Fix numerous sign-conversion warnings in compare256/compare258 and
longest_match related code.

Minor comments/whitespace cleanup

Reorder s->block_open and s->reproducible.

Remove s->method since it is always set to the same value and never read.

Move and reduce size of s->pending_buf_size

Rename ZEXPORT and ZEXTERN for consistency.

Rename ZLIB_INTERNAL to Z_INTERNAL for consistency.

Rename ZLIB_REGISTER to Z_REGISTER for consistency.

Fix more conversion warnings related to s->bi_valid, stored_len and misc.

Fix some of the old and new conversion warnings in deflate*

Changes to deflate's internal_state struct members:
- Change window_size from unsigned long to unsigned int
- Change block_start from long to int
- Change high_water from unsigned long to unsigned int
- Reorder to promote cache locality in hot code and decrease holes.

On x86_64 this means the struct goes from:
        /* size: 6008, cachelines: 94, members: 57 */
        /* sum members: 5984, holes: 6, sum holes: 24 */
        /* last cacheline: 56 bytes */

To:
        /* size: 5984, cachelines: 94, members: 57 */
        /* sum members: 5972, holes: 3, sum holes: 8 */
        /* padding: 4 */
        /* last cacheline: 32 bytes */

Renamed trees_p.h to trees.h.

Rename table headers with tbl suffix.

Revert "zng_tr_tally_lit: disable -Wtype-limits"

This makes MSVC unhappy:

https://github.com/zlib-ng/zlib-ng/pull/726#issuecomment-681128124

D:\a\zlib-ng\zlib-ng\deflate_p.h(36,9): warning C4068: unknown pragma 'GCC' [D:\a\zlib-ng\zlib-ng\zlib.vcxproj]

This reverts commit 24c442c606d9121da64b25693c9bb1a898b3553f.

Use mainline codecov repository since codecov/codecov-python#169 has been merged.

send_bits_trace: placate -Wformat

value, which can be uint64_t, is printed using %llx, which, strictly
speaking, is not correct, and triggers -Wformat.

Since we don't really know what type value can have (send_bits_trace
is a macro), don't use <inttypes.h>, but rather cast it to long long.
Also cast length to int in order to prevent similar issues in the
future.

zng_tr_tally_lit: disable -Wtype-limits

Some gcc versions complain that parameter c is always less than
MAX_MATCH-MIN_MATCH, and therefore the assertion that checks for this
is useless, but in reality some day MIN_MATCH and MAX_MATCH can change.

So disable the warning around the assertion.

Add optional support for thread local storage. (#733)

Reintroduce support for ZLIB_CONST in compat mode. (#704)

* Reintroduce support for ZLIB_CONST in compat mode.

Increase hash table size from 15 to 16 bits.
This gives a good performance increase, and usually also improves compression.
Make separate define HASH_SLIDE for fallback version of UPDATE_HASH.

Replace hash_bits, hash_size and hash_mask with defines.

Precalculate match+offset pointer similar to fast-zlib.

Check for match length exceeding lookahead each time a new best match is found. This reduces some code complexity in GOTO_NEXT_CHAIN by removing the need for RETURN_BEST_LEN.

Use unaligned 32-bit and 64-bit compare based on best match length when searching for matches.
Move TRIGGER_LEVEL to match_tpl.h since it is only used in longest match.
Use early return inside match loops instead of cont variable.
Added back two variable check for platforms that don't supported unaligned access.

Update SystemZ DFLTCC README

Add DFLTCC to CI

QEMU does not support DFLTCC, but this would still check whether it
builds and gracefully falls back to software.

Fix DFLTCC not writing header bits when avail_out == 0

Fix DFLTCC ignoring flush modes when avail_in == 0

Fix DFLTCC not flushing EOBS when creating raw streams

Implement switching between DFLTCC and software

Fix switching compression levels on older SystemZ machines

When switching to a compression level that is in general supported by
the hardware accelerator, the code doesn't check whether acceleration is
available or enabled.

Use 15-bit window for quick strategy when compiling with DFLTCC

Hardware accelerator does not support 13-bit window.

Fix building DFLTCC with clang

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