Merge pull request #698 from lz4/tidy

several minor style changes recommended by clang-tidy

several minor style changes recommended by clang-tidy

Merge pull request #697 from ldv-alt/fixes/test-amalgamation

Fix test-amalgamation

Merge pull request #696 from lz4/mactest

Fix dynamic library compilation on Mac OS-X

test-amalgamation: split the rule

Change test-amalgamation to follow each-rule-makes-a-single-target idiom.

Fixes: a7e8d394 ("[amalgamation] add test")

test-amalgamation: fix compilation options

Use the same compilation options to compile lz4_all.c and other object files.

Fixes: a7e8d394 ("[amalgamation] add test")

test-amalgamation: use a single cat command

Use the list of prerequisites instead of listing those files manually,
this way they will never fall out of sync.

Also update the amalgamation example to use a single cat command.

Fixes: a7e8d394 ("[amalgamation] add test")
Fixes: b192c86b ("[amalgamation] lz4frame.c")

test-amalgamation: fix the list of prerequisites

Add $(LZ4DIR)/lz4frame.c to the list of prerequisites as the rule uses
that file.

Fixes: b192c86b ("[amalgamation] lz4frame.c")

Merge pull request #695 from lembacon/makefile-os

lib/Makefile: Fix detection of `Darwin`.

added library build test on Mac OS-X

lib/Makefile: Fix detection of `Darwin`.

Merge pull request #693 from JPeterMugaas/master

More build imrpvements

More build imrpvements
Moved a few other tests to Makefiles.inc.  Other things might need to go there.
Made a test for symlink appropriateness.  Windows can NOT handle them the same way Unix-like operating systems do (if at all).  This is mostly the same as the Visual C projects.
embed version info into .dll and .exes that are redistributed.

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

updated NEWS for v1.9.1

Merge pull request #691 from lz4/list

--list

fix: no leak when LZ4F_dctx creation fails

strange, because it previous implementation, it would `exit()`,
so it should not matter ...

Merge pull request #689 from JPeterMugaas/jpm-makefile

Jpm makefile - as described in https://github.com/lz4/lz4/issues/688

updated code comments

--list gives block type

--list can resume after a failed file

Make programs/Makefile use the includes.

refactored --list function

better handling of special conditions,
better scoping of variables.

Also : updated man page

Fix a test for mingw

Try to put some tests I made in ONE place.  I also moved a test for "install" in one place to try to isolate it.

fixed C90 compliance

re-structure code, have everything into a single section of lz4io.c

Merge pull request #690 from gabrielstedman/dev

Implement --list

Initial commits from diff I submitted earlier

FR #598 - Correctly initialize cfinfo & cast malloc res to (char*)

FR #598 - Make fileSize unsigned long long

FR #598 - Move LZ4F_compFileInfo_t def to lz4frame.h discard output_file
assert if in om_list mode

Merge https://github.com/lz4/lz4 into dev

FR #598 - Make LZ4IO_getCompressedFileInfo internal and reword func

Merge pull request #686 from lz4/clistdout

cli: warning for stdout as default output

ensure tests work when `stdout` is not the console

ensure this case is continuously tested on travis.

Update documentation on implicit output,
invite to not rely on implicit output in scripts.

tests/Makefile : created CMP variable

for potential redirection, if need be.

note : should probably converge all comparison operations onto CMP

FR #598 Improve initial design, test mallocs, support C90.

correctly use unisgned int for index

Add --list option to display compressed file information.

cli: display a warning whenever default output is stdout while input != stdin

This behavior has been preserved for compatibility with existing ecosystem.
But it's problematic, as some environment start `lz4` without identifying stdout as console by default,
leading to a change of behavior for a same line of script.

A more sensible policy would be to default to stdout only when input is stdin.

Soft change for the time being : keep the behavior, just print a warning message.
User should prefer `-c` to explicitly select `stdout`.

Also : updated tests in Makefile to explicitly select `stdout` with `-c`.

bumped version number

to v1.9.1

Merge pull request #684 from lz4/cxxlz4f

lz4frame: initializers compatibility with C++

lz4frame: initializers compatibility with C++

fix #679, reported by @degski

Merge pull request #683 from lz4/fuzasan

fixed read-after input in LZ4_decompress_safe()

Merge pull request #682 from lz4/visual

fix minor Visual warnings

fixed read-after input in LZ4_decompress_safe()

ensure list of names is large enough

