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)
                                   ^~~~~~~~~

Merge pull request #415 from lz4/fasterDecodingXp

Faster decoding xp

minor change, to help store forwarding

in a marginal case (offset==4)

extended shortcut match length to 18

Merge pull request #408 from terrelln/time

[bench] Use higher resolution timer on POSIX

minor : coding style : use ML_MASK constant

added comments, as suggested by @terrelln

more complete shortcut - passes tests

Merge pull request #412 from mikir/symbolsVisibility

Separated visibility from LZ4LIB_API macro.

Separated visibility from LZ4LIB_API macro.

small modification of lz4 decoder to shortcut common case (short branch).

[bench] Use higher resolution timer on POSIX

The timer used was only accurate up to 0.01 seconds. This timer is accurate up to 1 ns.
It is a monotonic timer that measures the real time difference, not on CPU time.

Copied the benchmark code from https://github.com/facebook/zstd/commit/6ab4d5e9041aba962a810ffee191f95897c6208e

lz4cli : minor rewrite of lz4c legacy commands

for clarity

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

lz4cli : removed extension artefacts

It used to be useful for an old Windows variant which is no longer maintained.

Merge pull request #407 from odaira/useO2ppc64le

Use O2 for the decompression functions on ppc64le with gcc

Use the optimization level of O2 for the decompression functions on ppc64le with gcc, to avoid harmful unrolling and SIMDization with O3

Merge pull request #403 from felixhandte/lz4-cli-dict-support-tests

Support Dictionaries on the Command Line

Read the Dictionary into a Circular Buffer

Add some tests verifying command line dictionary functionality

Add Dictionary Support to the Command Line Tool

fixed decoding block checksum in lz4frame

fix #404

