graph_sgd: track number of nodes as a 'size_t' instead of 'int'

This squashes a number of compiler warnings and allows this struct to deal with
> 2³² - 1 nodes in future. Dealing with this number of nodes now is still not
possible due to locations where conversion back to `int` is necessary. In
future the source and target index arrays in this struct should probably be
converted to `size_t` too, but that was not done in this commit as it involves
more invasive changes.

introduce an abstraction for compacted boolean arrays

Merge branch 'smattr/F1948FD5-8F64-4412-8925-3321F5EAC143' into 'main'

misc gvplugin.c clean up

See merge request graphviz/graphviz!2343

gvplugin.c: remove unnecessary parens

gvplugin.c: fix comment typos

Merge branch 'fix-neatogen-lib-warnings' into 'main'

Fix storage-class related warnings i the neatogen lib

See merge request graphviz/graphviz!2352

set NEATOGEN_EXPORTS in sfdp.vcxproj

Removes 6 warnings like the one below:

LINK : warning LNK4217: symbol '_neato_init_node' defined in 'neatogen.lib(neatoinit.obj)' is imported by 'sfdp.lib(sfdpinit.obj)' in function '_sfdp_layout' [C:\Users\magja\graphviz\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

set NEATOGEN_EXPORTS in gvtwopigen.vcxproj

Removes 4 warnings like the one below:

LINK : warning LNK4217: symbol '_neato_init_node' defined in 'neatogen.lib(neatoinit.obj)' is imported by 'twopigen.lib(twopiinit.obj)' in function '_twopi_init_graph' [C:\Users\magja\graphviz\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

set NEATOGEN_EXPORTS in gvfdpgen.vcxproj

Removes 18 warnings like the one below:

LINK : warning LNK4217: symbol '_spline_edges1' defined in 'neatogen.lib(neatosplines.obj)' is imported by 'fdpgen.lib(layout.obj)' in function '_fdp_layout' [C:\Users\magja\graphviz\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

set NEATOGEN_EXPORTS in gvcircogen.vcxproj

Removes 4 warnings like the one below:

LINK : warning LNK4217: symbol '_neato_init_node' defined in 'neatogen.lib(neatoinit.obj)' is imported by 'circogen.lib(circularinit.obj)' in function '_circo_init_graph' [C:\Users\magja\graphviz\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

set NEATOGEN_EXPORTS in osage.vcxproj

Removes 6 warnings like the one below:

LINK : warning LNK4217: symbol '_neato_init_node' defined in 'neatogen.lib(neatoinit.obj)' is imported by 'osage.lib(osageinit.obj)' in function '_layout' [C:\Users\magja\graphviz\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

Merge branch 'remove-useless-exporting-of-symbols-in-gvc-def-file' into 'main'

Remove useless exporting of symbols in gvc.def file

See merge request graphviz/graphviz!2348

Merge branch 'smattr/9e8bb9bf-1a04-485d-ae0f-4920031a6a55' into 'main'

more boolean → bool modernization

See merge request graphviz/graphviz!2350

endpath: [nfc] take a C99 bool parameter instead of a boolean

remove lib/pack/ptest.c

This code has clearly bitrotted some time ago. E.g. it calls `graph_init` with
one parameter, but this function takes two parameters.

graph_init: [nfc] take a C99 bool parameter instead of a boolean

shape_clip0: [nfc] take a C99 bool parameter instead of a boolean

bezier_clip: [nfc] take a C99 bool parameter instead of a boolean

beginpath: [nfc] use a C99 bool for a parameter instead of a boolean

postproc.c: [nfc] use a C99 bool for 'Flip' instead of a boolean

input.c: [nfc] remove unnecessary parens

htmlmap_data_t: [nfc] use a C99 bool instead of a boolean

findStopColor: [nfc] use a C99 bool return type instead of boolean

Merge branch 'smattr/95e25ecc-fefe-433a-9c35-6909841b342b' into 'main'

some dijkstra clean up

See merge request graphviz/graphviz!2349

dijkstra.c: reflow some text

dijkstra.c: use standard 'FLT_MAX' instead of 'MAXFLOAT'

This code is simply relying on some initial value that is “larger than any
reasonable result from the following computation.” So `FLT_MAX` is clearer and
more portable.

dijkstra_bounded: remove an allocator optimization

Similar to the prior commit, the previous code was actually a _deoptimization_.

dijkstra: remove an allocator optimization

This function was using a static local for the index array under the assumption
that resizing an existing dynamically allocated buffer is cheaper than
allocating a new one. This is generally not true in this use case. Calling
`realloc` here was (incorrectly) telling the allocator the caller needed to
retain the prior content of the array. In other words, this was actually a
_deoptimization_. Undoing this and letting the allocator more accurately
perceive that this is a transient array allocation results in (1) more efficient
behavior and (2) less unnecessarily retained memory.

dijkstra.c: make 'DIST_MAX' a 'DistType' instead of 'double'

It is not clear to me why this value was type cast to a `double` when all uses
of it either explicitly or implicitly cast it back to `DistType`.

dijkstra: remove use of raw 'INT_MAX'

This code was relying on this value being the same as `DIST_MAX`. `DIST_MAX` is
currently set to `INT_MAX`, but using `INT_MAX` directly here implied it was
safe to change the definition of `DIST_MAX` without updating this location,
which was not true.

remove NONCORE-guarded code

Nothing in the build systems ever define this.

gvc: remove obsolete exporting of globals.h symbols from gvc.def

The symbols in common/globals.h have correct storage-class attributes
since commit e01f4a96848807ed07a00cee5351ec8b4d4dde49.

Note that the following symbols in globals.h were missing in the
gvc.def file:

Gvimagepath
G_gradientangl
N_margin
N_imagepos
N_gradientangle
E_fillcolor

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

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

An upcoming commit that removes the common/globals.h symbols from the
gvc.def file would otherwise cause errors like this:

gvmap.obj : error LNK2001: unresolved external symbol _Verbose [C:\Users\magja\graphviz\cmd\gvmap\gvmap.vcxproj]

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

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

An upcoming commit that removes the common/globals.h symbols from the
gvc.def file would otherwise cause errors like this:

minglemain.obj : error LNK2001: unresolved external symbol _Files [C:\Users\magja\graphviz\cmd\mingle\mingle.vcxproj]

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

lib/mingle: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit that removes the common/globals.h symbols from the
gvc.def file would otherwise cause errors like this:

minglelib.lib(edge_bundling.obj) : error LNK2001: unresolved external symbol _Verbose [C:\Users\magja\graphviz\cmd\mingle\mingle.vcxproj]

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

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

An upcoming commit that removes the common/globals.h symbols from the
gvc.def file would otherwise cause errors like this:

sparse.lib(DotIO.obj) : error LNK2001: unresolved external symbol _Verbose

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

sparse: correct storage-class declaration of Verbose when STANDALONE

An upcoming commit that removes the common/globals.h symbols from the
gvc.def file would otherwise cause errors like this:

sparse.lib(mq.obj) : error LNK2001: unresolved external symbol _Verbose [C:\Users\magja\graphviz\cmd\gvmap\gvmap.vcxproj]

This is for the STANDALONE case when Verbose is defined by the
application that uses lib/sparse rather than using the definition in
lib/common/globals.h. In this case, Verbose is not exported from any
library.

From lib/sparse/general.h:

/* Applications that do not use the common library can define STANDALONE
 * to get definitions/definitions that are normally provided there.
 * In particular, note that Verbose is declared but undefined.
 */

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

gvc: remove obsolete exporting of memory.h symbols from gvc.def

The symbols in common/memory.h have correct storage-class attributes
since commit a07297906fc7c90c3ca45d292412ddbd29b7f5bc.

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

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

An upcoming commit that removes the common/memory.h symbols from the
gvc.def file would otherwise cause errors like this:

matrix_market.obj : error LNK2019: unresolved external symbol _gmalloc referenced in function _SparseMatrix_import_matrix_market

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

gvc: remove useless exporting of intset.h symbol openIntSet from gvc.def

The openIntSet function in common/intset.h is only used in the common
and ortho libraries which are both static libraries that are built
into the gvc dynamic-link library.

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

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

Remove pathplan.def file

See merge request graphviz/graphviz!2338

pathplan: MSBuild: define GVDLL

pathplan: remove now empty and obsolete pathplan.def file

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

pathplan: MSBuild: remove use of now empty and obsolete pathplan.def file

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

pathplan: autotools: remove use of now empty and obsolete pathplan.def file

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

pathplan: CMake: remove use of now empty and obsolete pathplan.def file

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

Revert "pathplan: autotools: use pathplan.def"

This reverts commit b63144316c65e992712ed9d6385af59ab5a9bc7e.

This file is now empty and obsolete.

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

pathplan: remove now obsolete exporting of vispath.h symbols from pathplan.def

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

pathplan: add storage-class attribute declarations in vispath.h

This will allow removing the vispath.h symbols from the pathplan.def
file in an upcoming commit.

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

pathplan: remove now obsolete exporting of pathgeom.h symbols from pathplan.def

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

pathplan: add storage-class attribute declarations in pathgeom.h

This will allow removing the pathgeom.h symbols from the pathplan.def
file in an upcoming commit.

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

common: add explicit include of pathgeom.h in emit.c

emit.c explicitly uses e.g. freePath from the public pathgeom.h header
file, so even if it gets it implicitly through another header file,
including it explicitly is the right thing to do.

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

pathplan: remove now obsolete exporting of tri.h symbols from pathplan.def

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

pathplan: add storage-class attribute declarations in tri.h

This will allow removing the tri.h symbols from the pathplan.def file in an
upcoming commit.

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

pathplan: remove now obsolete exporting of pathplan.h symbols from pathplan.def

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

pathplan: add storage-class attribute declarations in pathplan.h

This will allow removing the pathplan.h symbols from the pathplan.def
file in an upcoming commit.

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

pathplan: remove now obsolete exporting of vis.h symbols from pathplan.def

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

pathplan: add support for the dllexport storage-class attribute in vis.h

Also, storage-class attributes are now added only when the GVDLL
symbol is defined.

This will allow removing the viz.h symbols from the pathplan.def file
in an upcoming commit.

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

pathplan: remove now obsolete exporting of pathutil.h symbols from pathplan.def

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

pathplan: add support for the dllexport storage-class attribute in pathutil.h

Also, storage-class attributes are now added only when the GVDLL
symbol is defined.

This will allow removing the pathutil.h symbols from the pathplan.def
file in an upcoming commit.

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

pathplan: add missing storage-class attribute declaration for in_poly

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

neatogen: remove duplicate declaration of in_poly in neatosplines.c

The in_poly function is already declared in lib/pathplan/pathutil.h.

Without this change, an upcoming commit that adds dllimport/dllexport
storage-class attribute declarations in lib/pathplan/pathutil.h would
cause the following warning for the duplicate definition:

C:\Users\magja\graphviz\lib\neatogen\neatosplines.c(27,48): warning C4273: 'in_poly': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\neatogen\neatogen.vcxproj]

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

pathplan: remove exporting of solve* symbols from pathplan.def

These functions are not used outside the pathplan library and are not
part of the public API.

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

Merge branch 'smattr/26E51B39-300A-4245-A02E-A0607F0E3141' into 'main'

more boolean → bool modernization

See merge request graphviz/graphviz!2347

flat_search: [nfc] use a C99 bool instead of boolean

merge_trapezoids: [nfc] use a C99 bool instead of boolean

makeBottomFlatEnd: [nfc] use a C99 bool instead of boolean

makeFlatEnd: [nfc] use a C99 bool instead of boolean

poly_gencode: [nfc] use a C99 bool instead of boolean

generateNonoverlapConstraints: [nfc] use a C99 bool instead of boolean

IMDS_given_dim: [nfc] use a C99 bool instead of boolean

constrained_majorization_vpsc: [nfc] use a C99 bool instead of boolean

stress_majorization_with_hierarchy: [nfc] use a C99 bool instead of boolean

ptest_nodesize: remove unused 'flip' parameter

Merge branch 'smattr/7ba222c3-1fe4-414e-a130-26d504d46cef' into 'main'

remove dead code

See merge request graphviz/graphviz!2346

remove unused 'glCompSelectionBox'

remove unused 'GetFixedOGLPoslocal'

remove unused 'GetFixedOGLPos'

remove unused 'seedMode'

remove unused 'nodeset' abstraction

remove unused 'size_edgelist'

remove prototype of unimplemented 'cmpNodeDegree'

remove prototypes of unimplemented ag callback functions

remove prototypes of unimplemented object set management functions

remove declaration of undefined 'AgAttrdisc'

remove unused 'NIL macros

Note that copies of these still exist in the shipped cgraph.h.

remove C++ guards on hashkey.h

These are no longer necessary now that this header contains no function
prototypes.

remove prototype for unimplemented 'strkey'

remove prototype for unimplemented 'html_inside'

remove unused 'colorCvt'

Merge branch 'smattr/2D1C5E3F-6FEC-4BF0-9291-DD8A9DBD3C12' into 'main'

more boolean → bool modernization

See merge request graphviz/graphviz!2344

constrained_majorization_new_with_gaps: [nfc] boolean → C99 bool

constrained_majorization_gradient_projection: [nfc] another boolean → C99 bool

constrained_majorization_gradient_projection: [nfc] boolean → C99 bool

constrained_majorization_new: [nfc] use a C99 bool instead of a boolean

power_iteration/iterativePCA_1D: [nfc] return C99 bools instead of boolean

stress_majorization_with_hierarchy: [nfc] replace a boolean with a C99 bool

user_spline: [nfc] use a C99 bool instead of boolean for 'warned'

gvconfig_libdir: [nfc] use a C99 bool instead of boolean for 'dirShown'

gvpack: [nfc] use a C99 bool instead of boolean for 'doPack'