fixed minor gcc warning

gcc-8 on Linux doesn't like usage of strncat :
`warning: ‘strncat’ output truncated before terminating nul copying as many bytes from a string as its length`.

Not sure what was wrong, it might be a false positive,
but the logic is simple enough to replaced by a simple `memcpy()`,
thus avoiding the shenanigans of null-terminated strings.

Merge pull request #1363 from facebook/backtrace

backtrace support compiled with more conditions

use #if BACKTRACE_ENABLE directly

as suggested by @terrelln

Merge branch 'backtrace' of github.com:facebook/zstd into backtrace

fileio.c: Disable backtrace when built with address sanitizer

Covers clang and gcc's sanitizer flags.
Can still be overridden through CFLAGS on commandline.

fixed explicit BACKTRACE order

and automatic linux backtrace detection :
__GLIBC__ must be tested after #include <features.h>

fixed BACKTRACE_ENABLE macro test

Merge pull request #1361 from facebook/streamdoc

Clarify streaming api doc

Merge pull request #1362 from felixhandte/fix-btlazy-skipping

`ZSTD_btlazy2`: Avoid Erroneously Trampling on Match with Worse Dictionary Match

backtrace support compiled with more conditions

following #1356,
only enable backtrace compilation on linux+glibc.

Also, disable backtrace by default from "release" compilation,
so that less platforms get impacted by the new requirements.
Can be manually enabled/disabled using BACKTRACE=1/0.

Merge pull request #1356 from maxice8/fix-musl

don't assume __linux__ means __GLIBC__ on BACKTRACES_ENABLE

Avoid Searching Dictionary in ZSTD_btlazy2 When an Optimal Match is Found

Bailing here is important to avoid reading past the end of the input buffer.

Clean Up Debug Log Statements

Remove Unused Variable

clarify streaming api doc

as suggested by @indygreg in #1360

don't assume __linux__ means __GLIBC__ on BACKTRACES_ENABLE

check for __GLIBC__ instead of __linux__ as musl libc doesn't provide
execinfo.h

fixes compilation on Alpine Linux and Void Linux musl arches.

fixed fullbench-lib target

Merge pull request #1351 from facebook/haikubuild

 portability macro flags updates, for Haiku

Merge pull request #1353 from facebook/paramgrill

fixed paramgrill wrong assert() conditions

Merge branch 'dev' of github.com:facebook/zstd into dev

added graph for ZSTD_compress_usingCDict() in v1.3.6

Merge pull request #1352 from facebook/visual

fixed VS2017Community build script

fixed paramgrill wrong assert() conditions

and slightly refactored affected function.

Honestly, the formula calculating variance should get a second reviewing round,
it's not clear if it's correct.

fixed VS2017Community build script

reported by @epicabsol

finalized minor warnings on Haiku

changed ZSTD_NANOSLEEP_SUPPORT definition

to please `-Wexpansion-to-defined`

portability macro flags updates, for Haiku

some non-trivial changes to platform.h and util.h,
initially related to compilation for Haiku,
but I used this opportunity to make them cleaner
and add some documentation.

Noticed several tests that could be improved
(too harsh conditions, useless exception, etc.)
but I did not dare modifying too many tests just before release.

Merge pull request #1330 from korli/haiku

Enable building zstd on Haiku.

fixed wrong assert() position

could fire on invalid input.
blocking for afl tests.

Merge pull request #1350 from facebook/fixmsan

fixed msan warning

Enable building zstd on Haiku.

fixed msan warning

on btlazy2 strategy with dictAttach

Merge pull request #1348 from facebook/donotdelete

Fix #1082

fixed static analyzer warnings

note : for some reason,
scan-build version on my laptop found problems within fastcover.c
that scan-build on travisCI does not flag.

They are, as usual, false positive :
the analyzer does not understand that a table (`offset`) is correctly filled before usage.

restored backtrace on failure

for Linux and Mac OS-X.

Note : the backtraces fires up through a trap
before the sanitizer get a chance to report.
There are situations where the sanitizer report is actually preferable.

It might be good to consider a kind of build macro
which can disable backtrace
when sanitizer is enabled.

updated NEWS in anticipation for v1.3.6

Merge pull request #1317 from felixhandte/split-logs

Independent Dictionary and Working Context Table Logs

./zstd -f do no longer overwrite destination file

