cluster_map: replace boolean array 'assigned' with a bit array

This is more memory efficient.

dijkstra_bounded: replace boolean array 'node_in_neighborhood' with a bit array

This is more memory efficient. This also undoes the retention and reuse of this
array making its live range more obvious to the compiler.

Merge branch 'add-mingw-static-autotools-build-jobs' into 'main'

Unfify the last storage class attribute declarations and add MinGW static autotools builds

Closes #2173

See merge request graphviz/graphviz!2397

ci: add MinGW32 static autotools build jobs

ci: add MinGW64 static autotools build jobs

ci/build.sh: autotools: add support building using autogen in static builds

ci: add support for configure options in out-of-source-configure.sh

gvpack: use GVDLL instead of _WIN32 to enable dllimport storage-class attribute in gvpack.c

Fixes these errors when building statically with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: gvpack_static-gvpack.o: in function `main':
C:\Users\magja\graphviz\cmd\tools/gvpack.c:857: undefined reference to `__imp_gvplugin_neato_layout_LTX_library'
collect2.exe: error: ld returned 1 exit status
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: gvpack-gvpack.o: in function `main':
C:\Users\magja\graphviz\cmd\tools/gvpack.c:857: undefined reference to `__imp_gvplugin_neato_layout_LTX_library'

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

gdtclft: use only GVDLL to enable dllexport storage-class attribute for Gdtclft_SafeInit

Instead of using __CYGWIN__ and __WIN32__, rely on that GVDLL is
correctly set for all platforms.

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

gdtclft: use only GVDLL to enable dllexport storage-class attribute for Gdtclft_Init

Instead of using __CYGWIN__ and __WIN32__, rely on that GVDLL is
correctly set for all platforms.

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

Merge branch 'smattr/c144f079-34fb-4177-84dd-821809ad0a39' into 'main'

more boolean → bool modernization

See merge request graphviz/graphviz!2396

gvprintnum: use C99 bools for locals 'showzeros', 'negative'

gvdevice_finalize: use a C99 bool for local 'finalized_p'

setbounds: use C99 bools for locals 'onleft', 'onright'

addLabelBB: use a C99 bool for parameter 'flipxy'

safefile: use a C99 bool for local 'onetime'

shape_clip: use a C99 bool for local 'left_inside'

point_gencode: use a C99 bool for local 'filled'

poly_gencode: use a C99 bool for local 'pfilled'

compassPort: use a C99 bool for local 'defined'

compassPort: use a C99 bool for local 'clip'

compassPort: use a C99 bool for local 'dyna'

compassPort: use a C99 bool for local 'constrain'

obj_state_s: set 1-bit bitfield with with C99 bool literals

nodeIntersect: take a C99 bool for 'explicit_tooltip'

nodeIntersect: take a C99 bool for 'explicit_iurl'

handle_keypress: return a C99 bool instead of boolean

Merge branch 'install-cygwin-from-cygwin.com' into 'main'

CI: install Cygwin from cygwin.com instead of with choco

See merge request graphviz/graphviz!2395

ci: install cygwin from cygwin.com instead of with choco

Merge branch 'smattr/9e86e6dc-0b65-411e-af31-5cb7d722ec63' into 'main'

move lib/mingle/nearest_neighbor_graph.c into C++

See merge request graphviz/graphviz!2384

nearest_neighbor_graph_ann: take array parameters as vector instead of C arrays

Now that all calls to this function are in C++, there is no need to degrade to C
data types when calling into it.

Gitlab: #2154

nearest_neighbor_graph_ann: remove forcing of C linkage

The only calls into this function are from C++ code now, so there is no need to
require C calling conventions.

Gitlab: #2154

nearest_neighbor_graph: remove manual memory management

Gitlab: #2154

nearest_neighbor_graph_ann: remove a level of indirection on array parameters

None of these parameters are updated during `nearest_neighbor_graph_ann`, so
taking them by pointer is irrelevant.

move lib/mingle/nearest_neighbor_graph.c to C++

