minor refactor

mostly on the compression code path

minor refactor of lz4io

for readability.
Mostly around --list capability

Merge pull request #950 from lz4/appvtest

appveyor artifact generation

appveyor artifact generation

fix presence of the dynamic library in produced package

Merge pull request #949 from lz4/cmakedll

fix cmake build of shared dll with visual

Merge pull request #946 from lz4/customMem

proposal for LZ4_USER_MEMORY_FUNCTIONS

fix cmake build of shared dll with visual

inspired from #891

updated NEWS for v1.9.3

minor : lz4frame include stdlib only if needed

proper fullbench's LZ4_calloc()

changed LZ4_calloc() to a 2-arguments signature

to remain similar to stdlib's calloc().

Updated test to use c++ compiler for stricter signature check.

fix fullbench's LZ4_free()

fix travis test

Merge branch 'dev' into customMem

first proposal for LZ4_USER_MEMORY_FUNCTIONS

makes it possible to replace at link time
malloc, calloc and free
by user-provided functions
which must be named LZ4_malloc(), LZ4_calloc() and LZ4_free().

answer #937

Merge pull request #944 from lz4/fix874

fix #874

Merge pull request #943 from lz4/fullbench_dec

Added decompression tests in fullbench

Merge pull request #945 from lz4/fix926

fixed remaining ubsan warnings

attempt at silencing cppcheck

fixed remaining ubsan warnings

fix #874

coverity reported a warning regarding a memcpy() overwrite.
This is a false positive (the memory area is large enough),
but it's true that it's not trivial to determine (encompassing struct),
and it's proper anyway to only memcpy() just the right amount of data.

fullbench: added LZ4F_decompress_noHint()

added test for LZ4_decompress_safe_withPrefix64k()

update obsolete section documentation

and update manuals.
fix #712

Merge pull request #942 from lz4/fix926

fix #926

fix minor UBs

- check alignment before casting a pointer
- saveDict : don't memmove() on NULL dst

added similar tests for LZ4_saveDict()

fast (non-HC) variant

fix #926

fix incorrect behavior of LZ4_saveDictHC()
when invoked right after initialization.

Merge pull request #941 from lz4/revertinline

Revert "Replace "static" to "LZ4_FORCE_INLINE" for small functions"

Merge pull request #940 from lz4/fix935

fix #935

Revert "Replace "static" to "LZ4_FORCE_INLINE" for small functions"

This reverts commit 0e3933edd435c54cc2e21e38f5d4ba7bf644a24e.

fix #935

minor: identical declaration and prototypes of `LZ4HC_compress_optimal()`

also :
very minor optimization of `LZ4_memcpy_using_offset()`

Merge pull request #939 from lz4/fix927

LZ4F_decompress requires a valid dctx state

LZ4F_decompress requires a valid dctx state

This is now explicitly documented and asserted.
fix #927

Merge pull request #936 from lz4/alignTest

More alignment tests

static state size

for better inter-version compatibility

re-enable alignment test on all targets

unified internal state declaration

align on `void*` instead : there is no `long long` inside the structure

document LZ4_ALIGN_TEST

unified alignment test

across lz4.c and lz4hc.c

Merge pull request #938 from terrelln/fuzzer-fix

[ossfuzz] Fix parallel builds

[ossfuzz] Fix parallel builds

With `make -j` multiple builds of `standaloneengine.o` happened in
parallel. Fix this by detecting `standaloneengine.o` and moving it to a
depedency.

fix minor explicit cast

Merge pull request #930 from remittor-pr/fix_msvc

Fix: The "inline" specifier do not use for LZ4_wildCopy8 and LZ4_wildCopy32

Replace "static" to "LZ4_FORCE_INLINE" for small functions

The "static" specifier does not guarantee that the function will be inlined.

Replace define LZ4_FORCE_O2_INLINE_GCC_PPC64LE to LZ4_FORCE_INLINE

There is no reason to separate these two definitions!

Merge pull request #928 from remittor-pr/human_redable

Made function LZ4HC_encodeSequence a human readable

Fix: The "inline" specifier do not use for LZ4_wildCopy8 and LZ4_wildCopy32

This problem was reproduced on MSVC 2015 (32-bit). Both functions were called using the operator "call".

[lz4hc] Made function LZ4HC_encodeSequence a human readable

reduce new test stack usage

use heap instead