if source file does not exist (#1082)

zstd -d -f do no longer erase destination file

when source file does not exist (#1082)

Revert Ability to Set HashLog and ChainLog on Context When Dict is Attached

This capability is not needed / used in the current unit of work. I'll
re-introduce it later, when we start allowing users to override the deduced
working context logs.

regroup name creation logic into its own function

for a cleaner main file decompression loop

Merge pull request #1347 from terrelln/werror

Add -Werror to *build rules

Merge pull request #1346 from terrelln/small-block-fse

[zstd] Fix resetting FSE tables

Restore Passing CParams to `ZSTD_insertAndFindFirstIndex_internal`

Use Working Ctx Logs when not in DMS Mode

We pre-hash the ptr for the dict match state sometimes. When that actually
happens, a hashlog of 0 can produce undefined behavior (right shift a long
long by 64). Only applies to unoptimized compilations, since when
optimizations are applied, those hash operations are dropped when we're not
actually in dms mode.

When Attaching Dictionary, Size Working Tables Based on Input Size Only

Remove Log Overriding for the Time Being

Remove Strategy-Dependency in Making Attachment Decision

Support Split Logs in ZSTD_btopt..ZSTD_btultra

Bump Split Log Support to ZSTD_btultra

Support Split Logs in ZSTD_greedy..ZSTD_btlazy2

Bump Split Log Support to ZSTD_btlazy2

Support Split Logs in ZSTD_dfast

Bump Split Log Support to ZSTD_dfast

Support a Separate Hash Log in ZSTD_fast

Stop Passing in `hashLog` and `stepSize` to `ZSTD_compressBlock_fast_generic`

Temporary Commit to Retain Requested Hash and Chain Logs During Dict Attach

Allow Setting Hash and Chain Logs on Contexts with Attached CDict

For Supported Strategies, Attach Dict Even When Params Don't Match

Split Attach and Copy Reset Strategies into Separate Implementation Functions

Pull Attachment Decision into Separate Function

And Then Avoid the Unused Parameter Warning

Move Asserts into Function to Avoid Unused Function Warning

Move Match State CParams Setting into `resetCCtx` and `continueCCtx`

Strengthen Assertion to Assert Equality

Remove CParams Arg from ZSTD_ldm_blockCompress

Stop Separately Passing CParams in ZSTD_lazy Internal Functions

Stop Separately Passing CParams in ZSTD_opt Internal Functions

Also Remove CParams from Table Filling Functions' Args

Also Assert Equivalency When Filling MatchState with Prefix

Remove CParams from Block Compressor Functions' Args

Assert `ctx` and `ms` cparams Equivalency

Remove cParams from CDict

Add ZSTD_compressionParameters to ZSTD_matchState_t

changed macro name

from EXIT_IF() to RETURN_IF()

EXIT could be misunderstood as exit(), which terminates program execution.
But the macro only leaves the function, not the program.

minor improvement in the multi-format suffix selection

Fix minigzip in std=c99 mode

[util] Fix lstat feature test macro

Add -Werror to *build rules

Fix another warning

Fix warnings

[zstd] Fix newly added  test case

[zstreamtest] Add failing test case

fixed unreachable section warning on Visual

Merge pull request #1343 from terrelln/faster-ci

[zstreamtest] Reduce memory of newapi tests

Merge branch 'dev' into donotdelete

Merge pull request #1345 from facebook/fixLegacy

fixed legacy compilation tests

fixed minor Visual conversion warnings

fixed longmatch test too

fixed legacy compilation tests

for some reason, these tests started failing recently on CircleCI

Merge pull request #1344 from terrelln/grub-fixes

Fixes for GRUB

minor refactoring of --list

trying to reduce recurrent patterns.

Restore ZSTD_noCompressBlock() for clarity

Reset number of threads less often

Change byte to value to avoid a GRUB typedef

Include stddef.h for size_t

[zstd] Add -Wmissing-prototypes

More aggressive limitations

[zstreamtest] Reduce memory of newapi tests

We could allocate up to 2^28 bytes of memory when using 2 threads with
window log = 24. Now, we limit it to 2^26 bytes of memory when not running
big tests.

I chose max window log = 22 since that is the maximum source size when
big tests are disabled. Hopefully this will be enough to reduce or
eliminate the test failures.

Merge pull request #1342 from facebook/fixcatyd

fix : huge (>4GB) chain of blocks