refactored ressource allocation to avoid goto

examples/Makefile : changed dependency order

static library *.a must come after source files *.c on linux

Merge branch 'dev' into frameCompress

travisci : ensure "clean" betweeb 2 tests

modified gpptest recipe

Merge pull request #459 from Tyilo/clang-stdc++14

Ensure LZ4_DEPRECATED("...") is before LZ4LIB_API

refactored frameCompress.c example code

compression function returns a struct.
Also : nested structure ressources->computation
to make it easier to manage multiple exit points.

fixed read size, as noticed by @terrelln

ensure proper dependencies are built for /examples

also : use liblz4.a static lib to share compilation time

refactored frameCompress example

to better reflect LZ4F API usage.

Always prefer c++14 attributes if available

Ensure LZ4_DEPRECATED("...") is before LZ4LIB_API

When using clang++ with std c++14 or c++17 you would get the error "an attribute list cannot appear here" when including "lz4.h" as the visibility attribute is before the c++ attribute.
This ensures that the [[deprecated]] c++ attribute is before everything
else in the function declarations.

Merge pull request #455 from terrelln/hc-default

[lz4hc] level == 0 means default, not level 1

[lz4hc] level == 0 means default, not level 1

Merge pull request #453 from lz4/clockfix

fix #450 : use more restrictive conditions for clock_gettime()

Merge pull request #454 from sunpoet/dev

Change file format back to ASCII (from UTF-8)

Change file format back to ASCII (from UTF-8)

- Replace U+00A0 by space
- Fix build failure of archivers/py-borgbackup in FreeBSD

Reference: https://bugs.FreeBSD.org/225235

use more restrictive conditions for clock_gettime()

added checkTag

checkTag verifies that provided tag and library version match.
It's started automatically in circleCI when a new tag is created.

restored .travis

lz4frame : removed some intermediate stage from LZ4F_decompress()