Appveyor: added compilation and runtime fuzzer tests

to all Windows compiler targets

preserver alignment test on Visual Studio x64

this it works fine in this environment
(only x86 is suspicious)

fixed x32 test on Travis

fix cppcheck unused variable warning

added LZ4_streamHC_t init test

which includes an alignment test

Merge pull request #925 from lz4/test_null

add LZ4F_decompress() tests with (NULL,0) input and output

define sentinelTest

to help scan-build detect the condition

make it possible to select SCANBUILD binary on command line

add LZ4F_decompress() tests with (NULL,0) input and output

fix one (rare & complex) issue discovered by this test

Merge pull request #924 from lz4/safixes

fix minor static analyzer warnings

make scan-build accept assert()

fix bad init scenario

added memcpy() related SA warning fixes

memcpy() on NULL is UB, even if length is 0.

Merge branch 'safixes' of github.com:lz4/lz4 into safixes

fix conversion warning

Merge branch 'dev' into safixes

fix minor static analyzer warnings

detected by scan-build, cppcheck and advanved compilation flags
fix #786

fix minor static analyzer warnings

detected by scan-build and cppcheck
fix #786

bump version number

to v1.9.3

Merge pull request #923 from lz4/fix784

fix efficiency of LZ4_compress_HC_destSize()

improved last literals run on LZ4_compress_destSize

applying new more accurate formula from LZ4_compress_HC_destSize()

also : fix some minor display issue in tests/frametest

ensure last match not too close to end

must respect MFLIMIT distance from oend

fix incorrect counting

after truncation of last sequence

Merge pull request #921 from lz4/doubleNull

fix compressing into NULL

Merge pull request #922 from XVilka/tinycc-support

Fix compilation with TinyCC

fix efficiency of LZ4_compress_HC_destSize()

LZ4_compress_HC_destSize() had a tendency
to discard its last match when this match overflowed specified dstBuffer limit.
The impact is generally moderate,
but occasionally huge,
typically when this last match is very large
(such as compressing a bunch of zeroes).

Issue #784 fixed for both Chain and Opt implementations.

Added a unit test suggested by @remittor checking this topic.

Fix compilation with TinyCC

fix compressing into NULL

fails properly
bug discovered by oss-fuzz

Merge pull request #919 from attilaolah/cli

Add CMake option LZ4_BUILD_CLI

Add CMake option LZ4_BUILD_CLI.

Defaults to ON so nothing changes, but it can be set to OFF so that the
CLI is not built at all, only the library.

comment bug on older versions of ZSTD_compress_destSize()

following investigation in #859

Merge pull request #918 from lz4/usan_null

Properly fix #847

fixed ubsan tests with dynamic library

ubsan: make pointer overflow recoverable

the way `base` is used must allow 2-complement address overflow.
`base` is effectively a virtual pointer, never dereferenced directly.
What matters is `base + index`.
This is the address that will be dereference and must be valid (it is properly validated).

fixed lz4frame with blocks of size 1

properly track history

Merge branch 'usan_null' of github.com:Cyan4973/lz4 into usan_null

Merge branch 'usan_null' of github.com:lz4/lz4 into usan_null

fix ubsan test

which now fails with a clear error as intended
(not just print a status and move on).
should be reproduced on travisCI

added the actual code change

Merge pull request #913 from lz4/usan_null

fix #847

fix #847

support NULL input
without triggering undefined sanitizer

added test triggering NULL arithmetic with usan

described in #847

Merge pull request #910 from lz4/extraInput

Fix issue #783

fixed minor pre-existing printf formatting

different signedness

fixed strict c90 support

fix #783

LZ4_decompress_safe_partial()
now also supports a scenario where
nb_bytes_to_generate is <= block_decompressed_size
And
nb_bytes_to_read is >= block_compressed_size.

Previously, the only supported scenario was
nb_bytes_to_read == block_compress_size.

Pay attention that,
if nb_bytes_to_read is > block_compressed_size,
then, necessarily, it requires that
nb_bytes_to_generate is <= block_decompress_size.
If both are larger, it will generate corrupted data.

Merge branch 'dev' into extraInput

Merge pull request #909 from lz4/fasterCount

Improved software byte count

Merge branch 'dev' into fasterCount

added documentation about LZ4_FORCE_SW_BITCOUNT

Also : added memory-frugal software byte count for big endian 64-bit cpus.
Disabled by default.