Move ATTR_NORETURN function declarations closer to the definition

Remove superfluous “extern” function declarations from a9e8810f

Function declarations are extern by default.

Merge remote-tracking branch 'afbjorklund/everything'

* afbjorklund/everything:
  Make configure work with older clang versions
  Add option for enabling more compiler warnings
  Add configure support for detecting clang
  Avoid bad function cast by using temporary
  Proper use of const qualifier char* strings
  Removed locally shadowed global variables
  Don't use undefined macros, use ifdef instead
  Add noreturn to functions that don't return
  Add missing extern prototypes, or make static
  Don't use reserved id macros in the headers
  Only compile getopt_long.c when needed
  Address issues found by lgtm.com alerts

Update NEWS

Fix configure arg --without-bundled-zlib

If the result of the AC_ARG_WITH check is 'no', assume system zlib.

Update uncrustify config and run uncrustify version 0.66.1_f

Remove bogus out-of-memory check in reformat()

Update NEWS

Improve debug mode documentation

Improve debug log filenames

The “<objectfile>.ccache-hashX” files are also input data, so I think
that calling the “human-readable” version “<objectfile>.ccache-input” a
bit unpedagogical. Changes:

<objectfile>.ccache-hashX -> <objectfile>.ccache-input-X
<objectfile>.ccache-input -> <objectfile>.ccache-input-text

Merge remote-tracking branch 'afbjorklund/hash_debug' into pr/279

* afbjorklund/hash_debug:
  Register debug_end as an exit function instead
  Avoid using strcat and small realloc
  Rename helper functions, as suggested in review
  Save debug file streams, instead of reopening
  Add section markers, to the text file
  Rename hash_debug, add matching end function
  Improve documentation with example file names
  Output log to debug, even without a logfile
  Avoid global md variable in mdfour file
  Add --hash-file command, for convenience
  Add functionality to debug the hashing
  Add feature to also log into a memory buffer
  Add config for debug option ($CCACHE_DEBUG)
  Add util to return the command as a string

Update NEWS

Improve documentation of file_stat_matches_ctime

Add option to disable ctime check for manifest

Make it possible to use e.g. "git-restore-mtime"*,
to use the original timestamps and avoid checksums.

* https://github.com/MestreLion/git-tools

Restoring ctime is much harder than restoring mtime,
since you need to change the system clock to do it.

Make configure work with older clang versions

Apparently clang-3.4 did not have -Wreserved-id-macro

Add option for enabling more compiler warnings

But remove some of the more annoying clang ones:

     18 [-Werror,-Wconversion]
     18 [-Werror,-Wdisabled-macro-expansion]
      9 [-Werror,-Wdouble-promotion]
      9 [-Werror,-Wfloat-conversion]
      1 [-Werror,-Wformat-nonliteral]
     52 [-Werror,-Wpadded]
     29 [-Werror,-Wshorten-64-to-32]
     75 [-Werror,-Wsign-conversion]

It's not really feasible to enable every warning.

Add configure support for detecting clang

Also note that both GCC and Clang are GNU

Avoid bad function cast by using temporary

The result after round/ceil _should_ work

Proper use of const qualifier char* strings

Removed locally shadowed global variables

Rename the local, or remove the parameter

Don't use undefined macros, use ifdef instead

Add noreturn to functions that don't return

Fix the extra semicolon and unreachable code

Add missing extern prototypes, or make static

Don't use reserved id macros in the headers

With an exception for the generated config.h

Only compile getopt_long.c when needed

ISO C disallows empty translation units

Address issues found by lgtm.com alerts

test: Fix various Solaris shell issues

Merge branch '3.4-maint'

* 3.4-maint:
  Prepare for v3.4.3
  Update NEWS
  Upgrade zlib to 1.2.11

Prepare for v3.4.3

Update NEWS

Upgrade zlib to 1.2.11

Register debug_end as an exit function instead

Merge branch '3.4-maint'

* 3.4-maint:
  Update NEWS
  Bail out on compiler option -save-temps=obj

Update NEWS

Bail out on compiler option -save-temps=obj

Also add a test, and handle default (=cwd)

Closes #299

Fix minor memory leak introduced in dde3543c

Fix “clang analyze” false positive

Update NEWS

process_preprocessed_file: Move gnu_getcwd() out of tight loop

Change process_preprocessed_file from calling getcwd() once per line
in the preprocessed source file to once at the start of the function.
The performance of getcwd() on Mac seems to be terrible compared to
Linux.  On a macOS 10.13 build machine, this change improves
process_preprocessed_file runtime on a 10MB preprocessed file from 75
seconds to .75 seconds.

Merge branch '3.4-maint'

* 3.4-maint:
  Update NEWS
  Add handling of separate -target parameter
  Bail out on too hard clang option “-MJ”

Make sure to install gperf for analyze

Also move the 32-bit libraries
to the 32-bit target (only).

Closes #290

Update NEWS

Add handling of separate -target parameter

Otherwise you will get an error in the log:

"x86_64-pc-linux-gnu is not a regular file,
 not considering as input file"

The argument seems to be specific to clang.

Update NEWS

Add a Vagrantfile for testing on FreeBSD

Compiler "cc" might still be the same as "clang"

It migt also be "gcc", but try to play it safe.

Add FreeBSD compatibility to test suites

Add patch for X/Open, from FreeBSD ports

Without this, e.g. "strndup" isn't visible.

Bail out on too hard clang option “-MJ”

See discussion in pull request #277.

Avoid using strcat and small realloc

