gvpack: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

gvpack.obj : error LNK2001: unresolved external symbol _Dtoset

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

graphml2gv: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

graphml2gv.obj : error LNK2001: unresolved external symbol _Dtoset

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

gxl2gv: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

gxl2gv.obj : error LNK2001: unresolved external symbol _Dtoset

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

dijkstra: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

dijkstra.obj : error LNK2001: unresolved external symbol _Dtoset

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

gml2gv: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

gv2gxl.obj : error LNK2001: unresolved external symbol _Dtoset

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

plugin/core: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

gvrender_core_json.obj : error LNK2001: unresolved external symbol _Dtoset

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

fdpgen: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

fdpgen.lib(grid.obj) : error LNK2001: unresolved external symbol _Dtoset

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

circogen: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

circogen.lib(edgelist.obj) : error LNK2001: unresolved external symbol _Dtoset

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

neatogen: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

neatogen.lib(neatosplines.obj) : error LNK2001: unresolved external symbol _Dtoset

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

expr: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

expr.lib(y.tab.obj) : error LNK2001: unresolved external symbol _Dtset

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

cgraph: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes to be
controlled by the GVDLL symbol in cdt.h.

Without this change, errors like this would occur:

refstr.obj : error LNK2001: unresolved external symbol _Dttree [C:\GitLab-Runner\builds\magjac\graphviz\lib\cgraph\cgraph.vcxproj]

Towards https://gitlab.com/graphviz/graphviz/-/issues/2173.

Merge branch 'post-remove-gvc-def-file-cleanup' into 'main'

Some cleanup of duplicate declarations

See merge request graphviz/graphviz!2379

plugin/gd: remove duplicate definition of Bezier in gvrender_gd_vrml.c

Removes this warning:

C:\Users\magja\graphviz\plugin\gd\gvrender_gd_vrml.c(41,86): warning C4273: 'Bezier': inconsistent dll linkage [C:\Users\magja\graphviz\plugin\gd\gvplugin_gd.vcxproj]

plugin/gd: remove duplicate declaration of shapeOf in plugin/gd/gvrender_gd_vrml.c

sparse: replace local declaration of colorxlate with including common/colorprocs.h in colorutil.h

sparse: MSBuild: add windows include dir to find config.h

An upcoming commit which will include config.h in colorprocs.h would
otherwise cause this error:

C:\Users\magja\graphviz\lib\common\colorprocs.h(13,10): fatal error C1083: Cannot open include file: 'config.h': No such file or directory [C:\Users\magja\graphviz\lib\sparse\gvsparse.vcxproj]

dot: unconditionally include globals.h in dot.c instead of duplicating declarations for WIN32_DLL

Merge branch 'remove-gvc-def-file' into 'main'

Remove the last symbols from gvc.def and remove the gvc.def file

Closes #2058

See merge request graphviz/graphviz!2376

gvc: remove now empty and obsolete gvc.def file

Closes https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvc: autotools: remove now obsolete use of gvc.def

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvc: CMake: remove now obsolete use of gvc.def

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvc: MSBuild: remove now obsolete use of gvc.def in project file

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvc: remove now obsolete exporting of render.h symbol rank2 from gvc.def

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

common: add missing declaration of rank2 to render.h

Also remove duplicated declaration of rank2 in rank.c in dotgen.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvc: remove now obsolete exporting of render.h symbols from gvc.def

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

common: correct storage class attributes in render.h

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvc: remove now obsolete exporting of gvcproc.h symbol gvrender_ptf from gvc.def

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvc: add storage-class attribute declaration of gvrender_ptf to gvcproc.h

This will allow removing this symbol from gvc.def in an upcoming commit.

Note that no other symbols from gvcproc.h were exported from gvc.def,
hence the attributes are applied only to gvrender_ptf.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

plugin/gd: replace local declaration of gvrender_ptf with including gvcproc.h in gvrender_gd_vrml.c

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

Merge branch 'smattr/659D906E-26CA-46C5-9E8A-CEC2CDBDE034' into 'main'

convert mingle to C++

See merge request graphviz/graphviz!2362

mingle: use safer C++ casts

mingle: remove unnecessary include

mingle: use C++ 'nullptr' instead of C 'NULL'

