ci/build.sh: autotools add support for CONFIGURE_OPTIONS for MinGW and Cygwin

ci/build.sh: autotools: add support for MinGW builds

neato_layout: use GVDLL instead _WIN32 to control storage-class attributes

Storage-class attributes should only be used for dynamic-link
libraries, but not for statically linked libraries.

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

plugin/neato_layout: MSBuild: define GVDLL

Without this change, the following error would occur in an upcoming
commit that changes the Windows specific storage-class attributes to
be controlled by GVDLL instead of by _WIN32:

LINK : fatal error LNK1181: cannot open input file 'gvplugin_neato_layout.lib' [C:\GitLab-Runner\builds\magjac\graphviz\cmd\tools\gvpack.vcxproj]

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

tclpathplan: autotools: add linking to tcl stubs

Fixes errors like this when using autotools with MinGW:

tclpkg\tclpathplan/tclpathplan.c:396: undefined reference to `tclStubsPtr'

tcldot: autotools: add --no-undefined also for libtcldot_builtin

Fixes this error using autotools with MinGW:

libtool:   error: can't build x86_64-w64-mingw32 shared library unless -no-undefined is specified

tcldot: autotools: add linking to tcl stubs

Fixes errors like this using autotools with MinGW:

C:\Users\magja\graphviz\tclpkg\tcldot/tcldot.c:176: undefined reference to `Tcl_InitStubs'

tclpkg/gdtclft: correct storage-class attribute declarations

Fixes these errors using autotools with MinGW:

gdtclft.c:1271:13: error: expected declaration specifiers or '...' before 'Gdtclft_Init'
 1271 | EXPORT(int, Gdtclft_Init) (interp)
      |             ^~~~~~~~~~~~
gdtclft.c:1297:13: error: expected declaration specifiers or '...' before 'Gdtclft_SafeInit'
 1297 | EXPORT(int, Gdtclft_SafeInit) (interp)
      |             ^~~~~~~~~~~~~~~~

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

common: correct storage-class attributes in memory.h

Fixes many errors like this using autotools with MinGW:

  CCLD     mm2gv.exe
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: matrix_market.o: in function `SparseMatrix_import_matrix_market':
C:\Users\magja\graphviz\cmd\tools/matrix_market.c:99: undefined reference to `__imp_gmalloc'

neatogen: add storage-class attribute declarations in neatoprocs.h

Fixes this error using autotools with MinGW:
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 `init_node_edge':
C:\Users\magja\graphviz\cmd\tools/gvpack.c:271: undefined reference to `neato_init_node'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: C:\Users\magja\graphviz\cmd\tools/gvpack.c:272: undefined reference to `user_pos'
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 `init_graph':
C:\Users\magja\graphviz\cmd\tools/gvpack.c:305: undefined reference to `init_nop'

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

sfdpgen: CMake: define NEATOGEN_EXPORTS

sfdpgen is part of the neato_layout plugin.

Fixes errors like this using CMake with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: lib/sfdpgen/libsfdpgen.a(sfdpinit.c.obj):sfdpinit.c:(.text+0xac): undefined reference to `__imp_neato_init_node'

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

twopigen: CMake: define NEATOGEN_EXPORTS

twopigen is part of the neato_layout plugin.

Fixes errors like this using CMake with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: lib/twopigen/libtwopigen.a(twopiinit.c.obj):twopiinit.c:(.text+0x142): undefined reference to `__imp_neato_init_node'

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

osage: CMake define NEATOGEN_EXPORTS

osage is part of the neato_layout plugin.

