smyrna: remove unused 'frame_length' field

smyrna: remove unused 'dfltEngine' field

smyrna: remove unused 'consoleText' field

smyrna: remove unused 'FontSizeConst' field

smyrna: remove unused 'visiblenodecount' field

smyrna: remove unused 'glade_file' field

smyrna: remove unused 'attr_file' field

smyrna: remove unused 'flush' field

smyrna: remove unused 'scripts' array field

smyrna: remove unused 'nodeshape' field and 'node_shape' type

The member `nodshapedot` also seems to have been misspelled, but never used
anyway, so oh well.

smyrna: remove unused 'nodesizewithdegree' field

Merge branch 'smattr/2F81BBCF-85DE-4DC9-B683-3F4784A7DD77' into 'main'

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

See merge request graphviz/graphviz!2235

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

Merge branch 'smattr/A172EA51-A44B-4BCE-9390-72176966B563' into 'main'

more header guard replacement with #pragma once

See merge request graphviz/graphviz!2232

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

Amusingly it looks like some time in the past (prior to version control) someone
got a little trigger happy find-and-replacing “CompText” with “CompFont”.

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

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

Merge branch 'smattr/E696CB4B-4EC4-479D-8073-C970EEF14ECC' into 'main'

CMake: only pass 'YY_NO_UNISTD_H' to Flex when unistd.h is not found

See merge request graphviz/graphviz!2229

CMake: only pass 'YY_NO_UNISTD_H' to Flex when unistd.h is not found

This macro tells Flex that #including unistd.h should be avoided. There is no
need for this on almost any platform except Windows, hence why `YY_NO_UNISTD_H`
is only mentioned elsewhere in the MS Build build system, not in the Autotools
build system. In the CMake build system which is meant to be used across Windows
and non-Windows platforms, we can do something more nuanced and depend on the
existence check of unistd.h itself.

Merge branch 'smattr/1737da18-1462-4672-ae09-c19621653f51' into 'main'

fix GVPR splitting and tokenization bug

Closes #2138

See merge request graphviz/graphviz!2228

extokens: fix missing NUL terminator append in GVPR tokenization

This is the second half of a bug fix following the prior commit.

Commit 971293551421455a0d939b9f8cea17356b7968f8 refactored this code to avoid
the use of an SFIO buffer, inadvertently introducing a bug. The change did not
account for the source buffer not being NUL terminated. This fix wicks closer to
the original code, not assuming a NUL terminator and copying a known number of
bytes into the destination.

Fixes #2138.

exsplit: fix missing NUL terminator append in GVPR splitting

Commit 7ef9d53e2e6dc53c44939ace7a9cad57c3aa00bf refactored this code to avoid
the use of an SFIO buffer, inadvertently introducing a bug. The change did not
account for the source buffer not being NUL terminated. This fix wicks closer to
the original code, not assuming a NUL terminator and copying a known number of
bytes into the destination.

Related to #2138.

add a test case for a GVPR bug

Related to #2138.

Merge branch 'Warray-param' into 'main'

Match function signature in definition with declaration

See merge request graphviz/graphviz!2233

Match function signature in definition with declaration

GCC11 throws the following warning:
graphviz/lib/pathplan/shortest.c:93:47: warning: argument 2 of type ‘Ppoint_t *’ {aka ‘struct Pxy_t *’} declared as a pointer [-Warray-parameter=]
   93 | int Pshortestpath(Ppoly_t * polyp, Ppoint_t * eps, Ppolyline_t * output)
      |                                    ~~~~~~~~~~~^~~
In file included from graphviz/lib/pathplan/pathutil.h:15,
                 from graphviz/lib/pathplan/shortest.c:16:
graphviz/lib/pathplan/pathplan.h:22:59: note: previously declared as an array ‘Ppoint_t[2]’ {aka ‘struct Pxy_t[2]’}
   22 |     extern int Pshortestpath(Ppoly_t * boundary, Ppoint_t endpoints[2],
      |                                                  ~~~~~~~~~^~~~~~~~~~~~
[2/20] Building C object lib/pathplan/CMakeFiles/pathplan.dir/route.c.o
graphviz/lib/pathplan/route.c:76:29: warning: argument 4 of type ‘Ppoint_t *’ {aka ‘struct Pxy_t *’} declared as a pointer [-Warray-parameter=]
   76 |                  Ppoint_t * evs, Ppolyline_t * output)
      |                  ~~~~~~~~~~~^~~
In file included from graphviz/lib/pathplan/pathutil.h:15,
                 from graphviz/lib/pathplan/route.c:17:
graphviz/lib/pathplan/pathplan.h:28:39: note: previously declared as an array ‘Pvector_t[2]’ {aka ‘struct Pxy_t[2]’}
   28 |                             Pvector_t endpoint_slopes[2],
      |                             ~~~~~~~~~~^~~~~~~~~~~~~~~~~~

Merge branch 'mingw64_cleanup_ifdef_gts' into 'main'

Unify usage of preprocessor constant HAVE_GTS.

See merge request graphviz/graphviz!2219

realign a 'HAVE_GTS' check with 'ifdef' style

Unify usage of preprocessor constant HAVE_GTS.

Merge branch 'smattr/264284AF-9FAA-4FDC-9985-6158D1454C97' into 'main'

plugin/: replace header guards with more modern #pragma once

See merge request graphviz/graphviz!2225

plugin/: replace header guards with more modern #pragma once

Merge branch 'smattr/652F0EAF-FDBE-4189-83D8-EE850C97868C' into 'main'

lib/neatogen/delaunay.c: some clean up

See merge request graphviz/graphviz!2226

UG_graph: use a C99 bool for 'removed'

delaunay.c: remove some unnecessary brackets

delaunay.c: reflow some unnecessarily wrapped lines

delaunay.c: remove some unnecessary casts

update legacy 'qsort_r' comment

The problem of “call `qsort` but also access some external context” is still
relevant, but this functionality has been standardized into `qsort_s` in C11,
not `qsort_r`.

Merge branch 'smattr/0A848998-1291-43E8-8525-5A355EC8AD10' into 'main'

Move back to 2.49 development series

See merge request graphviz/graphviz!2227

Move back to 2.49 development series

Merge branch 'smattr/AB9C539E-363F-4BFC-89A2-4C432B9E7190' into 'main'

Stable Release 2.49.2

See merge request graphviz/graphviz!2218

Stable Release 2.49.2

Merge branch 'smattr/238e3b6b-eb20-441a-a609-79496e66c3e0' into 'main'

gv2gxl: remove duplicate XML escaping functionality

See merge request graphviz/graphviz!2208

gv2gxl: remove duplicate XML escaping functionality

Merge branch 'smattr/93788a5c-6268-4813-8ab7-7bf81636d537' into 'main'

sfio and expr warning squashing

See merge request graphviz/graphviz!2209

SF_UNBOUND: define as a 'size_t' instead of casting to 'size_t' at every usage

There were also cases missing a cast, which triggered -Wsign-conversion
warnings.

print: squash a -Wsign-compare warning

print: remove variable shadowing of 'expr' global

prints: remove variable shadowing of 'expr' global

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