mingle: strip trailing space

mingle: replace C stdio with C++ iostream

mingle bundle: avoid some manual memory management

Note that this appears to actually fix a memory leak as `xx` was never freed
previously.

mingle genBundleColors: avoid some manual memory management

mingle: replace an 'agxbuf' with a 'std::ostringstream'

Related to #2154.

remove reference to non-existent cmd/mingle/ReadMe.txt

convert Mingle to C++

cmd/mingle depends on lib/mingle, which in turn depends on libANN, a C++
library. Thus the `mingle` binary needs to link against the C++ standard
library. There is no advantage to Mingle being implemented in C, and some nice
advantages to it being implemented in C++ instead.

This commit does a basic rename, as AFAICT nothing in minglemain.c has varying
semantics between C and C++. Future commits will use C++ features to simplify
the code.

Related to #2154.

mingle: cast results of 'malloc'

This has no effect at present. However, when moving this code to C++ in an
upcoming commit, it would trigger -fpermissive problems. This change
pre-emptively prevents this issue.

mingle: cast 'name' parameter values in 'agattr' calls

This has no effect at present. However, when moving this code to C++ in an
upcoming commit, it would trigger -Wwrite-strings warnings. This change
pre-emptively prevents these warnings.

mingle: convert usage text to a const char array

This makes the intent of the code slightly clearer, but also pre-emptively
avoids an upcoming -Wwrite-strings warning when converting this code to C++.

Related to #2154.

lib/mingle: add guards, making headers safe to include from C++

An upcoming change will convert cmd/mingle/minglemain.c to C++, which
includes some of these headers.

Related to #2154.

lib/sparse/color_palette.h: disambiguate an include

lib/sparse: add guards, making headers safe to include from C++

An upcoming change will convert cmd/mingle/minglemain.c to C++, which
transitively includes some of these headers.

Related to #2154.

Merge branch 'smattr/f2181a81-197c-481f-af2f-df55298be60e' into 'main'

remove cgraph macro aliases

See merge request graphviz/graphviz!2366

remove cgraph macro aliases

The functions removed in this commit were intended to be aliases for the
same-named macros declared in cgraph.h. The claimed purpose is debugger
introspection.

It is unclear to me what value these have in a modern development environment.
Some modern debuggers can understand macro definitions. Those that cannot are
_impeded_ by these aliases. A developer debugging looks at the alias definition
and thinks they understand the code, but they do not. The code that is executing
on their system is _not_ the code these functions compile to. To state something
stronger: it is unclear to me how these definitions were ever helpful. They
appear to (accidentally) obscure what is actually occurring in the machine at
runtime.

Note that these functions also bloated the cgraph .so/.a size by including
functions that were never called.

remove problematic 'agmkin', 'agmkout'