Fixes errors like this using CMake with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: lib/osage/libosage.a(osageinit.c.obj):osageinit.c:(.text+0x2ec): undefined reference to `__imp_neato_init_node'

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

neatogen: CMake: define NEATOGEN_EXPORTS

It is not automatically set for static libraries, only for shared
libraries.

Fixes errors like this using CMake with MinGW:

neatogen.lib(neatoinit.obj) : error LNK2019: unresolved external symbol __imp_circuit_model referenced in function kkNeato [C:\Users\magja\graphviz\build\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

fdpgen: CMake: define NEATOGEN_EXPORTS

fdpgen is part of the neato_layout plugin.

Fixes errors like this using CMake with MinGW:

fdpgen.lib(clusteredges.obj) : error LNK2019: unresolved external symbol __imp_Plegal_arrangement referenced in function compoundEdges [C:\Users\magja\graphviz\build\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

circogen: CMake: define NEATOGEN_EXPORTS

circogen is part of the neato_layout plugin.

Fixes errors like this using CMake with MinGW:

circogen.lib(circularinit.obj) : error LNK2019: unresolved external symbol __imp_spline_edges referenced in function circo_layout [C:\Users\magja\graphviz\build\plugin\neato_layout\gvplugin_neato_layout.vcxproj]

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

sfdpgen: autotools: define NEATOGEN_EXPORTS

sfdpgen is part of the neato_layout plugin.

Fixes these errors using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/sfdpgen/.libs/libsfdpgen_C.a(sfdpinit.o): in function `sfdp_init_node_edge':
C:\Users\magja\graphviz\lib\sfdpgen/sfdpinit.c:37: undefined reference to `__imp_neato_init_node'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/sfdpgen/.libs/libsfdpgen_C.a(sfdpinit.o): in function `tuneControl':
C:\Users\magja\graphviz\lib\sfdpgen/sfdpinit.c:248: undefined reference to `__imp_setSeed'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/sfdpgen/.libs/libsfdpgen_C.a(sfdpinit.o): in function `sfdp_layout':
C:\Users\magja\graphviz\lib\sfdpgen/sfdpinit.c:331: undefined reference to `__imp_spline_edges'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: C:\Users\magja\graphviz\lib\sfdpgen/sfdpinit.c:319: undefined reference to `__imp_spline_edges'

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

circogen: autotools: define NEATOGEN_EXPORTS

circogen is part of the neato_layout plugin.

Fixes these errors using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/circogen/.libs/libcircogen_C.a(circularinit.o): in function `circular_init_node_edge':
C:\Users\magja\graphviz\lib\circogen/circularinit.c:49: undefined reference to `__imp_neato_init_node'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/circogen/.libs/libcircogen_C.a(circularinit.o): in function `circo_layout':
C:\Users\magja\graphviz\lib\circogen/circularinit.c:249: undefined reference to `__imp_spline_edges'

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

fdpgen: autotools: define NEATOGEN_EXPORTS

fdpgen is part of the neato_layout plugin.

Fixes errors like this using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/fdpgen/.libs/libfdpgen_C.a(layout.o): in function `fdp_layout':
C:\Users\magja\graphviz\lib\fdpgen/layout.c:1101: undefined reference to `__imp_neato_set_aspect'

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

osage: autotools: define NEATOGEN_EXPORTS

osage is part of the neato_layout plugin.

Fixes these errors using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/osage/.libs/libosage_C.a(osageinit.o): in function `cluster_init_graph':
C:\Users\magja\graphviz\lib\osage/osageinit.c:69: undefined reference to `__imp_neato_init_node'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/osage/.libs/libosage_C.a(osageinit.o): in function `osage_layout':
C:\Users\magja\graphviz\lib\osage/osageinit.c:367: undefined reference to `__imp_spline_edges1'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: C:\Users\magja\graphviz\lib\osage/osageinit.c:363: undefined reference to `__imp_spline_edges0'

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

twopigen: autotools: define NEATOGEN_EXPORTS

twopigen is part of the neato_layout plugin.

Fixes errors like this when using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/twopigen/.libs/libtwopigen_C.a(twopiinit.o): in function `twopi_init_node_edge':
C:\Users\magja\graphviz\lib\twopigen/twopiinit.c:39: undefined reference to `__imp_neato_init_node'

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

neatogen: MSBuild: define NEATOGEN_EXPORTS

neatogen: autotools: define NEATOGEN_EXPORTS

lib/gvpr: autotools: fix incorrect MinGW path translation in DFLT_GVPRPATH string

Add an extra forward slash in front of $(pkgdatadir) to avoid the path
being translated to use backslashes under MSYS2/MinGW. This trick is
needed since DFLT_GVPRPATH is a string which is compiled and
backslashes in strings are interpreted as escape sequences. See
https://stackoverflow.com/a/14189687

