Merge pull request #343 from afbjorklund/clang-warning

Fix clang warning with older compiler versions

Fix clang warning with older compiler versions

Don't hardcode the clang version, apparently clang-3.8
did have the warning even though clang-3.4 did not...

Add NEWS for v3.5.1

Update copyright years

Add missing getopt_long.c source file to dist (#341)

Missed to include this when making compilation optional.

Closes #340.

Use double for limit_multiple

This allows for removing the -Wno-double-promotion and
-Wno-float-conversion options which are not available for the clang
version currently used for “make analyze”.

Fix constness of struct stats_info message member

Refactor conf item lookup code

- Extracted parse/format/verify functions into a separate confitems.c
  file.
- The *_lookup.c files are now compilation units of their own instead of
  being included inside conf.c. This feels cleaner, and also relieves
  cppcheck from having to check dirty, autogenerated code.

Ignore cppcheck “Resource leak” warnings in test code

It’s not worth the effort to write leak-free code for test failure code
paths.

Include error/warning ID in cppcheck output

Use sizeof instead of repeating buffer size

Don’t pass CFLAGS to the linker

Improve distcheck

- Build with more warnings.
- Added an extra build with clang.
- Build with -Werror to actually catch the errors.

Always enable “more warnings” in dev mode

Sort list

Fix missing header include / prototype

no previous prototype for function 'unify_hash'

Fix const-correctness of the config formatter

Both strdup and format are returning "char *"

Ignore return value from fwrite when writing debug log

This silences some compilers’ warnings about ignoring return value of
fwrite.

There’s not much we can do if they fail, and it’s also not important
enough to e.g. call failed() for the debug logs.

Remove unused return value from cc_dump_log_buffer

Add Xavier René-Corail to .mailcap

Prepare for v3.5

Add -Wno-implicit-fallthrough for zlib build in non-dev mode as well

Add some more silent make rules

Remove obsolete cppcheck suppression

Make cppcheck find config.h

Add assert in language_is_preprocessed

This seems to make scan-build happy.

Update NEWS

Tweak silent make rule printouts

Default to silent make rules unconditionally

Add configure option for silent make rules

Adopted from AM_SILENT_RULES, default to no

Sort commandline option list

Fix alignment and indentation

New command-line option -k/--get-config prints the value of a config key

We print exactly the value of the configuration setting, rather
than a full item pair "(origin) key = value" as --print-config does.

This is useful for scripting. For example, $(ccache -k cache_dir)
gives the cache directory path.

Use the per-item printers in conf_print_items

Add a confitem field for entry printing, and a printer for each type

Use memcpy instead of strncpy in append_log

We’re copying a known amount of bytes; strncpy is almost never the right
solution.

Pass -Wno-implicit-fallthrough when compiling bundled zlib

It’s of no use to get warnings for 3pp code.

Minor fixes for cygwin/msys2

Add -Wno-error=implicit-fallthrough, if the compiler supports it,  to
dev.mk so that zlib will compile with -Werror.

Disable -Wdeprecated-declarations in the mkstemp() shim in util.c so
that it compiles with -Werror.

Stop assuming that mkstemp() is broken on cygwin, I could not find any
documentation for this.

Fix a couple of erroneous checks in zlib that assume cygwin is the same
as win32.

Clean up includes

Fix bugs in debug mode logging

Need to dump log buffer as the last exit function to not lose any logs.
Also, made sure to print the final result log line even if the log mode
isn’t enabled.

Fix debug mode bug in cc_log_argv

Refactor debug mode code

Knowledge of the debug mode snuck into the mdfour struct and code, which
did not feel good.

* Introduced a new “struct hash”, which is used by the hash_* functions
  instead of “struct mdfour”. “struct hash” contains the mdfour state
  and also debug file handles that can be set via a new
  hash_enable_debug function.
* Removed “identifier” from the mdfour struct and code.
* Introduced hash.h and all documented hash_* functions.
* Introduced unify.h.
* Removed some global state from util.c.

Add LGTM.com code quality badges

Fix Clang warning “declaration shadows a local variable”

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