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

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