expr: move 'Excc_t', 'Exccdisc_t' into excc.c

Following the previous commit, these are no longer referenced anywhere else.

expr: make 'exccopen', 'exccclose' static

These are not used outside of their containing file, and making them private
will enable some more impactful clean up.

expr gen: remove dereferencing and parens on function pointer

The compiler knows how to call through a function pointer without needing to be
explicitly told.

expr exccopen: only pass SFIO stream to callback

This function invokes `dtwalk` passing the callback `global` to be called with
a state parameter it supplies. It was supplying `cc`, but the callback only uses
a single member of this struct. So we can simplify this code by just passing the
stream the callback writes to, `cc->ccdisc-text`, or equivalently `disc->text`.

Merge branch 'smattr/173D0FC2-B18B-4FFB-B2E6-6272F7273A5C' into 'main'

fix unchecked allocation failures, remove 'N_NEW' definitions

See merge request graphviz/graphviz!2777

ccomps: remove 'N_NEW' that was only used in one place

This brings us one step closer to having a single `N_NEW` macro in the tree and
not having to constantly wonder which of the `N_NEW` definitions is in scope at
any point where you see it called.

Pango plugin: remove unused 'NEW'

Pango plugin gv_get_ps_fontlist: postpone 'family_name' allocation

Rather than allocating this upfront and then potentially needing to free it
later, we can use a string view to postpone the allocation to when we know it is
actually needed. This works because the lifetime of the backing memory extends
until the call to `g_free(families)`. This also fixes the lack of allocation
failure checks; instead of `strdup` we now call `strview_str` that gracefully
exits on allocation failure.

Pango plugin get_font_mapping: fix unchecked allocation

Pango plugin gv_get_ps_fontlist: fix unchecked allocation

common genEllipticPath: fix unchecked allocation

common moveTo: fix unchecked allocation

gml2gv nameOf: fix unchecked allocation

gvgen makeTreeGen: fix unchecked allocation

gvgen getCnt: fix unchecked allocation

gvgen mkTree: fix unchecked allocations

gvgen makeTetrix: fix unchecked allocations

gvgen makeSierpinski: fix unchecked allocations

Merge branch 'smattr/dcb7bc23-5914-43ae-94d5-0003feef06b0' into 'main'

gvpr: more compiler warning squashing

See merge request graphviz/graphviz!2778

gvpr endString: squash a -Wconversion waning

`c` is known to be '\\' here.

gvpr parseID: simplify loop

gvpr readc: cast to squash -Wconversion warnings

These variables are known to be '\n' here.

gvpr: represent binding count as a size_t

Squashes 3 -Wsign-conversion warnings.

gvpr tvtypeToStr:  take a long long instead of int

This function is always called with long long values, so lets match it with its
callers. Squashes 2 -Wconversion warnings.

gvpr binary: remove unnecessary parens

gvpr binary: use local types that corresponds to the values being stored

Squashes 2 -Wconversion warnings.

gvpr validTVT: remove unnecessary parens

gvpr validTVT: return a more appropriate type

gvpr validTVT: take a long long instead of int

This function is always called with long long values, so lets match it with its
callers. Squashes 2 -Wconversion warnings.

gvpr setval: use a local type that corresponds to the value being stored in it

Squashes a -Wconversion warning. This has no effect on functionality because the
`validTVT` call is confirming the value is within the expected range.

Merge branch 'smattr/91ab2290-e3c6-4966-93b8-b1d10a129711' into 'main'

pathplan clean up and warning squashing

See merge request graphviz/graphviz!2776

neatogen printvis: remove unnecessary casts

The type of the fields of `pts` can vary between `float` and `double` depending
compile time options, which is what this code was attempting to deal with. But
`float` and `double` are treated identically with respect to `printf`, so there
is no need to cast.

pathplan compVis: remove always-0 parameter

pathplan Pobspath: squash a -Wconversion warning

Here again the proper solution would be to make the struct’s field also a
`size_t` but once again it is part of the public API that we would like to avoid
breaking.

pathplan Pobspath: replace an unchecked array 'malloc' with 'gv_calloc'