For performance reasons with large strings, we don't want to
call strlen multiple times or to realloc one byte at a time.

Rename helper functions, as suggested in review

Save debug file streams, instead of reopening

Also opens txt file in text mode, not binary

Add section markers, to the text file

Rename hash_debug, add matching end function

Remove the parameter stating if hit or miss

Improve documentation with example file names

Output log to debug, even without a logfile

Avoid some shellcheck excludes that do make sense

Add target to run shellcheck on test bash scripts

Add missing delimiter when hashing “-MF”

Improve printing of “stats zeroed” statistics

* Refactored how formatting functions work in “ccache -s” output. They
  now return a formatted string (or NULL for “don’t print statistics
  line at all”) instead of printing the value.
* Added a new format_timestamp function.
* Added STATS_ZEROTIMESTAMP to the common stats_info array instead of
  being a special case.
* Let the STATS_ZEROTIMESTAMP value be 0 for new caches so that the
  “stats zeroed” statistics line only shows up when “ccache -z” actually
  has been called.

Closes #285.

Update NEWS

Merge branch '3.4-maint'

* 3.4-maint:
  Remove debug log
  Improve logging related to hashing of the CWD

Add configure option to disable man pages

Building the man page requires asciidoc, if building from the source
repository. Add option to only install the program, but not the man.

When building from the release tarball, the man page is already built.
So we can still default to installing it, and requiring asciidoc/a2x.

Closes #264

Avoid global md variable in mdfour file

Pass the pointer as a parameter instead

Add --hash-file command, for convenience

Add functionality to debug the hashing

Add feature to also log into a memory buffer

This feature is needed for the debug config

Add config for debug option ($CCACHE_DEBUG)

Add util to return the command as a string

We had print_command for logging to a file

Remove debug log

Improve logging related to hashing of the CWD

Improve installation instructions

Merge branch '3.4-maint'

* 3.4-maint:
  Update NEWS
  Avoid TOCTOU issue when deciding if config is valid

Update NEWS

Avoid TOCTOU issue when deciding if config is valid

Previously, a separate call to access had been used to distinguish
between I/O errors and invalid configuration file. This could lead to
spurious errors if configuration file have been created in-between call
to conf_read and access. Use errno to tell those two cases apart.

Closes #260.

Fix code style

hashutil: fix memory-leak/double free

Fix some casting warnings

Update NEWS

Add the sanitize blacklist contents to the hash

Update NEWS

Simplify logic

Print the time when stats last updated

Merge branch '3.4-maint'

* 3.4-maint:
  Use double when calculating cache thresholds
  Add a 32-bit build target, using multilib (-m32)
  Convert float config to double, add rounding
  Upgrade clang to 5.0, for the Travis docker
  Make sure to export ASAN_OPTIONS properly
  Add small helper to run all Travis tests
  Make sure to call configure with right path
  Add a travis-build container, for running locally
  Look for -fdebug-prefix-map feature explicitly
  Improve the Travis build matrix
  Fix log printing of before/after cleanup
  Remove irrelevant variable assignment in wipe_dir
  win32: Fix CCACHE_COMPILERCHECK=mtime test case

Use double when calculating cache thresholds

We could at least in theory have large enough caches that float’s
precision won’t be enough.

Also removed include of <float.h> which I think is superfluous.

Add a 32-bit build target, using multilib (-m32)

Use gcc-multilib rather than doing a full cross-compile.
But set the --host, just in case something is listening.

Convert float config to double, add rounding

We were getting different results when converting "float" to
integers, on 32-bit versus 64-bit platforms. So use "roundf".

Also exposed a typo in another testsuite, where we only cleaned
files as a result of roundoff... Make cleaning explicit again.

Upgrade clang to 5.0, for the Travis docker

Make sure to export ASAN_OPTIONS properly

Add small helper to run all Travis tests

The idea behind this script is to run the same checks as .travis.yml
but locally instead. It is not identical, but should be close enough.

Note: this does not include any tests for the osx or cuda platforms,
only the linux targets - although that does include the mingw build.

Make sure to call configure with right path

Add a travis-build container, for running locally

This is not the actual Travis docker container, but it
should be similar enough for doing local verification:

make travis
make travis CC=clang
make travis CC=clang CFLAGS="-fsanitize=address -g" ASAN_OPTIONS="detect_leaks=0"
make travis CC=i686-w64-mingw32-gcc HOST="--host=i686-w64-mingw32" TEST="unittest/run.exe"

Look for -fdebug-prefix-map feature explicitly

Seems like old clang-3.4 didn't have this feature, while the
newer MinGW might. So remove the hardcoding, and do the check.

Improve the Travis build matrix

Run in containers where possible, save the installations for the
targets that actually need to do them (currently: mingw and cuda)

Fix log printing of before/after cleanup

Seems like MinGW doesn't have the %z (size_t) printf format
So add some %f casting variant, so that it works everywhere.

Remove irrelevant variable assignment in wipe_dir

win32: Fix CCACHE_COMPILERCHECK=mtime test case

Previously if compiler command was an .sh script, win32 version
automatically added an .EXE extension to the command line and
then test case failed. With this change .sh files won't get
appended by an EXE extension and test case successfully runs

Merge branch '3.4-maint'

* 3.4-maint:
  Prepare for v3.4.2
  direct .i mode: Don't create tmp.cpp_stderr file at all

Prepare for v3.4.2

direct .i mode: Don't create tmp.cpp_stderr file at all

Create tmp file only when it's actually needed.