xmlparse.c: Resolve writeRandomBytes_* looping code duplication

Merge branch 'david-loffredo-rand_s-test' (#302, fixes #299)

Changes: Document #299 and #302

xmlparse.c: Mention rand_s with stdlib include

replace loadlibrary.c and RtlGenRandom call with rand_s() on windows

Signed-off-by: David Loffredo <loffredo@steptools.com>

Make win32 stdlib.h define rand_s(), add writeRandomBytes_rand_s().

Signed-off-by: David Loffredo <loffredo@steptools.com>

Merge pull request #298 from libexpat/issue-240-stopping-parser-from-inside-end-element-handler

Fix stopping parser from inside end element handler (#240)

Changes: Document fix to #240

runtests.c: Add two test cases for/from issue #240

Many thanks to Rolf Ade (@pointsman)!

xmlparse.c: Fix effect of XML_StopParser for end element handlers (#240)

Merge branch 'clang-format-9' (#293, fixes #24)

Changes: Document mass-application of clang-format 9

Mass-apply clang-format 9 using ./apply-clang-format.sh

siphash.h: Prepare for application of clang-format

runtests.c: Mark some sections as to be excluded by clang-format

This will make clang-format 9 produce stable results.

xmltok_impl.c: Re-format use of LEAD_CASE macro

This will make clang-format 9 produce stable results.

Travis CI: Integrate clang-format 9

.clang-format: Comment out AlignConsecutiveMacros

Because clang-format 9.0.0-svn362770-1~exp1+0~20190607061653.2401~1.gbp2444fc
on Travis does not yet support it.

.clang-format: Tune

.clang-format: Update to clang-format 9 default style

.clang-format: Pin clang-format 8 default style

Trailing whitespace has been removed.

Merge branch 'issue-266-make-travis-ci-cover-mingw' (#266)

Travis CI: Cover MinGW

Travis CI: Compress some whitespace for upcoming extension

Makefile.am: Make "run-xmltest" robust to wine error output

qa.sh: Add missing DLL symlinks for MinGW

qa.sh: Show tests/runtests*.log on "make check" failure for MinGW

Merge branch 'drop-xmlwf-com-code' (#286)

Changes: Document #286

xmlwf: Drop COM code

Merge pull request #296 from libexpat/fix-qa-sh-integration

Fix qa.sh integration

Travis CI: Pass ${CONFIGURE_ARGS} to ./qa.sh

qa.sh: Pass command line arguments to ./configure

minicheck.c: Fix volatile variables (#235)

Travis CI: Move cppcheck according to alphabetic order

expat.h: Drop OpenVMS support leftover (#14)

Changes: Add missing entries

Merge branch 'clang-static-analyzer'

runtests.c: Address null pointer warnings by Clang Static Analyzer

structdata.c: Address Clang Static Analyzer warnings (#294)

xmlparse.c: Address Clang Static Analyzer false positive (#295)

runtests.c: Pull semicolon out of braces

runtests.c: Reduce code duplication

Travis CI: Drop "sudo: required"

Merge pull request #290 from manish3kumar/integrate-sa

Integrate cppcheck with travis-ci for libexpat

Integrate cppcheck with travis-ci for libexpat

Signed-off-by: Manish, Kumar <manish3.kumar@intel.com>

Merge branch 'travis-clang-9'

Travis CI: Use Clang 9 on Linux

qa.sh: Dump compiler version

AppVeyor|Travis: Re-introduce clone depth

Depth 1 made surpassed pushed fail so it was dropped.
No depth limit wastes time and hardware resources though, and
has some risk that depth 1 will be re-introduced, by mistake.
So let's try 50 now.

Related commits:
- 10c235458830fc894a6381cf6267246c68848adf — AppVeyor
- 88cf657da91ef7a83d705cf7951cfd6c6b7d98fc — Travis

Merge branch 'solve-warnings.marcomaggi-2018-10-30' (fixes #235)

fixed warning about potential null pointer dereference

runtests.c: Address warnings about variable shadowing

Merge branch 'marcomaggi-cleaner-test-for-mingw.marcomaggi-2018-10-28' (fixes #231)

configure.ac: Use curly braces for variable

configure.ac: Adhust spelling of MinGW

cleaner test for mingw as host system

Travis CI: Drop "git.depth: 1" because subsequent pushes were failing ongoing builds

See 10c235458830fc894a6381cf6267246c68848adf for AppVeyor and more details.

fixed use of compilation flags in coverage script (#229)

Merge branch 'marcomaggi-shell-scripts-review.marcomaggi-2018-10-21' (fixes #229)

clean_coverage.sh: Use portable shebang

added shebang and comments to clean_coverage script

internal.h: Fix UNUSED_P for GCC 3.3 (fixes #288)

runtests.c: Address cppcheck warning (fixes #283)

[tests/runtests.c:647]: (error) Address of local auto-variable assigned to a function parameter.

AppVeyor: Drop "clone_depth: 1" because subsequent pushes were failing ongoing builds

Symptom:
  Build started
  git config --global core.autocrlf input
  git clone -q --depth=1 --branch=master https://github.com/libexpat/libexpat.git c:\projects\libexpat
  git checkout -qf d4d8d486af1810bff52612d84533985b9b47b7a6
  fatal: reference is not a tree: d4d8d486af1810bff52612d84533985b9b47b7a6

Cause:
  A more recent push made d4d8d486af1810bff52612d84533985b9b47b7a6
  the second-last commit so that --depth=1 would create a commit
  graph without that commit.

Documented at:
https://www.appveyor.com/docs/how-to/repository-shallow-clone/#setting-depth-of-git-clone-command

xmlwf: Ensure that STDMETHODIMP_ is defined for cppcheck (fixes #285)

xmltok.c: Use memcpy instead of copying single bytes (fixes #282)

Fixes cppcheck's complaint as a side-effect.

tests: Silence cppcheck false positive (fixes #284)

Merge branch 'MohammedKhajapasha-issue_273' (#281, fixes #273)

Travis: Limit distcheck tests to plain distcheck

--enable-xml-attr-info is tested elsewhere and this
combination does not seem to provide additional value
to justify runtime.

Travis: Drop test of in-source CMake

Out-of-source CMake is still there and is more important.

Enable CI for XML_ATTR_INFO

Enable CI to cover XML_ATTR_INFO feature.

Signed-off-by: Mohammed Khajapasha <mohammed.khajapasha@intel.com>

Merge branch 'rewrite-qa-sh' (#278)

Travis: Move from exclusion to inclusion approach

Travis: Excluding UndefinedBehaviorSanitizer for macOS due to link errors

https://travis-ci.org/libexpat/libexpat/jobs/556567990#L529

Travis: Disable gcov mode on macOS for lack of -s argument

gcov: Unknown command line argument '-s'

Travis: Fix MemorySanitizer exclude for macOS

qa.sh: Pass -fsanitize=undefined to linker as well

qa.sh: Use clang++ to link (for ubsan)

qa.sh: Support GCC coverage reporting with Apple GCC as well

Travis: Cover MemorySanitizer and UndefinedBehaviorSanitizer

qa.sh: Re-write

- A bit more flexible and high level
- Drop support for ncc
- Fix coverage rendering, drop split mode

Merge branch 'kkkunche-issue_239' (#239, #277)

Changes: Document #239 and #277

CMake: Write expat_config.h as late as needed

Previously, our post-processing of HAVE_GETRANDOM did
was not applied to expat_config.h.

CMake: Resolve code duplication by use of a macro

CMake: Support bypassing auto-detection of syscall SYS_getrandom

CMake: Turn getrandom build flag into a tribool

CMake: Move reset of HAVE_GETRANDOM after ConfigureChecks.cmake

CMake: Option to Disable the usuage of getrandom after autodetect

Merge branch 'issue-268-address-variable-shadowing' (#270, fixes #268)

Changes: Mention #268 and #270

Changes: Mention #244

Merge branch 'MohammedKhajapasha-master' (#264)

Changes: Document #264

coverage.sh: Simplify if statement

Conistent with unicode_enabled check above

coverage.sh: Make xml_attr_info_enabled match unicode_enabled

configure: Make --enable-xml-attr-info help match --enable-static

Enable XML_ATTR_INFO in build

enable XML_ATTR_INFO in build to cover XML_ATTR_INFO in CI

Signed-off-by: Mohammed Khajapasha <mohammed.khajapasha@intel.com>

xmlparse.c: Limit scope of new_connector to save sizeof(char*) on the stack