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.

neatogen computeHierarchyBoundaries: remove unused 'n' parameter

neatogen mkConstraintG: remove unused 'g' parameter

neatogen bfs_bounded: remove unused 'n' parameter

neatogen initLayout: remove unused 'graph' parameter

neatogen voronoi: remove unused 'triangulate' parameter

Merge branch 'smattr/5fa804c5-668e-4a54-b373-00bc933032d5' into 'main'

more compiler warning squashing

See merge request graphviz/graphviz!2770

smyrna on_attrAddBtn_clicked: use const type to squash a -Wcast-qual warning

Unfortunately we cannot also do the same for `attr_name` in this function
because it is passed to a Graphviz function that accepts a non-const pointer.

smyrna safestrdup: accept a const pointer

This will simplify an upcoming change.

smyrna set_refresh_filters: use const types to squash some -Wcast-qual warnings

Unfortunately we cannot also do the same for `attr_name` in this function
because it is passed to a Graphviz function that accepts a non-const pointer.

smyrna doApply: remove unused 'doAll' code path

smyrna doApply: remove unused 'widget' parameter

Note this also makes `widget` unused in `on_attrApplyBtn_clicked` which is a
Glade callback and cannot have its corresponding parameter removed. So we squash
the resulting warning there.

smyrna on_attrAddBtn_clicked: squash -Wunused-parameter warnings

55efb8e562bca6c4090a73712342ac006e133704 that squashes other such warnings in
this file missed that these are also unused in `on_attrAddBtn_clicked`.

smyrna mOpenSlot: squash -Wunused-parameter warnings

bd869436742887a0089f5ad817ecb36a1546987c that squashed other such warnings in
this file, missed that these are also unused in `mOpenSlot`.

smyrna btnToolFit_clicked: use float literals to simplify some code

smyrna btnToolZoomFit_clicked: rephrase unorthodox ternaries

This function was using assignments on the right hand sides of ternary
expressions. This is technically allowed, but leads to very hard to read code.

smyrna toolboxcallbacks: remove unnecessary parens

This also adjusts the location of some variable definitions to C99 style.

smyrna toolboxcallbacks: remove excess white space