This is straight renaming; no C++ adaptations made yet.

Gitlab: #2154

nearest_neighbor_graph: cast the result of calls to 'malloc'

This pre-emptively suppresses some `-fpermissive` warnings when transitioning
this code to C++ in an upcoming commit.

Gitlab: #2154

Merge branch 'unify-storage-class-declarations-in-gvc++' into 'main'

Unify storage class declarations in gvc++

See merge request graphviz/graphviz!2394

gvc++: [nfc] align storage-class declarations to Graphviz standard in GVRenderData.h

gvc++: use GVDLL instead of _WIN32 to control storage-class attributes in GVRenderData.h

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

gvc++: [nfc] align storage-class declarations to Graphviz standard in GVLayout.h

gvc++: use GVDLL instead of _WIN32 to control storage-class attributes in GVLayout.h

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

gvc++: [nfc] align storage-class declarations to Graphviz standard in GVContext.h

gvc++: use GVDLL instead of _WIN32 to control storage-class attributes in GVContext.h

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

Merge branch 'smattr/F5A2D01F-EFF0-42DC-81F5-2985BE24E104' into 'main'

Revert Perl CFLAG mangling in tclpkg

Closes #2177

See merge request graphviz/graphviz!2388

Revert "sed cleanup of CFLAGS from perl which break on opencsw - should be harmless elsewhere"

This reverts commit a750d28463a9248fe5a42d3dae5768d38853d837. It is unclear to
me exactly what flag the original code was attempting to suppress, but this
regex is too broad. The specific case causing problems right now is that it
incorrectly mangled `-mno-xop -mbmi` into `-mno-mbmi`. Given the original code
appears to have been working around a 2011 OpenCSW packaging mistake, lets
assume this was fixed sometime in the last decade and see if anyone downstream
complains when we remove the work around.

Gitlab: fixes #2177

Revert "clang doesn't support -fstack-clash-protection"

This reverts commit e50fd757a297e9917f801cd2781559535f03ce13. Clang added
`-ftack-clash-protection` in 2020.¹

Gitlab: #2177

¹ https://reviews.llvm.org/D68720

Merge branch 'unify-storage-class-declarations-in-plugins' into 'main'

Unify storage class declarations in plugins

See merge request graphviz/graphviz!2393

plugin/pango: use GVDLL instead of _WIN32 to enable dllexport storage-class attribute in gvplugin_pango.c

plugin/pango: [nfc] remove whitespace in preprocessor directives in gvplugin_pango.c

plugin/neato_layout: [nfc] remove whitespace in preprocessor directives in gvplugin_neato_layout.cpp

plugin/gdiplus: use GVDLL instead of _WIN32 to enable dllexport storage-class attribute in gvplugin_gdiplus.c

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

plugin/gdiplus: 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 gvplugin_gdiplus.c.

Without this change, errors like this would occur:

EXEC : error : failed to resolve gvplugin_gdiplus_LTX_library in C:\Users\magja\graphviz\Debug\Graphviz\bin\gvplugin_gdiplus.dll

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

plugin/gdiplus: [nfc] remove whitespace in preprocessor directives in gvplugin_gdiplus.cpp

plugin/gd: [nfc] align storage-class declarations to Graphviz standard in gvrender_gd.c

plugin/gd: use GVDLL instead of _WIN32 to control storage-class attributes in gvrender_gd.c

plugin/gd: [nfc] remove whitespace in preprocessor directives in gvrender_gd.c

plugin/gd: use GVDLL instead of _WIN32 to enable dllexport storage-class attribute in gvplugin_gd.c

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

plugin/gd: 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 gvplugin_gd.c.

Without this change, errors like this would occur:

EXEC : error : failed to resolve gvplugin_gd_LTX_library in C:\Users\magja\graphviz\Debug\Graphviz\bin\gvplugin_gd.dll

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

plugin/gd: [nfc] remove whitespace in preprocessor directives in gvplugin_gd.c

