getdyn: remove variable shadowing of 'expr' global

evaldyn: remove variable shadowing of 'expr' global

exerror/exwarn: allow GNU-compatible compilers to detect misuse

This lets compilers like GCC and Clang notice when these functions are called
with a format string that is inconsistent with the remaining arguments and issue
a warning. This detects problems like those fixed in the prior commit.

fix misuse of 'exerror'

These issues are detectable by tagging `exerror` with
`__attribute__((format(printf, 1, 2)))`.

deref: remove unnecessary 'break' statements

Merge branch 'smattr/63959121-2587-4D2B-82E1-87E5132AC015' into 'main'

lib/cdt: replace header guards with more modern #pragma once

See merge request graphviz/graphviz!2224

lib/cdt: replace header guards with more modern #pragma once

Merge branch 'smattr/CA6A6F72-A9FA-4694-9BE4-52D0EA99C11E' into 'main'

lib/gvpr: replace header guards with more modern #pragma once

See merge request graphviz/graphviz!2220

lib/gvpr: replace header guards with more modern #pragma once

Merge branch 'smattr/BD47AC77-0B88-41A8-8983-2D057C6598C7' into 'main'

libedgepaint: replace header guards with more modern #pragma once

See merge request graphviz/graphviz!2215

libedgepaint: replace header guards with more modern #pragma once

Merge branch 'smattr/527996E1-267D-4746-BF2A-20A2A66E028B' into 'main'

lib/ast: replace header guards with more modern #pragma once

See merge request graphviz/graphviz!2222

lib/ast: replace header guards with more modern #pragma once

Merge branch 'smattr/4755772E-A42F-4DB2-A389-E4304FB44BBC' into 'main'

smyrna: more dead code removal

See merge request graphviz/graphviz!2214

smyrna: remove unused 'DOT_SIZE_CORRECTION_FAC'

smyrna: remove unused 'SPHERE_SLICE_COUNT'

smyrna: remove unused 'DOUBLE_IT'

smyrna: remove unused 'GL_DOTSIZE_CONSTANT'

smyrna: remove unused node limit constants

smyrna: remove unused Z plane constants

smyrna: remove unused 'UNHIGHLIGHTED_ALPHA'

smyrna: remove unused 'RAD2DEG'

smyrna: remove unused 'ZOOM_STEP', 'ZOOM_STEPS'

smyrna: remove unused 'GLOBAL_Z_OFFSET'

smyrma: remove duplicate 'xml' declaration

This is declared in gui.h that is already included in this target file.

smyrna: remove unused union names

Merge branch 'smattr/2633d9ac-c5e9-4556-b4cd-b4b879e4add7' into 'main'

some ps_txt-related clean up

See merge request graphviz/graphviz!2217

user_init: use a C99 bool for 'saw_bb'

cat_libfile: use a C99 bool for 'use_stdlib'

remove unnecessary 'Show_boxes' cast

`Show_boxes` is already a `const char**`.

ps_txt: fuse array strings, accelerating 'cat_libfile'

The `ps_txt` array is only used in calls to `cat_libfile`, which does not need a
single line per array entry. By fusing all strings into the first `ps_txt` array
entry, we can reduce the number of eventual `gvputs` calls, accelerating the
entire operation.

disable clang formatting in ps.h and reflow lines

Following 96136309542ef71101600335147fcd19253a5fde, in retrospect it seems this
file is more readable with reflowing of long lines disabled.

Merge branch 'fix_windows_if_have_gts' into 'main'

The preprocessor constant HAVE_GTS should be undefined like others similar flags.

See merge request graphviz/graphviz!2221

The preprocessor constant HAVE_GTS should be undefined like others similar flags.

Merge branch 'addendum-fix-bom-in-graphviz-version-file' into 'main'

Don't generate useless GRAPHVIZ_VERSION file in macOS CMake build CI job

See merge request graphviz/graphviz!2213

CI: move generation of COLLECTION from macOS build template to individual jobs

This is no functional change, but makes the macOS build jobs more
similar to the Linux build jobs and opens up for letting the macOS
CMake and autotools builds use different COLLECTION in the future.

CI: don't generate GRAPHVIZ_VERSION file in macOS CMake build job

The version is generated internally to a variable in the CMake build
job and the downstream test job generates the file itself since commit
fea17b6a9738eb96f037e22538d1885a75f20698.