These functions were introduced in c9b7b0719776e022d760195fae2727b84be82eda to
match `AGMKOUT` and `AGMKIN` (#1212). But there are no exposed prototypes for
these. So the only way to call them is to externally prototype them with a
best-guess type signature and hope Graphviz never changes. This commit removes
them to reduce portability problems.

Merge branch 'smattr/59EAC8A6-959D-4CE7-86FD-A4205260BFC3' into 'main'

enable Mingle on CentOS

See merge request graphviz/graphviz!2335

remove 'ANN' toggle in RH RPM configuration

libANN is now installed in all Red Hat environments in CI (CentOS and Fedora)
and building Mingle is enabled in all. So we remove this configuration flag that
is no longer used.

enable Mingle on CentOS 7 in CI

Related to #1835.

install libANN on CentOS 7 in CI

Related to #1835.

remove CentOS 8 testing in CI

CentOS 8 was EOLed on 2021-12-31.¹ Note that some unorthodox roadmap changes by
Red Hat means CentOS 8 is EOLed _before_ CentOS 7.

¹ https://www.centos.org/centos-linux-eol/

Merge branch 'remove-geomprocs-h-symbols-from-gvc-def-file' into 'main'

Remove geomprocs.h symbols from gvc.def

See merge request graphviz/graphviz!2374

common: remove now obsolete exporting of geomprocs.h symbols from gvc.def

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

common: correct storage-class attribute declarations in geomprocs.h

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

Merge branch 'remove-colorprocs-h-symbols-from-gvc-def-file' into 'main'

Remove colorprocs.h symbols from gvc.def

See merge request graphviz/graphviz!2373

common: remove now obsolete exporting of colorprocs.h symbols from gvc.def

Note that the canontoken and colorCvt functions in colorprocs.h were
not exported from gvc.def.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

common: add storage-class attribute declarations in colorprocs.h

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

common: include config.h in colorprocs.h to get GVDLL in autotools builds

An upcoming commit that corrects the storage-class attribute
declarations in colorprocs.h would otherwise cause this error:

/usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: ../../lib/sparse/.libs/libsparse_C.a(DotIO.o): in function `Import_coord_clusters_from_dot':
/home/magja/graphviz/lib/sparse/DotIO.c:577: undefined reference to `colorxlate'

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

Merge branch 'smattr/a45c8ab0-d6e4-4cb7-8f28-61abb60c7359' into 'main'

migrate some more bool arrays to bit arrays

See merge request graphviz/graphviz!2356

Merge branch 'remove-gvcint-h-symbols-from-gvc-def-file' into 'main'

Remove gvcint.h symbols from gvc.def

See merge request graphviz/graphviz!2371

gvc: remove now obsolete exporting of gvcint.h symbols from gvc.def

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvc: add storage-class attribute declarations in gvcint.h

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

extract_adjacency: use bit arrays instead of bool arrays for neighbours lists

This reduces memory usage of this function.

sgd_graph: use 'size_t' for 'sources' and 'targets'

Following on from 1c42bb82cd73cb45d3abee845c21819add00d982, this uses a more
appropriate type for these members, permitting collections that exceed 2³² - 1
and avoiding sign conversion issues.

Merge branch 'smattr/FF411856-89EB-4528-B888-34D18055BAC2' into 'main'

osage: fix some function prototyping

See merge request graphviz/graphviz!2357

prototype osage_layout and osage_cleanup in osage.h

These are the only functions lib/osage exports. Yet neither were prototyped in a
header. It seems like this header could now be #included in
plugin/neato_layout/gvlayout_neato_layout.c, removing the need for that file to
prototype these functions. Except that it gives slightly different prototypes
for them.

remove osage prototypes of unimplemented functions

Merge branch 'smattr/47AD173A-739B-4728-A786-2C40BEE7D75C' into 'main'

some neatogen linkage clean up

See merge request graphviz/graphviz!2354

move_node: make a static function

This function is not used outside of stuff.c.

make_spring: make a static function

This function is not used outside of stuff.c.

heapup: make a static function

This function is not used outside of stuff.c.

heapdown: make a static function

This function is not used outside of stuff.c.

fpow32: make a static function

This function is not used outside of stuff.c.

remove unused 'final_energy'

distvec: make a static function

This function is not used outside of stuff.c.

D2E: make a static

This function is not used outside of stuff.c.

choose_node: make a static function

This is not used outside of stuff.c.

remote prototype for unimplemented 'avoid_cycling'

Merge branch 'smattr/e18d313d-2db6-47c6-8bc4-3f8a7714252f' into 'main'

fix: configure output when Lefty is disabled

See merge request graphviz/graphviz!2360

Merge branch 'remove-gvio-h-symbols-from-gvc-def-file' into 'main'

Remove gvio.h symbols from gvc.def

See merge request graphviz/graphviz!2368

fix: configure output when Lefty is disabled

Commit 1a4ad00b59cc1e1e9302fcbd1a205495ee59cab5 disabled Lefty by default in the
Autotools build system but accidentally made the configure output say nothing
next to "Lefty: " when Lefty is disabled.

gvc: remove now obsolete exporting of gvio.h symbols from gvc.def

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvc: correct storage-class attribute declarations in gvio.h

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

Merge branch 'remove-gvplugin_loadimage-h-symbols-from-gvc-def-file' into 'main'

Remove gvplugin_loadimage.h symbols from gvc.def

See merge request graphviz/graphviz!2367

gvc: remove now obsolete exporting of gvplugin_loadimage.h symbols from gvc.def

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvc: correct storage-class attribute declarations in gvplugin_loadimage.h

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

Merge branch 'remove-timing.h' into 'main'

Remove timing.h

See merge request graphviz/graphviz!2363

common: remove now unused timing.h

common: autotools: remove now unused timing.h

tred: include common/utils.h instead of duplicate defs from common/timing.h

tred: rename DIST macro to avoid shadowing

The next commit in this series will include types.h which transitively
includes geom.h which has another DIST macro defined.

Without this change, this error would occur:

FAILED: cmd/tools/CMakeFiles/tred.dir/tred.c.obj
C:\tools\msys64\mingw64\bin\ccache.exe  gcc -DDEMAND_LOADING=0 -DDIGCOLA -DENABLE_LTDL -DGVDLL -DORTHO -DSFDP -IC:/Users/magja/graphviz/build -IC:/Users/magja/graphviz/lib -IC:/Users/magja/graphviz/cmd/tools -IC:/Users/magja/graphviz/build/cmd/tools -IC:/Users/magja/graphviz/lib/cdt -IC:/Users/magja/graphviz/lib/cgraph -IC:/Users/magja/graphviz/lib/common -IC:/Users/magja/graphviz/lib/gvc -IC:/Users/magja/graphviz/lib/pack -IC:/Users/magja/graphviz/lib/pathplan -Werror -Wall -std=gnu99 -MD -MT cmd/tools/CMakeFiles/tred.dir/tred.c.obj -MF cmd\tools\CMakeFiles\tred.dir\tred.c.obj.d -o cmd/tools/CMakeFiles/tred.dir/tred.c.obj -c C:/Users/magja/graphviz/cmd/tools/tred.c
C:/Users/magja/graphviz/cmd/tools/tred.c:38: error: "DIST" redefined [-Werror]
   38 | #define DIST(ninfo,n) (ninfo[AGSEQ(n)].dist)
      |
In file included from C:/Users/magja/graphviz/lib/common/types.h:31,
                 from C:/Users/magja/graphviz/cmd/tools/tred.c:24:
C:/Users/magja/graphviz/lib/common/geom.h:53: note: this is the location of the previous definition
   53 | #define DIST(p,q)  (sqrt(DIST2((p),(q))))
      |

tred: MSBuild: add pathplan include directory

An upcoming commit that changes tred.c to include utils.h instead of
timing.h would otherwise cause this error:

C:\Users\magja\graphviz\lib\common\types.h(33,10): fatal error C1083: Cannot open include file: 'pathgeom.h': No such file or directory

tred: MSBuild: add gvc include directory

An upcoming commit that changes tred.c to include utils.h instead of
timing.h would otherwise cause this error:

C:\Users\magja\graphviz\lib\common\types.h(32,10): fatal error C1083: Cannot open include file: 'gvcext.h': No such file or directory

tred: autotools: link to the shared gvc lib instead of the static common lib

The common lib is built into the gvc lib.

An upcoming commit that changes tred.c to include utils.h instead of
timing.h would otherwise cause these errors:

/usr/lib/gcc/x86_64-pc-cygwin/11/../../../../x86_64-pc-cygwin/bin/ld: tred.o: in function `process':
/cygdrive/c/GitLab-Runner/builds/magjac/graphviz/cmd/tools/tred.c:296: undefined reference to `__imp_start_timer'
/usr/lib/gcc/x86_64-pc-cygwin/11/../../../../x86_64-pc-cygwin/bin/ld: /cygdrive/c/GitLab-Runner/builds/magjac/graphviz/cmd/tools/tred.c:299: undefined reference to `__imp_elapsed_sec'

tred: CMake: link to the gvc lib instead of the common lib

The common lib is a static lib which is built into the shared gvc lib.

An upcoming commit that changes tred.c to include utils.h instead of
timing.h would otherwise cause these errors:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: cmd/tools/CMakeFiles/tred.dir/tred.c.obj:tred.c:(.text+0xb47): undefined reference to `__imp_start_timer'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: cmd/tools/CMakeFiles/tred.dir/tred.c.obj:tred.c:(.text+0xb7a): undefined reference to `__imp_elapsed_sec'

Merge branch 'remove-pointset-h-symbols-from-gvc-def-file' into 'main'

Remove pointset.h symbols from gvc.def

See merge request graphviz/graphviz!2365

common: remove now obsolete exporting of pointset.h symbols from gvc.def

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.