plugin/dot_layout: use GVDLL instead of _WIN32 to enable dllexport storage-class attribute in gvplugin_dot_layout.c

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

plugin/dot_layout: 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 gvplugin_dot_layout.c.

Without this change, errors like this would occur:

EXEC : error : failed to resolve gvplugin_dot_layout_LTX_library in C:\Users\magja\graphviz\Debug\Graphviz\bin\gvplugin_dot_layout.dll

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

plugin/dot_layout: [nfc] remove whitespace in preprocessor directives in gvplugin_dot_layout.c

plugin/core: use GVDLL instead of _WIN32 to enable dllexport storage-class attribute in gvplugin_core.c

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

plugin/core: [nfc] remove whitespace in preprocessor directives in gvplugin_core.c

Merge branch 'install-python-with-nuget' into 'main'

CI: install Python with nuget instead of with choco

See merge request graphviz/graphviz!2392

Merge branch 'smattr/DFDACC2F-624B-4360-BC9C-9B2A116EB11C' into 'main'

more boolean → modernization

See merge request graphviz/graphviz!2391

ci: install python with nuget in windows-cygwin-build-using-autogen job

ci: remove now obsolete choco cache from windows-msbuild-Win32-debug-build job

ci: remove now obsolete choco cache from windows-msbuild-Win32-release-build job

ci: remove now obsolete choco cache from windows-cmake-x64-debug-build job

ci: remove now obsolete choco cache from windows-cmake-x64-release-build job

ci: remove now obsolete choco cache from windows-cmake-Win32-debug-build job

ci: remove now obsolete choco cache from windows-cmake-Win32-release-build job

ci: install python with nuget in windows_build_definition job template

ci: remove now obsolete choco cache from windows-mingw64-cmake-build job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

ci: remove now obsolete choco cache from windows-mingw32-cmake-build job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

ci: remove now obsolete choco cache from windows-mingw64-build-step2-make job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

ci: remove now obsolete choco cache from windows-mingw64-build-step1-configure job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

ci: remove now obsolete choco cache from windows-mingw32-build-step2-make job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

ci: remove now obsolete choco cache from windows-mingw32-build-step1-configure job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

is_internal_to_cluster; return a C99 bool instead of an ambiguous int

is_nonconstraint: return a C99 bool instead of an ambiguous int

is_empty: return a C99 bool instead of an ambigous int

is_cluster: return a C99 bool instead of an ambiguous int

is_a_cluster: return a C99 bool instead of ambiguous int

This function was already morally returning a boolean value.

mapbool: take a 'const char*' parameter instead of 'char*'

This parameter is not modified internally.

mapbool: return a C99 bool instead of a boolean

mapBool: take a 'const char*' parameter instead of 'char*'

This parameter is not modified within the function.

mapBool: use C99 bools for boolean parameters

Note that this required a somewhat awkward work around in `getAdjustMode` that
was abusing being able to fit something other than a boolean into the type used
for booleans.

late_bool: use C99 bools for boolean parameters

ortho segment: remove unused 'flipped' member

node_distinct_coloring: take 'weightedQ' parameter as a C99 bool

gvmap plot_dot_map: use a C99 bool for local 'plot_polyQ'

gvprintpointflist: replace a boolean literal with C99 bool 'true'

Merge branch 'unify-storage-class-declarations-in-xdot' into 'main'

Unify storage class declarations in xdot

See merge request graphviz/graphviz!2390

xdot: [nfc] align storage-class declarations to Graphviz standard in xdot.h

xdot: [nfc] remove useless default extern declaration of functions in xdot.h

xdot: use GVDLL instead of _WIN32 to control storage-class attributes in xdot.h

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

xdot: [nfc] remove whitespace in preprocessor directives in xdot.h

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

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

Without this change, errors like this would occur in lib/gvc:

LINK : fatal error LNK1104: cannot open file 'C:\Users\magja\graphviz\Debug\Graphviz\bin\xdot.lib'
Done building project "gvc.vcxproj" -- FAILED.

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