API BREAK: use C99 bools for 'port' fields instead of booleans

API BREAK: use a C99 bool for 'isOrtho' instead of a boolean

API BREAK: use a C99 bool for 'ignoreSwap' instead of a boolean

API BREAK: return a C99 bool from 'swapEnds instead of a boolean

API BREAK: return a C99 bool from 'splineMerge' instead of a boolean

API BREAK: return a C99 bool from 'insidefn' instead of a boolean

Merge branch 'smattr/449D4938-41BE-4359-B1B0-6C19D1570E24' into 'main'

some Python tweaks to test code

See merge request graphviz/graphviz!2398

shell-quote commands echoed during testing

It is not typical that testing paths will contain things like spaces and other
characters that require quoting. But this change goes the extra mile to make
sure these echoed commands are always copy-pastable to re-execute them.

run_c: [nfc] factor out an array addition

This will make an upcoming commit clearer and more concise.

run_c: check return code of compiled program before returning

Every call to `run_c` was immediately checking that the compiled program
succeeded. So by moving the check inside `run_c` itself we can write more
concise test cases.

Merge branch 'smattr/5dbb61ed-6813-47f4-918d-6a6047f05fa9' into 'main'

simplification and reduction of macros

See merge request graphviz/graphviz!2402

remove no-op 'agattrrec' casts

remove no-op 'agdatadict' casts

API BREAK: remove 'NIL' macros

remove use of 'NIL'

use C99 initialization to simplify 'Dtdisc_t' definitions

API BREAK: remove 'NOT' macro

Merge branch 'smattr/7e470f6a-2599-44f5-92e8-0d1f12a223ff' into 'main'

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

See merge request graphviz/graphviz!2401

edge_compatibility_full: use a C++ bool for 'flipped'

Gitlab: #2154

edge_compatibility: use a C++ bool for 'flipped'

Gitlab: #2154

remove unused 'pedge_export_mma'

remove unused 'pedge_realloc'

force_directed_edge_bundling: remove useless '1*'

force_directed_edge_bundling: remove manual memory management of 'force_a'

Gitlab: #2154

force_directed_edge_bundling: remove manual memory management of 'force_t'

Gitlab: #2154

remove unused 'pedge_flip'

lib/mingle/edge_bundling.cpp: replace 'MIN' with 'std::min'

Gitlab: #2154

lib/mingle/edge_bundling.cpp: replace 'MAX' with 'std::max'

Gitlab: #2154

lib/mingle: move 'edge_bundling.c' into C++

This is a straight rename. No C++-specific changes yet.

Github: #2154

force_directed_edge_bundling: suppress -Wunused-parameter warning

lib/mingle/edge_bundling.c: explicitly cast the results of 'realloc'

This has no effect in C, but this is pre-emptively squashing some warnings that
appear when moving this file to C++.

Gitlab: #2154

lib/mingle/edge_bundling.c: explicitly cast the results of 'malloc'

This has no effect in C, but this is pre-emptively squashing some warnings that
appear when moving this file to C++.

Gitlab: #2154

Merge branch 'smattr/36cf258d-8d34-4eb5-a1f1-601aab72970e' into 'main'

more boolean → C99 bool modernization, and a bitarray optimization

See merge request graphviz/graphviz!2399

standardize on C99 bool literals for calling 'agbindrec'

gvrender_core_pic.c: use a C99 bool for global 'onetime'

use C99 bool literals to set 'polygon_t.regular'

init_nop: use a C99 bool for local 'didShift'

implement Small String Optimization in the bitarray API

Following the prior commit, this further reduces memory usage. On e.g. x86-64,
arrays of ≤ 64 bits can be managed fully on the stack.

simplify bitarray API to assume arrays are only resized at construction time

The bit array API was written to allow arbitrary growing or shrinking of an
array after its construction, while maintaining its content. After fully rolling
it out, it turns out Graphviz only ever creates such arrays once at a fixed size
and then uses them until free. Carrying a more generic API than was necessary
had a number of negative consequences:

  1. `bitarray_resize` was more complicated than it needed to be, in order to
     cope with an arbitrary initial size of the input array.

  2. `bitarray_resize` used `realloc;memset` instead of the more efficient
     `calloc` because it was assuming the caller needed to maintain the original
     content.

  3. `bitarray_resize` performed a “loose” allocation with the capacity of the
     backing buffer larger than the size of the array itself, assuming that
     amortizing allocation costs across repeated resizes was a relevant concern.
     Dropping this and making the allocation “tight” not only reduces heap
     pressure, but allows dropping the `.capacity` member.

  4. `bitarray_array_resize_or_exit` had a more awkward calling convention than
     necessary, as exemplified by how it is streamlined in this commit.

As such, this change not only simplifies the code but also reduces memory usage.

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.