This commit moves the generation of the GRAPHVIZ_VERSION file from the
build template to the macOS autotools build job.

Merge branch 'fix-awk-crlf-agnosticity-on-macos-catalina' into 'main'

Fix problem with awk crlf agnosticity on macOS Catalina

See merge request graphviz/graphviz!2210

common: CMake: make awk crlf agnostic when generating svgcolor_lib only when using Cygwin

common: autotools: make awk crlf agnostic when generating svgcolor_lib only when using Cygwin

Merge branch 'smattr/B8F865EA-1BE4-4990-96CF-2D5DCEB70122' into 'main'

libgvc: replace header guards with more modern #pragma once

See merge request graphviz/graphviz!2212

libgvc: replace header guards with more modern #pragma once

Merge branch 'smattr/0c5e6e8f-e750-47bf-b91f-d2c13242d5d1' into 'main'

remove generation of ps.h and commit generated output instead

See merge request graphviz/graphviz!2211

remove generation of ps.h and commit generated output instead

The generation of ps.h from ps.txt was trivial and ps.txt was no easier to edit
or update than the resulting generated header. Removing it and using a static
source instead accelerates all three builds, increases portability and moves
towards dropping the dependency on Awk.

Related to #2118.

Merge branch 'smattr/a5361ad7-b54f-484d-9fde-b9b85f00e0d4' into 'main'

lib/dotgen: clean up and warning squashing

See merge request graphviz/graphviz!2203

sameport: remove unused parameter

sameport.c: remove obsolete code

zapLayers: remove unused parameter

completeregularpath: squash a -Wunused-parameter warning

make_regular_edge: use a proper boolean for checking edge type is spline

This change also removes a -Wshadow compiler warning.

dotsplines.c: rephrase some open coded fmin, fmax

dotsplines.c: remove commented out code

dotsplines.c: remove unnecessary bracketing

portcmp: rephrase to avoid conversion to int

This function was seemingly incorrectly casting the (float) result of a float
subtraction to an integer. Apart from triggering compiler warnings, this had the
effect of concluding points that were merely close together were actually equal.

_dot_splines: remove unnecessary cast and address operations

Merge branch 'smattr/EF03B314-9D8C-4B6C-8466-8DC715A590E2' into 'main'

MS Build build system: remove minimal rebuild enabling

See merge request graphviz/graphviz!2207

MS Build build system: remove minimal rebuild enabling

This setting controls the `/Gm` flag to the compiler which is deprecated.¹ When
using this, the compiler warns:

  cl : command line warning D9035: option 'Gm' has been deprecated and will be
    removed in a future release

¹ https://docs.microsoft.com/en-us/cpp/build/reference/gm-enable-minimal-rebuild?view=msvc-160

Merge branch 'smattr/E25E87C9-5338-4CDA-81A0-47CA52A1067A' into 'main'

smyrna: more dead code removal

See merge request graphviz/graphviz!2204

smyrna: remove prototype for unimplemented 'btnTVUnhighlightAll_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVHighlightAll_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVUnselectAll_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVSelectAll_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVFilterClear_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVGotopage_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVLast_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVNext_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVPrevious_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVFirst_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVFilter_clicked_cb'

smyrna: remove prototype for unimplemented 'btnTVDelete_clicked_cb'

smyrna: remove 'cgbTVSelect_toggled_cb' no-op

smyrna: remove 'cgbTVVisible_toggled_cb' no-op

smyrna: remove 'cgbTVHighlighted_toggled_cb' no-op

Merge branch 'smattr/DFDF0E87-1EF5-490D-A08A-AB53B614A394' into 'main'

remove lib/spine

See merge request graphviz/graphviz!2206

remove lib/spine

Despite being compiled and hooked into all three build systems, nothing in
Graphviz uses this and none of its artifacts are installed. Its compilation
appears to be an exercise without purpose.

libcommon: remove unused 'UF_remove'

Merge branch 'fix-bom-in-graphviz-version-file' into 'main'

CI: avoid BOM in GRAPHVIZ_VERSION file in windows build jobs

Closes #2133

See merge request graphviz/graphviz!2205

CI: remove archiving of GRAPHVIZ_VERSION from MinGW and Cygwin CMake jobs

There're no downstream jobs to use it anyway and future CMake test
jobs can generate it themselves.

CI: remove archiving of GRAPHVIZ_VERSION from the windows-cygwin-build-using-autogen job

