disambiguate lib/spine/*.c #includes

Related to #1785.

remove unnecessary lib/spine include directory

disambiguate lib/sparse/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/sparse include directory

dismbiguate lib/sfdpgen/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/sfdpgen include directory

disambiguate lib/rbtree/*.c #includes

Related to #1785.

remove unnecessary lib/rbtree include directory

disambiguate lib/pathplan/*.c #includes

Related to #1785.

disambiguate lib/patchwork/*.c #includes

Related to #1785.

remove unnecessary lib/patchwork include directory

disambiguate lib/pack/*.c #includes

Related to #1785.

remove unnecessary lib/pack include directory

disambiguate lib/osage/*.c #includes

Related to #1785.

remove unnecessary lib/osage include directory

disambiguate lib/ortho/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/ortho include directory

disambiguate lib/neatogen/*.c #includes

Related to #1785.

remove unnecessary lib/neatogen include directory

disambiguate lib/mingle/*.c #includes

Related to #1785.

remove unnecessary lib/mingle include directory

disambiguate lib/label/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/label include directory

disambiguate lib/inkpot/*.c #includes

Related to #1785.

disambiguate lib/ingraphs/*.c #includes

Related to #1785.

remove unnecessary lib/ingraphs include directory

disambiguate lib/gvpr/*.c #includes

Related to #1785.

further disambiguate inclusion of colortbl.h

This generated header needs some extra acrobatics. Related to #1785.

remove unnecessary lib/gvpr include directory

disambiguate lib/gvc/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/gvc include directory

disambiguate lib/glcomp/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/glcomp include directory

disambiguate lib/fdpgen/*.c #includes

Related to #1785.

remove unnecessary lib/fdpgen include directory

disambiguate lib/expr/*.c #includes

Related to #1785.

remove unnecessary lib/expr include directory

disambiguate lib/edgepaint/*.c #includes

Related to #1785.

remove unnecessary lib/edgepaint include directory

disambiguate lib/dotgen/*.c #includes

Related to #1785.

remove unnecessary lib/dotgen include directory

disambiguate lib/common/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/common include directory

disambiguate lib/circogen/*.c #includes

Related to #1785.

remove unnecessary lib/circogen include directory

include internal headers with <...> in lib/ast/*.c

It turns out to deal with #1785, we will need to include internal headers this
way as well as external headers. 2ecb837a05be3d69971001708e7acccfd470ad97 should
have gone in this direction to begin with.

make lib/cgraph/*.c #includes unambiguous

Related to #1785.

remove unnecessary cgraph #include path

remove #define EXTERN trick in cgraph

This was designed to allow one declaration of Ag_G_global serve as both
declaration and definition. However, this obscures the actual definition of this
variable from any code browser unable to evaluate macro expansion (e.g. Cscope).
There's no real need for this trick, as we can just define the variable as
normal.

disambiguate lib/cdt/*.c #includes

Related to #1785.

remove unused cdt 'Version'

Merge branch '3a0128e0-8665-418d-b92d-642f114890ce' into 'master'

remove unused SFIO 'disciplines'

See merge request graphviz/graphviz!1556

remove unused SFIO 'disciplines'

Merge branch 'fcec7644-4a9d-47cb-ba10-81305417378d' into 'master'

remove ad hoc name mangling for lib/expr/grammar.y

See merge request graphviz/graphviz!1553

remove unnecessary intermediate build artifact

This is the equivalent of c498c4bb3e12e48e3bab9aba8d3162535e163c92 for
exparse.y. Related to #1806.

use Bison's api.prefix instead of name mangling in expr

This is the equivalent of cc2cb52756f545d6863d6fd801fa23762a81704c. My original
intent was to split the changes that remove the build-time symbol mangling (the
equivalent of 9240da12ab7857975aa34279520f1f6aad113d48) into a separate commit.
However, the expression parser does a lot of messy tricks to collide with and
suppress Bison-generated macros. It was simpler to undo all of this in one go
then try to thread this convoluted needle. Related to #1806.

explicitly name output of lib/expr/exparse.y rule

This has no effect right now, but an upcoming change modifies api.prefix which,
without this, would alter the name of the generated parser. Related to #1806.

explicitly drop Yacc, Bison < 3 compatibility in exparse.y

Yacc was already not supported as this parser is using Bison extensions, but
this at least makes explicit what our requirements are. This is the equivalent
of 4a13c9598ad3d950a251e5d3e5be8d05c59c6214. Related to #1806.

suppress warnings from using Bison extensions in lib/expr/exparse.y

This is the equivalent of 8c099fc032074a0bb5961d67ef4df05ddca7eba4 but for a
different parser. Surprisingly exparse.y was already using Bison extensions, so
this actually suppresses two warnings:

  bison -y -dtv ../../lib/expr/exparse.y
  ../../lib/expr/exparse.y:131.1-7: warning: POSIX Yacc does not support %binary [-Wyacc]
   %binary       <op>    EQ      NE
   ^~~~~~~
  ../../lib/expr/exparse.y:132.1-7: warning: POSIX Yacc does not support %binary [-Wyacc]
   %binary       <op>    '<'     '>'     LE      GE
   ^~~~~~~

Related to #1806.

remove incorrect EXSTYPE redefinition

Bison's -pPREFIX -- which, incidentally, this parser is not using -- does *not*
miss YYSTYPE. Upcoming changes expose that this detrimentally interferes with
attempts to properly prefix the expr parser. Related to #1806.

Merge branch 'remove-warnings-related-to-issue-1800' into 'master'

Remove warnings related to issue 1800

See merge request graphviz/graphviz!1554

Remove 1 -Wswitch-default warning in attr.c

Remove 1 -Wunused-but-set-variable warning in attr.c

Remove 2 -Wsign-conversion warnings in attr.c

Remove 1 -Wconversion warning in attr.c

Remove 4 -Wmissing-prototypes warnings in attr.c by adding static

Remove 2 -Wmissing-field-initializers warnings in attr.c

Remove 1 -Wswitch-default warning in agdelrec

Remove 3 -Wunused-parameter warnings in rec.c

Remove 2 -Wconversion warnings in set_data

Remove 1 -Wswitch-default warning in agbindrec by removing redundant switch

Remove 1 -Wconversion warnings in cccomps

Remove 2 -Wsign-conversion warnings in cccomps

Remove 3 -Wsign-conversion warnings in ccomps

Remove 3 -Wsign-conversion warnings in pccomps

Remove 2 -Wconversion warnings in setPrefix

Remove 1 -Wconversion warning in clMarkFn

It's safe to cast to char since the value is always 0 or 1.

Remove 1 -Wconversion warning in markFn

It's safe to cast to char since the value is always 0 or 1.

Remove 1 -Wbad-function-cast warning in agdelsubg

Restores the public API to what's documented in cgraph.3 by instead of
casting the void pointer that dtdelete returns to long, returns an int
which is 0 if the pointer is NULL and 1 otherwise. Internally the
return value is however not used today.

Remove 1 -Wbad-function-cast warning in agdtdelete

Changes the *internal* API by instead of casting the void pointer that
dtdelete returns to long, returns an int which is 0 if the pointer is
NULL and 1 otherwise. It doesn't really matter though, since the
return value is not used anywhere today.

Remove 1 -Wunused-parameter warning in agrefstrdump when -DDEBUG

Remove 1 -Wconversion warning in aghtmlstr

Remove 9 -Wsign-conversion warnings in _DTCMP

Casting to size_t is safe since sz is always positive when it happens.

Remove 1 -Wunused-parameter warning in idopen

Remove 1 -Wconversion warning in idalloc

It's safe to cast to int from long since the value is always FALSE (0).

Remove 2 -Wconversion warnings in idmap

Its safe to cast to int from long since the value is always TRUE (1).

Merge branch 'fix-silent-fail-of-windows-ci-builds' into 'master'

Fix silent fail of Windows builds in CI

Closes #1815

See merge request graphviz/graphviz!1560

Fix silent fail of Windows build in CI

Fixes https://gitlab.com/graphviz/graphviz/-/issues/1815.

Merge branch 'C1AB8FAC-C63E-4D45-AF41-31FC5100516E' into 'master'

remove unnecessary NUL_FILE abstraction

See merge request graphviz/graphviz!1550

remove now unused NUL_FILE

remove unnecessary NUL_FILE use

The Cairo API supports simply passing NULL to indicate you do not want the
output written to a file.

Merge branch '2be5439a-0e4a-4a18-98ad-2a13d8c5b9fd' into 'master'

fix failure to link gvpr with debug enabled

Closes #1807

See merge request graphviz/graphviz!1549

fix failure to link gvpr with debug enabled

When using `CFLAGS=-DDEBUG CXXFLAGS=-DDEBUG ./configure --enable-debug`, gvpr
would fail to link due to missing libraries. Fixes #1807.

Merge branch '5213EAD3-14D1-45A4-A269-4C7D9DA5F44B' into 'master'

remove unused sfdcslow()

See merge request graphviz/graphviz!1545

remove unused sfdcslow()

Merge branch 'c6f76f21-830c-4108-aafa-806719ed59d2' into 'master'

clean up RxSpencer CMake warnings

See merge request graphviz/graphviz!1544

remove advanced target suppression for RxSpencer in CMake builds

This section suppressed detailed messages about where (or not) RxSpencer was
found. Given the majority of CMake builds are performed by either CI or advanced
users, it did not make sense to suppress this information that is very useful
for debugging failures.

fix case warnings for RxSpencer during CMake builds

This fixes the following CI warning:

  -- Found RXSPENCER: ...rxspencer.lib
  CMake Warning (dev) at ...FindPackageHandleStandardArgs.cmake:272 (message):
    The package name passed to `find_package_handle_standard_args` (RXSPENCER)
    does not match the name of the calling package (RxSpencer).  This can lead
    to problems in calling code that expects `find_package` result variables
    (e.g., `_FOUND`) to follow a certain pattern.
  Call Stack (most recent call first):
    cmake/FindRxSpencer.cmake:5 (find_package_handle_standard_args)
    CMakeLists.txt:90 (find_package)
  This warning is for project developers.  Use -Wno-dev to suppress it.

Merge branch '064bf64e-752b-45fc-8da7-e66050fbe8b5' into 'master'

remove ad hoc name mangling in cgraph parser

See merge request graphviz/graphviz!1543

remove unnecessary intermediate build step