ensure some strange jump cases are not possible
(they were already not possible, but static analyzer couldn't understand it).

Merge branch 'dev' into coverity_scan

minor : try to tell static analyzer that we don't care if fseek() fails

as already explained in comments.

ensure a ptr is non-null

with an assert()
to help static analyzer understanding this condition.

modified formulation for LZ4F_compressBound()

previous version used an intentional overflow,
which is defined since it uses unsigned type,
but static analyzer complain about it.

programs/datagen : attempt to please static analyzer

with an assert()
to reduce false positive

lz4io: fixed minor ressource leak

revert project name

minor name update

Merge branch 'master' into coverity_scan

Merge pull request #449 from lz4/dev

v1.8.1.2

update man page

nicer console message for `make clean`

updated manuals to v1.8.1

Merge pull request #448 from sunpoet/dev

Fix lz4 version

Fix lz4 version

updated LZ4F_decompress() documentation

Merge pull request #446 from lz4/dev

LZ4 v1.8.1

updated NEWS for v1.8.1

Merge pull request #442 from terrelln/441

[lz4io] Fix decompression file stat with --rm

Merge pull request #443 from terrelln/440

[lz4f] Skip memcpy() on empty dictionary

Merge pull request #444 from felixhandte/publish-static-functions

Add Option to Make lz4frame_static.h Functions Visible in Shared Objects

Add Option to Make lz4frame_static.h Functions Visible in Shared Objects

In some contexts, *cough*like at facebook*cough*, dynamic linking is used in
contexts which aren't truly dynamic. That is, the guarantee is maintained that
a program will only ever execute against the library version it was compiled
to interact with.

For those situations, introduce a compile-time flag that overrides hiding
these unstable APIs in shared objects.

Merge pull request #434 from lz4/pattern

conditional pattern analysis

Merge pull request #435 from lz4/destSize

lz4opt is now compatible with _destSize variant

[lz4f] Skip memcpy() on empty dictionary

[lz4io] Refuse to set file stat for non-regular files

[lz4io] Fix decompression file stat with --rm

Merge pull request #439 from eli-b/doc-nohup-behavior

Docs: describe behavior under nohup

Merge pull request #438 from eli-b/grammar-fix

Grammar: "to silent" -> "to silence"

Docs: describe behavior under nohup

Grammar: "to silent" -> "to silence"

object files in /tests

object files created from /tests are compiled with specific flags (`-g -DLZ4_DEBUG=1`)
which are not welcomed in release binary.
`lib/*.o` files created from /tests are now stored in /tests, to avoid unintentional mix.

lz4opt supports _destSize

no longer limited to level 9

/tests programs compiled with LZ4_DEBUG=1

to enable assert() within /lib

conditional pattern analysis

Pattern analysis (currently limited to long ranges of identical bytes)
is actually detrimental to performance
when `nbSearches` is low.

Reason is : `nbSearches` provides a built-in protection for these cases.
The problem with patterns is that they dramatically increase the number of candidates to visit.
But with a low nbSearches, the match finder just aborts early.

In such cases, pattern analysis adds some complexity without reducing total nb of candidates.
It actually increases compression ratio a little bit, by filtering only "good" candidates,
but at a measurable speed cost, so it's not a good trade-off.

This patch makes pattern analysis optional.
It's enabled for levels 8+ only.

Merge pull request #431 from lz4/newopt

new level 10

new level 10

lz4opt is only competitive vs lz4hc level 10.
Below that level, it doesn't match the speed / compression effectiveness of regular hc parser.

This patch propose to extend lz4opt to levels 10-12.
The new level 10 tend to compress a bit better and a bit faster than previous one (mileage vary depending on file)

The only downside is that `limitedDestSize` mode is now limited to max level 9 (vs 10),
since it's only compatible with regular HC parser.
(Note : I suspect it's possible to convert lz4opt to support it too, but haven't spent time into it).

Merge pull request #427 from lz4/registerLess

remove `register` keyword

Merge pull request #424 from MaxRis/msbuild-properties-#423

Msbuild properties #423

EnableWholeProgramOptimization and UseStaticCRT msbuild custom properties

remove `register` keyword

deprecated in newer C++ versions,
and dubious utility

Merge pull request #426 from aliceatlas/cli_chmod_fix

Fix cli sometimes calling UTIL_setFileStat on /dev/null

Fix bug which could sometimes result in the lz4 cli chmodding /dev/null if running as root

minor updates to examples

see https://github.com/lz4/lz4/commit/810e2ca27b3561e0f6bfa7a88e0fde6faf807064#commitcomment-25810887

API : changed a few variables' names for clarity

updated relevant doc.
This patch has no impact on ABI/API, nor on binary generation.

updated NEWS

Merge pull request #416 from lz4/newopt

Improve Optimal parser

added code comments

added constant TRAILING_LITERALS

which is more explicit than its value `3`.
reported by @terrelln

lz4opt: simplified match finder invocation to LZ4HC_FindLongerMatch()

added code comments

fixed LZ4HC_reverseCountPattern()

for multi-bytes patterns
(which is not useful for the time being)

removed the ip++ at the beginning of block

The first byte used to be skipped
to avoid a infinite self-comparison.
This is no longer necessary, since init() ensures that index starts at 64K.

The first byte is also useless to search when each block is independent,
but it's no longer the case when blocks are linked.

Removing the first-byte-skip saves
about 10 bytes / MB on files compressed with -BD4 (linked blocks 64Kb),
which feels correct as each MB has 16 blocks of 64KB.

Merge pull request #418 from lz4/fasterCount

faster LZ4_count()

removed legacy version of LZ4HC_InsertAndFindBestMatch()

ensure `pattern` is a 1-byte repetition

removed useless `(1 && ...)` condition

as reported by @terrelln

improved LZ4HC_reverseCountPattern() :

works for any repetitive pattern of length 1, 2 or 4 (but not 3!)
works for any endianess

fixed LZ4HC_countPattern()

- works with byte values other than `0`
- works for any repetitive pattern of length 1, 2 or 4 (but not 3!)
- works for little and big endian systems
- preserve speed of previous implementation

added LZ4_FORCEINLINE to counter gcc regression

as recommended by @terrelln

2-stages LZ4_count

separate first branch from the rest of the compare loop
to get dedicated prediction.

measured a 3-4% compression speed improvement.

build: minor : `make lz4` doesn't compile liblz4 anymore

since it's not needed.

Merge pull request #417 from sylvestre/dev

When building with a C++ compiler, remove the 'register' keyword to silent a warning

Only ignore with C++17

When building with a C++ compiler, remove the 'register' keyword to silent a warning

For example, with clang:

lz4.c:XXX:36: error: 'register' storage class specifier is deprecated and incompatible with C++17 [-Werror,-Wdeprecated-register]
static unsigned LZ4_NbCommonBytes (register reg_t val)
                                   ^~~~~~~~~

fixed minor static analyzer warning

dead assignment

minor comment edit

unified HC levels

LZ4_setCompressionLevel() can be users accross the whole range of HC levels
No more transition issue between Optimal and HC modes

moved ctx->end handling from parsers

responsibility better handled one layer above (LZ4HC_compress_generic())

removed ctx->searchNum

nbSearches now transmitted directly as function parameter
easier to track and debug

LZ4_compress_HC_continue_destSize() now compatible with optimal parser

levels 11+

removes matches[] table

saves stack space
clearer match finder interface (no more table to fill)

removed useless parameter from hash chain matchfinder

used to be present for compatibility with binary tree matchfinder

removed code and reference to binary tree match finder

reduced size of LZ4HC state

improved level 11 speed

optimized skip strategy for level 12

new level 11 uses 512 attempts

more generic skip formula

improving speed

small adaptations for intermediate level 11