Fixes errors like this when using autotools with MinGW:

gvpr.c:228:41: error: incomplete universal character name \U
  228 |      pathp = path = concat(DFLT_GVPRPATH, path);
      |                                         ^
gvpr.c:228:41: warning: unknown escape sequence: '\m'
gvpr.c:228:41: warning: unknown escape sequence: '\g'
gvpr.c:228:41: warning: unknown escape sequence: '\s'
gvpr.c:228:41: warning: unknown escape sequence: '\g'
gvpr.c:228:41: warning: unknown escape sequence: '\g'

autotools: add WITH_MINGW to AM_CONDITIONAL

This will allow an upcoming commit in this series to introduce
MinGW-specific functionality in the autotools build system.

pathplan: autotools: add define of PATHPLAN_EXPORTS

Fixes errors like this using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/cvt.o: in function `Pobsopen':
C:\Users\magja\graphviz\lib\pathplan/cvt.c:80: undefined reference to `__imp_visibility'

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

common: autotools: define also GVC_EXPORTS

Fixes errors like this using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/common/.libs/libcommon_C.a(arrows.o): in function `arrow_type_curve':
C:\Users\magja\graphviz\lib\common/arrows.c:696: undefined reference to `__imp_Bezier'

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

label: autotools: define GVC_EXPORTS

Fixes this error using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/label/.libs/liblabel_C.a(xlabels.o): in function `xlnew':
C:\Users\magja\graphviz\lib\label/xlabels.c:41: undefined reference to `__imp_zmalloc'

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

pack: autotools: define GVC_EXPORTS

Fixes errors like this using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/pack/.libs/libpack_C.a(pack.o): in function `fillLine':
C:\Users\magja\graphviz\lib\pack/pack.c:127: undefined reference to `__imp_addPS'

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

gvc: autotools: define GVC_EXPORTS

Fixes errors like this using autotools with MinGW:

C:\Users\magja\graphviz\lib\gvc/gvevent.c:201: undefined reference to `__imp_overlap_edge'

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

gvc: autotools: remove now obsolete linking to xdot

The core plugin that previously relied on this explicitly links to
xdot itself since commit 6d646c82d94efdfb9933328f08301f6e1600943e.

The gvc source files do not refer to xdot explicitly and any implicit
references are now resolved by the transitive dependency through
common.

Fixes these errors using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/xdot/.libs/libxdot.dll.a(d000007.o):(.text+0x0): multiple definition of `parseXDotF'; ../../lib/xdot/.libs/libxdot_C.a(xdot.o):C:\Users\magja\graphviz\lib\xdot/xdot.c:475: first defined here
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/xdot/.libs/libxdot.dll.a(d000002.o):(.text+0x0): multiple definition of `freeXDot'; ../../lib/xdot/.libs/libxdot_C.a(xdot.o):C:\Users\magja\graphviz\lib\xdot/xdot.c:927: first defined here

common: autotools: add linking to xdot

E.g. lib/common/emit.c. uses parseXDotF from lib/xdot/xdot.c.

This removes the need to link to xdot for downstream libraries that do
not explicitly use xdot.

Fixes these errors using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/common/.libs/libcommon_C.a(input.o): in function `graph_cleanup':
C:\Users\magja\graphviz\lib\common/input.c:881: undefined reference to `__imp_freeXDot'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/common/.libs/libcommon_C.a(emit.o): in function `init_xdot':
C:\Users\magja\graphviz\lib\common/emit.c:58: undefined reference to `__imp_parseXDotF'

lib: autotools: build xdot before common

E.g. lib/common/emit.c. uses parseXDotF from lib/xdot/xdot.c.

This is the first step in fixing these errors using autotools with
MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/common/.libs/libcommon_C.a(input.o): in function `graph_cleanup':
C:\Users\magja\graphviz\lib\common/input.c:881: undefined reference to `__imp_freeXDot'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../../lib/common/.libs/libcommon_C.a(emit.o): in function `init_xdot':
C:\Users\magja\graphviz\lib\common/emit.c:58: undefined reference to `__imp_parseXDotF'

The next commit in this series will add linking to xdot.

pathplan: autotools: use pathplan.def

Fixes these errors when using autotools with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/cvt.o: in function `Pobsopen':
C:\Users\magja\graphviz\lib\pathplan/cvt.c:80: undefined reference to `__imp_visibility'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/cvt.o: in function `Pobspath':
C:\Users\magja\graphviz\lib\pathplan/cvt.c:108: undefined reference to `__imp_ptVis'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: C:\Users\magja\graphviz\lib\pathplan/cvt.c:115: undefined reference to `__imp_makePath'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/inpoly.o: in function `in_poly':
C:\Users\magja\graphviz\lib\pathplan/inpoly.c:32: undefined reference to `__imp_wind'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: .libs/shortestpth.o: in function `makePath':
C:\Users\magja\graphviz\lib\pathplan/shortestpth.c:98: undefined reference to `__imp_directVis'

cgraph: autotools: add definition of export symbols

Fixes errors like this with MinGW:

lib\cgraph/apply.c:48: undefined reference to `__imp_agfstsubg'

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

