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.

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'

gvpack cloneGraph: [nfc] replace a boolean with a C99 bool

bend: [nfc] remove unnecessary parens

limitBoxes: [nfc] abbreviation some accumulation

Merge branch 'smattr/03D71147-1B60-4F43-83A3-9DD0DB36BFFC' into 'main'

mingle: fix mismatched format string to 'agerr'

See merge request graphviz/graphviz!2330

mingle: fix mismatched format string to 'agerr'

Based on line 395 of this same file, it is clear this is what this code was
intended to look like. It is unclear to me why this was not caught by the GNU
`printf` annotations on `agerr`.

Merge branch 'smattr/271D49D3-BD64-4100-9DED-91D288393801' into 'main'

makeStraightEdge: remove array allocation optimization

See merge request graphviz/graphviz!2342

makeStraightEdge: remove array allocation optimization

This code had two alternative paths, one that used a heap-allocated array for
edge counts above 20 and one that used a stack-allocated array for edge counts
of 20 or less. The latter was an optimization to avoid heap allocation for
smaller graphs. Modern dynamic allocators are fast enough that this kind of
optimization is no longer relevant.

Merge branch 'smattr/BCD82356-CBC7-4DAA-9AA9-4FE6D9B9A55F' into 'main'

upgrade Pylint

See merge request graphviz/graphviz!2339

simplify test_1855 call to 'dot'

Using the wrapper added in 297ed0bc7b61b2a6616c335726334836c8ffcc10, we can
write this code more simply and concisely.

upgrade Pylint

use Python context manager idiom when reading and writing files

Newer versions of Pylint suggest this.

use Python 'subprocess.Popen' as a context manager

Newer versions of Pylint suggest this.

rtest.py: remove use of 'f3' as a global

This allows us to convert the use of this file to a more idiomatic `with` block.

remove an unnecessary Pylint 'global' statement

Newer versions of Pylint identify this.

specify an explicit mode and UTF-8 encoding in Python open calls with text mode

Newer versions of Pylint warn about this.

explicitly specify latin-1 encoding when dealing with PS files in tests

PostScript files are encoded in a character set called CCSID 1276.¹ This code
was working despite not specifying an encoding because the default encoding on
each platform coincidentally lines up with that used in their sample files. More
recent versions of Pylint warn that encoding should always be specified in
`open` calls, and attempting to use either ASCII or UTF-8 encoding fails. The
Linux sample files contain invalid ASCII bytes and the Windows sample files
contain invalid UTF-8 bytes. Using latin-1, which seems to be the closest
available encoding to CCSID 1276, works cross platform.

In future, this code should perhaps be adapted to do I/O in binary mode, thus
avoiding any encoding concerns.

¹ https://en.wikipedia.org/wiki/PostScript_Standard_Encoding

convert some legacy Python '.format' and string addition to f-strings

Newer versions of Pylint warn about this.

use 'run' instead of 'Popen' where more concise in test code

Not only does this lead to more concise and efficient code, this avoids some
upcoming warnings in newer Pylint releases.

Merge branch 'smattr/7042B009-FC97-470F-9196-E28ED6BEB967' into 'main'

add Fedora 35 testing in CI

See merge request graphviz/graphviz!2340

add Fedora 35 testing in CI

Related to #2163.

remove Fedora 33 Docker file

2a70afbdf049d5aa93470dcad5cf03c60841b6b3 removed used of this file and should
have removed the file itself as well.

Merge branch 'remove-lab-gamut-def-file' into 'main'

Remove lab_gamut.def file

See merge request graphviz/graphviz!2337

lib/edgepaint: remove obsolete lab_gamut.def file

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

lib/edgepaint: add support for the dllexport storage-class attribute in lab_gamut.h

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

This removes the need to use a .def file.

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

lib/edgepaint: MSBuild: define GVDLL in edgepaintlib project file

An upcoming commit will change lab_gamut.h to use the GVDLL symbol to
enable storage-class attributes.

This change ensures that the lab_gamut symbols will continute to be
declared with the dllimport attribute.

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

lib/edgepaint: MSBuild: define GVDLL in lab_gamut project file

An upcoming commit will change lab_gamut.h to use the GVDLL symbol to
enable storage-class attributes.

This change ensures that the lab_gamut symbols will be defined with
the dllexport attribute, which will eliminate the need for a .def
file.

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

Merge MR !2331

CMake: propagate include flag for harfbuzz on Windows

Pango >=1.44.1 changed the way harfbuzz is included
xref: https://github.com/GNOME/pango/commit/a7b17aa2

This is not required on *nix because the PkgConfig CMake module
already takes care of this. For example, on Ubuntu it adds:
-I/usr/include/harbuzz to PangoCairo_CFLAGS

Merge branch 'smattr/A6C297DD-D456-46E0-99BD-E1043C7F5AB1' into 'main'

ci: standardize shell scripts on the same e,u,x,o options

See merge request graphviz/graphviz!2336

ci: standardize shell scripts on the same e,u,x,o options

All these scripts have the same expectation, of failing on anything unexpected.
So setting the same options the same way everywhere leads to more consistency
and predictability.

ci: apply '${ID_LIKE:-}' pattern from build script to install

This allows in future enabling `set -u` in this script.