This function has no way of reporting failure to the caller. So this turns an
allocation failure from a messy crash into a more graceful exit.

pathplan Pobsopen: call 'calloc' instead of 'malloc' when allocating arrays

This is the generally preferred way of doing this that avoids various pitfalls,
such as integer overflow during multiplication.

pathplan Pobsopen: use a 'size_t' when calling 'malloc'

Squashes a -Wsign-conversion warning and makes explicit a previously implicit
assumption.

pathplan Pobsopen: use a 'size_t' when counting objects

This squashes 3 -Wsign-conversion warnings and is generally closer to what we
would like to do here. The “proper” fix is for fields like `vconfig_t.N` to
become `size_t` instead of `int`. But unfortunately they are part of the public
API, and it seems undesirable to break API for this.

Note that an assumption previously implicit in this function, that all inputs
had a non-negative polygon count, is now an explicit assertion.

pathplan Pobsopen: fix unchecked allocation failures

The first `malloc` of `rv` was checked for failure but the subsequent ones were
assumed to succeed.

pathplan: remove 'mymalloc' wrapper

The calling code does not rely on the semantics of `malloc` returning `NULL`
when called with a 0 size. It unconditionally frees these arrays in `Pobsclose`
so allocations returning a non-`NULL` pointer for 0-sized allocations (which
Glibc typically does) is fine. Maintaining a wrapper that enforces this is
unnecessary.

Merge branch 'smattr/b76f3f9f-2903-4d65-bbfd-8ce53a769f47' into 'main'

warning squashing and removing unused 'flag' parameters

See merge request graphviz/graphviz!2775

sfdpgen SparseStressMajorizationSmoother_new: remove 'scale_initial_coord' param

This is always set to true.

sfdpgen SparseStressMajorizationSmoother_new: remove inverse distance weighting

This scheme is unused.

gvmap makeMap: remove now unused 'flag' local

gvmap make_map_from_rectangle_groups: remove now unused 'flag' parameter

gvmap make_map_internal: remove 'flag' parameter

This is always set to 0 and conveys no information to the caller.

sparse QuadTree_get_nearest: remove 'flag' parameter

This is always set to 0 and conveys no information to the caller.

sparse QuadTree_get_nearest_internal: remove 'flag' parameter

This is always set to 0 and conveys no information to the caller.

sfdpgen: remove unnecessary casts of 'Operator.data' pointers

sfdpgen SparseMatrix_solve: remove 'flag' parameter

This is always set to 0 and conveys no information to the caller.

sfdpgen SparseMatrix_solve: remove unused Jacobi matrix solving

This also removes the `method` parameter to this function, which was only ever
set to a single value. This squashes 8 compiler warnings and drops a lot of dead
code.

neatogen constrained_majorization_vpsc: s/fabs/fabsf when dealing with floats

This squashes a -Wfloat-conversion warning.

neatogen stress_majorization_cola: remove unused 'nedges_graph' parameter

neatogen tweakEnd: remove unused 'pl' parameter

neatogen addTriEdge: remove unused 'd' parameter

This function was ignoring the parameter and calculating the edge distance
itself internally.

Merge branch 'point-test-doc' into 'main'

pointset: doxygen comments

See merge request graphviz/graphviz!2700

pointset: doxygen comments

Merge branch 'partition-doc' into 'main'

partition.h: add doxygen comments

See merge request graphviz/graphviz!2707

partition.h: add doxygen comments

partition.h: add names of arguments

and remove by the way redundant `extern`

Merge branch 'smattr/312b41d1-07fa-4afd-911f-b761fdd0d6db' into 'main'

gvpr warning squashing

See merge request graphviz/graphviz!2774

gvpr canontoken: use char instead of unsigned char, introduce casts

It is not clear to me why this code was using unsigned char when it can do the
same thing with less typing and fewer compiler warnings using char. This also
introduces casts to squash warnings from some of the more pedantic compiler
implementations. See 6c29170f9f29466374fbc6e8e62a1b6916c6bc59 for details.

gvpr canontoken: remove commented out code

gvpr readLine: squash -Wconversion warnings

`c` is guaranteed to fit in a char here.

gvpr freadFile: squash a -Wsign-compare warning