cdt: autotools: define EXPORT_CDT

Fixes errors like this when using autotools with MinGW:

lib\cdt/dtclose.c:23: undefined reference to `__imp_dtview'

Also fixes warnings like:

dtview.c:91:7: warning: 'dtview' redeclared without dllimport attribute: previous dllimport ignored [-Wattributes]
       91 | Dt_t* dtview(Dt_t* dt, Dt_t* view)
          |       ^~~~~~

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

Merge branch 'smattr/6bb70c50-e9ea-453e-83e1-3cd6f7667bc3' into 'main'

lib/common warning squashing

See merge request graphviz/graphviz!2285

free_fspan: squash -Wunused-parameter warnings

free_fitem: squash -Wunused-parameter warnings

free_citem: squash -Wunused-parameter warnings

free_item: squash -Wunused-parameter warnings

free_ritem: squash -Wunused-parameter warnings

mkFont: make it more obvious to the compiler the extent of 'flags'

All calls to this function use literals that fit in an unsigned char. Squashes
a -Wconversion warning. This also introduces a new header constant,
`GV_TEXTFONT_FLAGS_WIDTH`.

stylefn: cast a bitwise not, squashing a -Wconversion warning

htmllex.c: consistently use int types for 'warn' and 'error'

Squashes a -Wconversion warning.

lexstate_t: standardize on more modern '//' comments

cellborderfn: fix a mismatched cast

Squashes a -Wsign-conversion warning.

lib/common/utils.h: clang-format

Merge branch 'smattr/26e14db7-4192-414c-9232-3bd8a4982b1a' into 'main'

more boolean modernization and a type confusion fix

See merge request graphviz/graphviz!2312

gvjobs_output_langname: return a C99 bool instead of boolean

fix misuse of 'gvjobs_output_langname'

Many callers of `gvjobs_output_langname` were written as if this function
returns `NO_SUPPORT` when given an unsupported language. But this is incorrect.
It returns `FALSE`.

tclpkg/tcldot/tcldot-graphcmd.c: [nfc] remove commented out code

gvplugin_install: use a C99 bool return instead of boolean

GVC_s.graph_sets_*: use C99 bools instead of boolean

remove unused 'GVC_s.graph_sets_rotation'

GVC_s.config_found: use a C99 bool instead of boolean

Merge branch 'smattr/6f4441fa-d5fc-4b46-abe0-963f12023272' into 'main'

gvgen warning squashing

See merge request graphviz/graphviz!2315

gvgen: remove unnecessary bracketing

gvgen init: squash -Wimplicit-fallthrough warning

gvgen init: squash -Wswitch-default warning

gvgen readPos: squash -Wconversion warning

gvgen errexit: squash -Wconversion warnings

This function is called with int values, but the values are the result of
`getopt`. `getopt` only ever returns -1 (which is handled separately) or a valid
`char` value, so the cast here is always safe and never truncating.

Merge branch 'smattr/de1695d5-8ea1-48b3-a301-2f4f0ccc2c6b' into 'main'

more warning squashing and dead code removal

See merge request graphviz/graphviz!2318

ming: remove disabled code

expr: remove disabled code

topfish: remove disabled code

glcomp: remove glcomppanel disabled code

glcomp: remove disabled glcompimage code

glcomp: remove disabled PNG code

glcomp: remove disabled prototypes

None of these functions were implemented either.

gvrender_begin_graph: remove disabled code

gvpack: remove disabled 'redoBB'

poly_path: squash -Wunused-parameter warnings

remove unimplemented 'html_path'

gvFreeLayout: squash -Wunused-parameter warning

This parameter cannot be removed without breaking API. Doing this does not seem
worth it just to squash a warning. Additionally this function may use the `gvc`
parameter in future.

cmpKey: squash -Wunused-parameter warnings

These parameters cannot be removed because this function is used as a callback
in scenarios that expect its current calling convention.

NullFunction: squash -Wunused-parameter warning

This parameter cannot be removed because this function is used as a callback in
instances that expect its current calling convention.

compoundEdges: squash -Wunused-parameter warnings

This parameter cannot be removed because this function is used as a callback in
some instances that expect its existing calling convention.

xinit_params: squash -Wunused-parameter warnings

This parameter cannot be removed because it _is_ used when `DEBUG` is defined.

pr2graphs: mark static and guard with 'DEBUG'

This function is not used outside of its containing file. It needs to be guarded
with `DEBUG` in order not to generate new -Wunused-function warnings.

Merge branch 'smattr/ffae0c5f-9668-4af7-be3c-5a5860888641' into 'main'

smyrna warning squashing

See merge request graphviz/graphviz!2316

smyrna switch2D3D: squash -Wunused-parameter warning

This parameter cannot be removed because `switch2D3D` is used as a callback in
an API that expects this calling convention.

smyrna menu_click_add_camera: remove unused 'p' parameter

smyrna create_color_theme: squash a -Wsign-compare warning

smyrna activate: remove unused 'doClear' parameter

smyrna refreshViewport: remove unused 'doClear' parameter

smyrna gl_main_expose: squash -Wunused-parameter warning

This parameter cannot be removed because `gl_main_expose` is used as a callback
in a function call that expects this calling convention.

smyrna close_graph: remove unused 'graphid' parameter

smyrna: use a bool for 'editable' to squash some -Wconversion warnings

smyrna: remove 'freeSmGraph' no-op

smyrna get_active_frame: squash -Wstrict-prototypes warning

smyrna Init: squash a -Wstrict-prototypes warning

Merge branch 'smattr/BEB24E57-745A-4ABD-AD32-15986AFBE7A5' into 'main'

[nfc] cast inputs to ctype.h 'is*' functions

See merge request graphviz/graphviz!2319

fail Cygwin CMake CI on compiler warnings

As of the previous commit, this build is warning-free.

[nfc] cast inputs to ctype.h 'is*' functions

The ctype.h `is*` functions are typically implemented as macros using a lookup
table. This is fine, but on Cygwin the compiler is somewhat picky about this:

  lib/ast/fmtesc.c:116:59: warning: array subscript has type char
    [-Wchar-subscripts]
    116 |                           (c == '#' && (b == f || isspace(*(b - 1)))
        |                                                           ^~~~~~~~

This change squashes these warnings.

Merge branch 'smattr/852C7D79-71DB-4538-83A3-550FDB732529' into 'main'

Start 2.50 development series

See merge request graphviz/graphviz!2320

Start 2.50 development series

Merge branch 'smattr/83CFF424-1C87-4175-B95E-F999D8F935E9' into 'main'

Stable Release 2.50.0

See merge request graphviz/graphviz!2310

Stable Release 2.50.0

I expect this release to be more turbulent than others in recent times, simply
because of the greater-than-average number of changes since the last release,
both in aggregate as well as specifically to the release process itself.

Merge branch 'smattr/2836D239-2D25-473C-82FF-F7DE1604C1C1' into 'main'

use 'sed' instead of 'rm;git-checkout' to fix CRLF line endings in Cygwin CI

See merge request graphviz/graphviz!2317