fuzzer : reduced stack usage

fuzzer: reduce stack usage to please Visual static analyzer

address a few minor Visual warnings

and created target cxx17build

Merge pull request #678 from lz4/decFast

Fix out-of-bound read in LZ4_decompress_fast()

fix several minor static analyzer warnings

fuzzer: fixed scan-build leak warning

the program exit(),
so there is no need to track and dealloc every buffer.

fuzzer: fix explicit cast for C++

re-enable FORCE_INLINE

was disabled for tests

fix out-of-bound read within LZ4_decompress_fast()

and deprecate LZ4_decompress_fast(),
with deprecation warnings enabled by default.

Note that, as a consequence of the fix,
LZ4_decompress_fast is now __slower__ than LZ4_decompress_safe().
That's because, since it doesn't know the input buffer size,
it must progress more cautiously into the input buffer
to ensure to out-of-bound read.

fuzzer: added tests to detect LZ4_decompress_fast() out of bound read

Merge pull request #674 from nrgiii/dev

Fix

Merge branch 'fuzzerAix' into dev

Fix AIX errors/warnings

fixed version number for lz4hc

Merge pull request #673 from lz4/dev

minor update for v1.9.0

minor benchmark update

was using wrong memory timing

ensure consistent definition and usage of FREEMEM

as suggested by @sloutsky in #671

minor news update

for msys2

Merge pull request #670 from lz4/dev

v1.9.0

updated NEWS for v1.9.0

Merge pull request #669 from lz4/fixframe

Fix lz4frame inefficiency

simplified output_directive

fix comma for pedantic

Merge branch 'dev' into fixframe

added a fake initialization to please an analyzer

which is unable to understand that the variable is necessarily initialized
in spite of an assert just before.

fix fuzzer tests on dirty context

context is no longer dirty after a failed compressed block.
Actually, all indexes are fine.
It remains compatible with continued streaming, and reset*_fast().

unified limitedOutput_directive

between lz4.c and lz4hc.c .
was left in a strange state after the "amalgamation" patch.

Now only 3 directives remain,
same name across both implementations,
single definition place.

Might allow some light simplification due to reduced nb of states possible.

Merge branch 'dev' into fixframe

added more margin for canary in frametest

decompress*_fast() function do not generate deprecation warnings

they are classified as deprecated in the API documentation (lz4.h)
but do not yet trigger a warning,
to give time to existing applications to move away.

Also, the _fast() variants are still ~5% faster than the _safe() ones
after Dave's patch.

fixed lz4frame with linked blocks

when one block was not compressible,
it would tag the context as `dirty`,
resulting in compression automatically bailing out of all future blocks,
making the rest of the frame uncompressible.

frametest: removed uninitialized warning

was a false positive, but better remove it anyway

assert() the failed test

ensure canary remains within buffer limits

fixed frametest

One test could write a canary value out of bound
in exceptional conditions involving multiple flushes
triggered by -s3421 -t462948.

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

fixed incorrect assertion condition

output can use the full length of output buffer

Merge pull request #668 from lz4/visual

Fix some Visual conversion warnings

updated benchmark table

and even more visual conversion warnings

Merge pull request #667 from lz4/pipeMultiple

Fix combination -m -c

some more Visual conversion warning

fixed minor Visual conversion warnings

fix minor visual warning

yet some overly cautious overflow risk flag,
while it's actually impossible, due to previous test just one line above.
Changing the cast position, just to be please the thing.

fixed minor Visual warnings

since Visual 2017,
worries about potential overflow, which are actually impossible.
Replaced (c * a) by (c ? a : 0).
Will likely replaced a * by a cmov.
Probably harmless for performance.

added tests for -d -m -c

seems to work properly

fixed lz4 -m -c

can compress multiple files into stdout

Merge pull request #666 from lz4/blockIndep

added command -BI for "Independent Blocks"

added command -BI for "Independent Blocks"

This is the reverse of `-BD`, and the current default.

This command can be useful to reverse a previous `-BD` command.
It may in the future be more important
if `lz4` switches to generating dependent blocks by default.

updated man page

with --favor-decSpeed

updated doc to underline difference between block and frame

as this is a very frequent source of confusion for new users.

Merge pull request #665 from lz4/dict

Dictionary compression

Merge pull request #664 from lz4/maxdist

introduce LZ4_DISTANCE_MAX build macro

improved documentation for LZ4 dictionary compression