There's no downstream job to use it anyway and a future test job can
generate it itself.

CI: add generation of GRAPHVIZ_VERSION also for macOS CMake test job

Commit b4047f93aa3c1a3718b39493e5814532fa0850fe added this for all
other jobs, but missed the macOS test job for some reason.

CI: avoid BOM in GRAPHVIZ_VERSION file in windows-cygwin-build-using-autogen job

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

CI: avoid BOM in COLLECTION file in windows build jobs

CI: avoid BOM in GRAPHVIZ_VERSION file in windows build jobs

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

Merge branch 'smattr/05f4cdeb-0e09-4cfc-ac55-d8822807d4a5' into 'main'

libcommon: provide more generic XML escaping

See merge request graphviz/graphviz!2198

common: remove 'xml_url_string'

As of the previous commit, all calls to `xml_url_string` have been replaced with
calls to `xml_escape`.

Related to #1868.

replace 'gvputs(… xml_url_string(…))' with 'xml_escape(…)'

Similar to previous `xml_string` and `xml_string0` related changes, this avoids
dynamic allocations and makes the affected code thread safe.

Related to #1868.

common: remove 'xml_string0'

All calls of this function have now been refactored to use `xml_escape`.

Related to #1868.

common: remove 'xml_string'

All calls of this function have now been refactored to use `xml_escape`.

Related to #1868.

gv2gml emitAttr: replace 'fprintf(… xml_string(…))' with 'xml_escape(…)'

Similar to the prior commits, this avoids dynamic allocation and improves thread
safety. This is intended to be a functional no-op.

Related to #1868.

pango_textlayout: replace 'agxbput(… xml_string0(…))' with 'xml_escape(…)'

Similar to the prior commit, this avoids dynamic allocating – well, some,
`axbput` itself is still dynamically allocating – and improves thread safety.
This is intended to be a functional no-op.

Related to #1868.

replace 'gvputs(… xml_string0(…))' with 'xml_escape(…)'

This is, again, a functional no-op. But it has the effect of removing dynamic
allocations and making this code thread-safe.

Related to #1868.

replace 'gvprintf(… xml_string(…))' with 'gvputs_xml'

A first demonstration of how to use the `xml_escape` functionality introduced in
preceding commits. Note that, while this is intended to be a non-functional
change, this does side step intermediate buffering in `xml_string` and escaped
output is now passed directly to `gvputs`.

Related to #1868.

Visio plugin: replace redeclaration of 'xml_string' with a #include

There is no need for this duplicate prototype, or to lie about its type
signature.

Related to #1868.

implement a 'gvputs' alternative that does XML-escaping

This will be used in an upcoming commit to replace `gvputs` when used in
combination with `xml_string`.

Related to #1868.

plugin core: swap duplicated prototypes for missing includes

This file is calling functions `xml_string` `xml_url_string` without including
the header that prototypes them, utils.h. There seems to be no reason to avoid
this and locally re-prototype these functions.

Related to #1868.

expose a primitive 'xml_escape' for generic XML escaping

This function is intended to be a flexible XML-escaper that subsumes
`xml_string`, `xml_string0`, and `xml_url_string`. Unlike the preceding named
functions, calling `xml_escape` is thread-safe and does not mandate dynamic
allocation.

Related to #1868.

xml_url_string: call xml_core instead of repeating logic

Related to #1868.

xml_core: make '-' and ' ' escaping configurable

This is a non-functional change. It is preparation to allow `xml_url_string` to
use `xml_core`.

Related to #1868.

xml_string0: outline the inner loop body functionality

The motivation for this change is to move towards consolidating XML-escaping
functionality. The introduced function, `xml_core`, is intended to be usable by
`xml_url_string` and other Graphviz XML escaping functions in future.

This aim of consolidation hopefully also explains why this change looks more
complicated than might be expected. It is intended to be functionally a no-op.
But `xml_core` takes a generalized callback and state because it is imagined a
future caller may want to pass in, e.g. `gvputs` as their target sink for
XML-escaped data. That is, future changes may side step the static buffer usage
and allocation that happens in `xml_string0` and instead pump bytes directly
into a final target file.

Related to #1868

xml_isentity: take a const pointer instead of a mutable one

This function does not modify the pointed to data of its parameter, and this
will simplify some upcoming changes.

Related to #1868.