Remove HAVE_CONFIG_H check

config.h should always be included, a build without it never happens.
The checks on Linux always succeed, because autotools automatically
defines HAVE_CONFIG_H on all the Makefiles. On Windows this is not the
case. Removing all the checks revealed that multiple projects did not
include config.h because HAVE_CONFIG_H wasn't defined. This was the case
for at least the following projects:
- acyclic
- bcomps
- ccomps
- dijkstra
- ingraphs
- gc
- gml2gv
- graphml2gv
- gv2gml
- gvcolor
- gvgen
- mm2gv
- nop
- pathplan
- rbtree
- sccmap
- topfish
- tred

Added `windows\include` to the additional include directories and adding
some library paths solves some, but not all problems that emerged when the
check was removed and some code actually got compiled that wasn't compiled
before.

Merge pull request #1159 from ErwinJanssen/fix/travis-build

Update Criterion submodule to fix Travis build

Update Criterion submodule to fix Travis build

For some reason the Travis build fails when pulling the Criterion submodule. Updating the submodule resolves this issue.

Re-added -lstdc++ to fix build breakage

Merge pull request #1157 from ErwinJanssen/fix-cast-warning

Use `strcmp` in instead of `gvrender_comparestr`.

Merge pull request #1156 from ErwinJanssen/travis-centos

Add build on CentOS 7 to Travis

Change cast of strcmp in bsearch.

Casting `strcmp` to `__compar_fn_t` doesn't work on Windows, the build
fails. Instead, cast `strcmp` to the required function type:
`(int(*)(const void*, const void*)`

Use `strcmp` in instead of `gvrender_comparestr`.

`bsearch` is used in this function to find a string in a string array. It's not needed to pass the address of `tok` to the function, because `tok` is already a pointer. `gvrender_comparestr` should then cast the values to `(const char)` instead of `*(char**)`, because `s1` and `s2` are strings, not pointers to a string. With this change the function only calls `strcmp`, so passing `strcmp` to `bsearch` has the same effect and `gvrender_comparestr` can be removed.
This gets rid of all the `-Wcast-qual` warnings as well.

Travis: Add build on CentOS 7 using Docker.

There are now two environments, one where the variable DOCKER_BUILD is true and one where it is false. False means a normal build on the Travis build image, true means docker will be used to run a build on a CentOS 7 image. The setup of the CentOS 7 image is done in the Dockerfile in `ci/centos7`. It takes a clean CentOS 7 image and installs the required build utilities and dependencies for a minimal build.

Added error checking to ci/build_and_test.sh

Instead of executing all command even if some fail, stop when a command fails and print an error message.

Removed `-lstdc++` from Makefiles

This flag causes errors in the Docker build. Removing it still results in a succesfull build and the previous changelogs indicate that this flag should be removed.

Use `build_and_test.sh` script in Travis build.

This script makes it easier to add multiple platforms using docker.

revert change - not right

drop some strange casts to cllean up some warnings

Merge pull request #1155 from ErwinJanssen/windows-release

Various improvements to Windows build

Merge pull request #1154 from ErwinJanssen/refactor-sign-conversion

Resolve some conversion warnings

Appveyor: split build into two jobs.

Split the build into two jobs: one for Release and one for Debug.

Windows: added gdiplus to build.

The gdiplus Visual Studio project was previously removed, but now added
again and cleaned in the same way as the other projects.

Windows: Use $(Configuration) in output directory

Instead of using the name of the configuration (Debug or Release), use the
variable that contains this value.

Removed `QCoreApplication::UnicodeUTF8` arguments