gvpr fwriteFile: squash a -Wsign-compare warning

gvpr cloneO: squash a -Wswitch-default warning

This switch is exhaustive.

gvpr copy: squash a -Wswitch-default warning

This switch is exhaustive.

gvpr rindexOf: return a long instead of int

The only use of this function stores the return value in a long, so lets just
return a long, squashing two compiler warnings.

gvpr cmppair: squash -Wunused-parameter warnings

This function is used as a callback, so cannot easily have these unused
parameters removed.

Merge branch 'smattr/a2930eb8-ba18-4789-bf34-93446083513e' into 'main'

lib/ast warning squashing and simplification

See merge request graphviz/graphviz!2773

ast onematch: simplify string comparisons by using string views

ast onematch: fuse comparison against 'xdigi' and 't'

Freed of the limitations of hash function in the previous commit, we can express
this as a single comparison.

ast onematch: rephrase hash-based switch into string comparisons

This code was using a hand rolled hash function to implement a series of string
comparisons as a jump table. I guess at some point this must have been a
necessary optimization due to limitations of the day’s compilers/machines. In a
modern environment, this is a deoptimization, impeding the compiler’s ability to
understand your intent. Modern compilers know the string comparison functions as
built-ins and can use SIMD¹/SWAR² tricks to emit a short string comparison as a
single instruction. They are also capable of transforming an if-then-else ladder
into a switch if their heuristics predict it will be worthwhile.

¹ https://en.wikipedia.org/wiki/Single_instruction,_multiple_data
² https://en.wikipedia.org/wiki/SWAR

ast gobble: squash -Wswitch-default warning

ast strmatch: squash -Wundef warnings

ast stresc: squash -Wswitch-default warning

ast stresc: remove return value

The computation of the return value for this function was relying on string
lengths that fitted in an `int`, something that is generally but not always
true. The compiler complained about this with -Wconversion. The only caller of
this function does not use the return value, so lets just remove it.

ast: prototype 'pathcanon' in library header instead of inline near its use

This squashes a -Wmissing-prototypes, is better style, and is generally inline
with the rest of lib/ast.

ast fmtbuf: squash -Wsign-compare warning

This is computing the number of bytes remaining in `buf`; the number of bytes
between a pointer to the current offset and the end of `buf`. Thus it is always
non-negative.

ast chrtoi: squash -Wswitch-default warning

ast chrtoi: remove unnecessary cast

ast chrtoi: squash -Wsign-compare warning

Merge branch 'smattr/f2b32419-4ad7-4884-83ff-cc27d7a0667f' into 'main'

sfdpgen warning squashing

See merge request graphviz/graphviz!2772

sfdpgen: remove unimplemented 'DistanceMatrix_restrict_filtering'

sfdpgen maximal_…: remove unused 'randomize' parameter

sfdpgen: make 'UniformStressSmoother_new' a static function

This function is not used outside of its containing file.

sfdpgen UniformStressSmoother_new: remove unused 'flag' parameter

sfdpgen UniformStressSmoother_new: remove unused 'x' parameter

sfdpgen UniformStressSmoother_new: remove unused 'dim' parameter

sfdpgen stress_model: remove unused 'A' parameter

sfdpgen: make 'get_stress' static

This function is unused outside of its containing file and is only used when
`DEBUG_PRINT` is defined.

sfdpgen get_stress: remove 'weighted' parameter that is always 1

sfdpgen get_stress: remove unused 'data' parameter

sfdpgen jacobi: remove unused 'flag' parameter

Merge branch 'smattr/45dc48f2-fa91-42ed-8530-8bcaf9fdb303' into 'main'

neatogen compiler warning squashing

See merge request graphviz/graphviz!2771

neatogen call_tri: remove unused 'dim' parameter

neatogen CMDS_orthog: remove unused 'g' parameter

neatogen constrained_majorization_new_with_gaps: remove unused 'ndim' parameter

Note that the call to this function from
`constrained_majorization_new_with_gaps` also seems confused about the meaning
of its parameters. But no attempt is made to correct this.

neatogen: remove unused 'constrained_majorization_gradient_projection'

This function has seemingly never been used.