static analyzer `cppcheck` complains about a shift-by-32 on a 32 bits value,
which is an undefined behavior.
However, the flagged code path is never triggered in 32-bits mode,
(actually, it's not even generated if DCE kicks in),
the shift-by-32 is necessarily performed on a 64-bits value.

While it doesn't change anything regarding lz4 code generation, for both 32 and 64 bits mode,
(can be checked by md5sum on the generated binary),
the shift has been rewritten in a way which should please this static analyzer,
since it now pretends to shift by 16 on 32-bits cpu (note : it doesn't matter since the code will not even be generated in this case).

Note : this is a blind fix, the new code has not been tested with cppcheck, because cppcheck only works on Windows.
Other static analyzer, such as scan-build, do not trigger this false positive.

minor lz4frame code refactor

try to improve code readability.
minor optimization on condition to preserve history.

Merge pull request #402 from felixhandte/fix-dict-segfault

Fix dict segfault

Fix Segfault When Copying Dict

dctx must have been initialized before we can copy the dictionary in.

minor improvements to examples

cosmetic : better display
added optional variable MOREFLAGS

made clang warnings fail (-Werror)

in order to catch them in CI tests

fixed a bunch of -Wcomma warnings

reported by @rvandermeulen (#398)

added -Wcomma to travisCI clang test

fix #397 : decompression failed when using a combination of extDict + low memory address

Reported and fixed by @jscheid

Note : we are missing a test case to include it in the CI

bench : made decompression speed evaluation same time as compression

minor : slightly modified an example do avoid disabling a gcc warning through #pragma

minor Makefile fixes

updated lib/README

clarifications, improved wording

complementary information for #394

clarified documentation of streaming decompression functions

(synchronous bufferless mode)
answering questions by @jtbandes (#394)

minor typo fix

fixed FS-independent file order in /lib

identified by @bmwiedemann

build: source files sorted in a FS independent manner

to be more compatible with reproducible builds.
patch inspired by @bmwiedemann

updated NEWS with fixes

Merge pull request #386 from lz4/parallelMake

`make test` compatible with parallel execution (`-j#`)

removed fasttest from circleCI

removed test-fasttest

Merge pull request #393 from terrelln/deprecate2

Don't use C++11 deprecation attribute with clang

fixed make recurrence from /tests

removed fasttest

minor : added header license

Merge branch 'dev' into parallelMake

Don't use C++11 deprecation attribute with clang

Clang doesn't accept a C++11 attribute following `LZ4LIB_API`.
Use the GNU attribute instead.

Merge pull request #392 from m-hennecke/fix-arm-openbsd

Fix: Add return statement to main function

Merge pull request #391 from tcpan/dev

FIX: added prefix to FORCE_INLINE to prevent redefinition error durin…

Merge pull request #390 from lz4/installVars

fixed man directory installation (#387)

FIX: added prefix to FORCE_INLINE to prevent redefinition error during compilation when used with other libraries that define FORCE_INLINE

Fix: Add return statement to main function

fixed man directory installation (#337)

also : lz4c is now a symlink to lz4

make test is compatible with parallel execution (-j#)

each test section runs with its own set of files

fix : asan error in dctx, due to increased maximum frame header size, reported by Craig Young

Merge pull request #382 from lz4/installVars

better respect GNU standard Makefile conventions

updated lz4frame manual

Merge branch 'dev' into installVars and fixed conflicts

Merge pull request #383 from lz4/blockChecksum

Block checksum

Merge pull request #380 from lz4/dictID

Frame Dictionary API

lz4c legacy commands are now enabled at runtime based on link/binary name "lz4c"

instead of selected at compilation time depending on a macro.
This design makes it possible to have a single binary which supports both modes.

An advantageous side effect is that when doing `make; make install`
no additional binary is created during `make install`
(it used to create `lz4c`, because `make` would only build `lz4`)

fix minor markdown display issues

better respect GNU standard Makefile conventions

supports lowercase directory variables
add an "Installation" section in README.md
added an INSTALL file

cli : restored command -BX to enable block checksum (#322)

restored block checksum capability at lz4frame API level

Merge pull request #378 from deymo/dev

Allow to predefine FORCE_INLINE macro.

implemented lz4frame decompression API

support dictionary compression with independent blocks

fixed gcc prototype warning

dictionary compression correctly uses compression level

Not obvious : copying the state was copying cdict's compression level

fixed C++ conversion warnings

fixed uninitialization error in lz4frame

implemented dictionary compression in lz4frame

note : only compression API is implemented and tested
still to do : decompression API

fixed frameCompress example

added dictID inside LZ4F_frameInfo_t

Compressor can set dictID on LZ4F_compressBegin()
Decompressor can retrieve it using LZ4F_getFrameInfo()

updated Frame specification

Restored DictID field in Frame header
Bumped specification version to v1.6.0

Allow to predefine FORCE_INLINE macro.

FORCE_INLINE macro is defined based on the compiler used. When using
gcc, it will include "__attribute__((always_inline))" forcing gcc to
always inline all the functions marked as FORCE_INLINE. However, this
can cause a performance degradation of about 15%.

This patch allows to set the FORCE_INLINE macro from the compiler
command line to either "static" or "static inline" giving allowing it
to inline functions as needed when performing optimizations.

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

Merge pull request #376 from ido/patch-1

Fix typos preventing installation of static lib.

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

updated man page

Fix typos preventing installation of static lib.

Merge pull request #374 from ferdnyc/patch-2

Generate updated man page from Markdown source

Generate updated man page from Markdown source

Merge pull request #373 from ferdnyc/patch-2

Fix formatting of concatenation example

Fix formatting of concatenation example

The "Concatenation of .lz4 files" section contains example commands
that are run together on one line, making them invalid. Wrap them
in a code block and clean up surrounding formatting.

fix #369

The bug would make the bt search read one byte in an invalid memory region,
and make a branch decision based on its value.
Impact was small (missed compression opportunity).
It only happens in -BD mode, with extDict-prefix overlapping matches.

The bt match search is supposed to work also in extDict mode.
In which case, the match ptr can point into Dict.
When the match was overlapping Dict<->Prefix,
match[matchLength] would end up outside of Dict, in an invalid memory area.
The correction ensures that in such a case,
match[matchLength] ends up at intended location, inside prefix.

Merge pull request #371 from jku/tests-LIBDIR

tests/Makefile: don't use LIBDIR as variable

tests/Makefile: don't use LIBDIR as variable

LIBDIR may be overriden with a environment variable: In this case make
clean breaks in tests/. Use another variable name.

Merge pull request #367 from Chocobo1/fallthrough

Fix gcc7 Wimplicit-fallthrough warnings

Merge pull request #368 from Chocobo1/readme

Update README.md

Update README.md

Update appveyor badge URL
Remove empty line

Fix gcc7 Wimplicit-fallthrough warnings

For the default Wimplicit-fallthrough=3 level,
the comment should start with "fall*"

made level 10 a bit faster

at the expense of a little bit of compression ratio.

Now speed is intermediate on calgary corpus :
25 - 12 - 8 - 3

-g compilation flag not by default for lz4 cli

clarified lz4frame api comment (#350)

report where decompression ends (#313)

suggested by @ehem
note : only works for files < 2 GB