From the Qt documentation (http://wiki.qt.io/Transition_from_Qt_4.x_to_Qt5#QCoreApplication::UnicodeUTF8_is_deprecated):

QCoreApplication::UnicodeUTF8 is deprecated

This enum type used to define the 8-bit encoding of character string arguments to translate(). This enum is now obsolete and UTF-8 will be used in all cases. So remove all instances of QCoreApplication::UnicodeUTF8.

In gvedit, replaced `toAscii()` with `toLatin()`

From the Qt documentation
(http://wiki.qt.io/Transition_from_Qt_4.x_to_Qt5#toAscii.28.29_and_fromAscii.28.29_Methods_are_deprecated):

toAscii() and fromAscii() Methods are deprecated

Replace all instances of

```
fromAscii()
toAscii()
```

to

```
fromLatin1()
toLatin1()
```

Windows: added ccomps to build

The ccomps Visual Studio project was previously removed, but now added
again and cleaned in the same way as the other projects.

Removed windows/libann

This folder only contained a project file, no code. Also, lib ANN is
included in windows\dependencies\ANN.

Resolved "macro redefinition" warnings.

On Windows, the Linux build does not throw this warning, the warning C4005
"macro redefinition" is thrown several hundered times. Most of these could
be prepended by an `#undef`, this resolves the warning.

Windows: Generate no debug information for Release.

The Release configuration doesn't shouldn't contain debug information,
because this is going to be released to the general public.

Windows: set <DebugInformationFormat> to default.

Set the <DebugInformationFormat> to default for the release configuration.
The value it had (Zi) is the same as the default value, so the
<DebugInformationFormat> can be safely removed.

Appveyor: Add Debug build and artifact.

Run both a Release and a Debug build create artifacts for both
configurations.

Windows: Change release output directory

Use $(SolutionDir)Release\Graphviz\bin\ instead of
$(SolutionDir)Graphviz\bin\. This is more consistent and allows for an
easier artifact selection on Appveyor.

shapes.c: Initialize `pointf p` variable.

Because this variable was used uninitialized, it caused a runtime error on
Windows.

Windows: enabled Debug build for all projects.

Not all project would build when using the Debug configuration, and some
projects had conflicting settings that didn't allow a Debug build. Both
are fixed.

Appveyor: produce two artifacts

Produce two artifacts: one with all the files that are generated during
the build (PDB files for debug, etc.) and one without. The last one is
most suited as a release.

Appveyor: Fix typo in appveyor.yml.

Added a missing 'p' to .zip file.

Windows: Remove <LinkIncremental> from vmalloc.

The Debug configuration still had the <LinkIncremental> property.

Windows: Removed false <RandomizedBaseAddress>

Some project had <RandomizedBaseAddress> set to false. Setting this to
false disables ASLR which is a nice security feature. These options are
removed because the default value for <RandomizedBaseAddress> is true.

Appveyor: Reduce empty lines appveyor.yml

Windows: dependencies copied on post-build step.

Dependencies (DLLs from 3rd parties) are copied on the post-build step to
the Graphviz\bin folder.

Windows: clean additional library directories

Removed unused and used `$(OutDir)` instead of
`$(SolutionDir)Graphviz\bin\` for release and
`$(SolutionDir)Debug\Graphviz\bin\` for debug.

Windows: clean additional include directories.

Removed unused include directories and reordered the remaining.

Windows: Removed <LinkIncremental> property

With the current configuration, this property is ignored, so using the
default value is fine.

Windows: Redefined intermediate build directories.

Specifying the configuration name as intermediate build directory is not
needed. The default behavior is the same, so it can be safely removed.

For the projects in cmd/tools, $(Configuration)\ can be used instead of
the actual configuration name.

Appveyor: Reduce build output noise.

Before the build, delete the file "C:\Program Files
(x86)\MSBuild\14.0\Microsoft.Common.targets\ImportAfter\Xamarin.Common.targets",
Otherwise several "target does not exist and will be ignored" messages are
printed.

Windows: Compiling with /MT instead of /MD

On a release build, compile all projects with /MT instead of /MD. This
means that required system libraries are staticly linked instead, so no
DLL is needed. Before this change, the user was required to install the
SDK, this is no longer the case.

The only exception is gvplugin_gd. This can't be compiled with /MT,
because it's dependency libgd is compiled with /MD.

Ignoring Visual Studio Code user folder.

Windows: Disable warning C4996

Disable MSVC warning C4996. This warning mostly fires on completely valid
code and fixing this warning compromises code portability, while the code
quality isn't improved substantially.

Add documentation on json output.

Merge branch 'master' of https://github.com/ellson/graphviz

Fix bug in SparseMatrix_copy tries to copy a pattern matrix which does
have values.

If a matrix is constructed using _from_coordinate_arrays, it calls
from_coordinate_arrays_internal with SUM_REPEATED_ALL. The latter function then
calls sum_repeated_entries. This causes values to be added to the array

Later, if the matrix is passed to make_undirected, this calls symmetrize,
which may call _copy. If _copy uses _new, the new array will use the input
array's type (pattern) and not create memory values, causing the memcpy to fail.

Resolved 24 warnings in tclpathplan.c

Changed the type of some local variables from `int` to `size_t`.

Resolved 14 warnings in labels.c

Some field could be made a `size_t` instead of an `int`.
The cast now matches the type of the assigned variable.

Resolve one -Wsign-conversion in htmltable.c

The local variable `cnt` could be made of type `size_t`.

Resolve 7 -Wconversion warnings in htmllex.c

The return value of `toupper` can be safely cast to a char, as long as the input of `toupper` is actually a char.

Resolve 7 warnings in colxlate.c

In three function, the type of two local variables could safely be changed to `size_t`.

Resolved two warnings in arrows.c

The local variable `namelen` is always positive, so the type `size_t` could be used.

Resolved one -Wsign-conversion in cvt.c

There are multiple -Wsign-conversion warnings, but only one could be resolved without analysing or changing more than a few lines of code.

Resolved four -Wsign-conversion in rec.c

`arg_rec_size` is always positive, so it can be an unsigned integer.

Resolve four warnings in grammar.y

Setting the proper type for `len` resolved them.

Resolve one Wsign-conversion in attr.c

This cast that resolves this warning does not change the sign of `sz`, because the if statement above it makes sure it isn't a negative number.

Resolve three Wsign-conversion in agxbuf.c

These three warnings could be resolved without making functional changes in the code. Also initialized the three local variables in this function to zero.

convert 'unsigned long' to 'uint64_t' for Windows portability

Merge pull request #1151 from arjamizo/master

fix typo TB->BT in docs:shapes

fix typo TB->BT in docs:shapes

Replace cluster with subgraph, as the output will also reflect non-cluster
subgraphs.

Add schema for dot output in json format.

Merge pull request #1150 from ErwinJanssen/configure-ac

Removed POSIX version flag from configure.ac

Removed POSIX version flag from configure.ac

With the new way of enabling C99 (`AC_PROG_CC_C99` instead of `-std=c99`) this flag is no longer needed for a successfull compilation.

el5 breakage, need ,  in dist target, but not in install target

remove -D_POSIX_C_SOURCE=200112L as it breaks ./configure test for php.h

Merge branch 'master' of github.com:/ellson/graphviz

disable php in fedora24 till we can work out what broke

try updating php includedirs to match php-config --includes

 build manpages unconditionally on Travis hsost

Merge pull request #1149 from ErwinJanssen/improve-travis

Improve Travis + deploy make dist results

Small formatting change.

Edges attached to a cluster were not getting assigned an index number.

quote  as it contains spaces when No

suppress "unused parameter" warning during ./configure

Travis: deploy result of `make dist`

Every commit will still result in a new untagged release on Github, but the results of `make dist` will now be attached to these releases.

Travis: clean build steps.

Because the results of `make install` when using a prefix aren't binaries suitable to be released, some build steps can either be removed or simpified.

Travis: color gcc output

Use colorgcc to add color to the output of gcc. This improves readability and makes it easier to find warnings.

Merge pull request #1148 from ErwinJanssen/unit-test

Introduction unit test for current commnand line behavior

Tests added for current behaviour command line.

This commit adds some tests that "document" the current behavior of the command line interface. When refactoring command line interface (like the parsing code), the observable behavior of the command line interface should stay the same, this is to ensure backwards compatibility. These tests are a start, it covers some basic command line operations (printing the version and usage), but show some interesting things. Mainly that the order of the flags is very important.

- `dot -Vrandom` prints the version, but `dot -randomV` causes an error.
- `dot -?V` prints the usage, but `dot -V?` prints the version.

While it may be desirable to change this, when refactoring this behavior should stay the same. Changing the existing interface should be done in Graphviz 3.

Travis: Install Criterion

Adds the installation of Criterion to the `install` step in the Travis configuration. A submodule is used so changes in the Criterion repository can't break the build. A submodule points to a specific commit.

Simple unit test added.

This unit test serves to validate the configuration and installation of the Criterion unit testing framework. This test will only be compiled and run if Criterion is installed.

Check for unit test framework in configure.ac

Criterion is a unit testing framework for C and C++ and it's C99 compatible. It allows for fast test writing because, unlike a lot of other C unit test frameworks, it doesn't require a lot of boilerplate code.

This check in configure.ac checks if Criterion is installed on the system.

Merge pull request #1147 from GadgetSteve/master

Changed README.md reference github for documentation.

Changes to README.md to reference github for documentation

Adresses #88 and #77.

Merge pull request #3 from ellson/master

Sync with master

some -W flags not available on centos[56]

Fix bug in printing error message about graphs with loops or multiedges; note port restriction in
man page.

cgraph is not a configure option these days

improve handling of extra warnings if C99 available

more incompatible warning switches

-Wtrampolines -Wdouble-promotion  - cause errors on older (centos[56]) compilers

Merge pull request #1145 from ErwinJanssen/refactor

Enable C99, enable more warnings and solve some warnings.

add missing layers support in -Tsvgz device

Fixed Wundef warnings in various files in sfio.

The fixed header files are included in multiple locations, causing a reducution of around 2000 Wundef warnings after these changes.

Fixed various Wundef warnings.

Surrounded _BLD_cdt with defined() in cdt.h.

This solves hundreds of Wundef warnings.

Surrounded _PACKAGE_ast with defined() where needed.

This solves more than a thousend Wundef warnings.

Fixed Wundef warnings in ast_common.h.in

There were several Wundef warnings in ast_common.h.in. Because this file is included in a lot of other files, these warnings showed up often. This change fixed 1119 warnings in total.