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

tclpkg sgnarea: [nfc] make sign extraction more obvious to the compiler

Squashes two -Wfloat-equal warnings. Equivalent of the previous commit, but
applied to tclpkg code that looks copy-pasted from neatogen.

neaotgen sgnarea: [nfc] make sign extraction more obvious to the compiler

Squashes two -Wfloat-equal warnings.

neato_layout: [nfc] squash a -Wsign-conversion warning

`Pack` is known non-negative in this branch.

gvbisect: [nfc] rephrase an open coded 'fabs'

lib/neatogen/edges.c: [nfc] remove unnecessary casts

Merge branch 'smattr/9F6685FF-CBD7-4AEA-B566-EA006B3BC33D' into 'main'

remove unnecessary casting

See merge request graphviz/graphviz!2306

myiddisc_open: remove unnecessary casts of 'gctx'

graphcmd: remove unnecessary cast of 'interp'

SparseMatrix_import_matrix_market: remove unnecessary casts of 'val'

poppler plugin: remove unnecessary casts around 'us->data'

remove unnecessary casts on 'lt_symlist_t' members