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

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

Only selective artifacts in the repository are sensitive to line endings. E.g.
C/C++ and Python code is happily parsed by the compiler/interpreter regardless
of which line ending style it uses.

This should hopefully accelerate the Cygwin CI tasks slightly. The assumption is
that running `sed` on a few files should be faster than deleting all files and
recreating them.

Merge branch 'smattr/43bafefc-38dd-4b6b-9336-4fe8e8909ae7' into 'main'

de-dupe fig_string, mp_string, pic_string

Closes #2051

See merge request graphviz/graphviz!2311

mpcolor: [nfc] simpler definition

Makes it more obvious to the compiler it can unroll the use of this in a loop.

pic plugin: replace 'pic_string' with 'gvputs_nonascii'

This finishes the process of de-duping `fig_string`, `mp_string`, and
`pic_string`. This change makes this plugin more thread safe (no static buffer
is no longer used), and improves its efficiency (bytes are now written directly
into the output instead of a temporary buffer, requiring dynamic allocation and
two copies).

Closes #2051.

mp plugin: replace 'mp_string' with 'gvputs_nonascii'

This partially de-dupes some code (`pic_string` is a duplicate of `mp_string`),
makes this plugin more thread safe (no static buffer is no longer used), and
improves its efficiency (bytes are now written directly into the output instead
of a temporary buffer, requiring dynamic allocation and two copies).

Related to #2051.

fig plugin: replace 'fig_string' with 'gvputs_nonascii'

This partially de-dupes some code (`mp_string` and `pic_string` are duplicates
of `fig_string`), makes this plugin more thread safe (no static buffer is no
longer used), and improves its efficiency (bytes are now written directly into
the output instead of a temporary buffer, requiring dynamic allocation and two
copies).

Related to #2051.

lib/gvc/gvc.def: [nfc] strip trailing whitespace

implement next 'gvputs_nonascii' function

This is based on `fig_string`, `mp_string`, and `pic_string`, but outputting
data directly into the output file rather than a temporary buffer.

Related to #2051.

Merge branch 'smattr/E7C21AC3-3A42-4520-8BEC-C9657B7DC8F9' into 'main'

GDI+ plugin: some clean up and modernization

See merge request graphviz/graphviz!2307

GDI+ plugin: rewrite 'FileStream' reference counting to be thread safe

Based on Microsoft sample code,¹ it seems like the expectation is that consumers
of the `IStream` API may expect to concurrently manipulate reference counts.
This change guards against this by making the referencing counting code use
lock-free atomics, based on the same Microsoft sample.

Note that this is essentially proactively addressing a latent issue as it is, in
general, unsafe to use Graphviz libraries or plugins in a multithreaded
environment right now.

¹ https://docs.microsoft.com/en-us/previous-versions/windows/desktop/ms752876(v=vs.85)

GDI+ plugin: [nfc] avoid directly modifying 'FileStream' refcount

This change redirects all reference count modifications through `AddRef` instead
direction increments. Based on Microsoft sample implementation.¹

¹ https://docs.microsoft.com/en-us/previous-versions/windows/desktop/ms752876(v=vs.85)

GDI+ plugin: [nfc] remove assumptions on lifetime of 'FileStream' filename

Avoiding an assumption on the lifetime of `us->name` provides one less thing to
think about when iterating on this code.

GDI+ plugin: [nfc] use a const pointer for filename

The `FileStream` class does not need to modify this internally, so this avoids
the need for sketchy type casts.

GDI+ plugin: use 'override' instead of 'virtual' on 'FileStream' methods

This C++11 mechanism is an improvement as it causes compilation to error out if
any of these methods were not already declared virtual in a parent class. That
is, it provides a sanity check that these overrides are really doing what they
intend.

GDI+ plugin: [nfc] remove legacy use of a C-style cast

GDI+ plugin: [nfc] strip trailing whitespace

Merge branch 'smattr/18D413B7-67B7-4F3B-A544-823A3810D062' into 'main'

fix missing author+date line in Debian changelog

See merge request graphviz/graphviz!2309

fix missing author+date line in Debian changelog

Commit 42d7c87c291781ccaff47126a5ad3a2469e310cc omitted this required
information, resulting in warnings like the following in the build log:

  dpkg-gencontrol: warning:     debian/changelog(l7): found start of entry where
    expected more change data or trailer

Merge branch 'smattr/2504A45F-EA4A-41EA-BF7F-E758CD4AE122' into 'main'

lib/neatogen clean up

See merge request graphviz/graphviz!2308

solve: [nfc] squash a -Wmissing-prototypes warning

lu_decompose: [nfc] squash -Wmissing-prototypes warning

lu_solve: [nfc] squash -Wmissing-prototypes warning

matinv: [nfc] squash -Wmissing-prototypes warning

find_intersection: [nfc] squash -Wunused-parameter warnings