expr: remove dereferencing and parens around calling 'Exdisc_t.keyf'

The compiler does not need to be explicitly told how to call through a function
pointer.

expr: remove unused parameters from 'Exdisc_t.keyf'

I guess when I committed 2e9753d9c9886f5bc013212420528e9a18bc592b it was not
obvious to me that this is entirely part of private internal interfaces and the
callback signature involved is not exposed publicly.

expr: remove dereferencing and parens around calling 'Exdisc_t.stringof'

The compiler does not need to be explicitly told how to call through a function
pointer.

expr: remove dereferencing and parens around calling 'Exdisc_t.typename'

The compiler does not need to be explicitly told how to call through a function
pointer.

expr: remove unused 'pg' parameter from 'Exdisc_t.typename'

expr: remove dereferencing and parens around calling 'Exdisc_t.binaryf'

The compiler does not need to be explicitly told how to call through a function
pointer.

expr: remove unused parameters from 'Exdisc_t.binaryf'

I guess when I committed b0ec7b2eb448a2cb68ffb3751e9e054aecc60c24 it was not
obvious to me that this is entirely part of private internal interfaces and the
callback signature involved is not exposed publicly.

expr: remove dereferencing and parens around calling 'Exdisc_t.convertf'

The compiler does not need to be explicitly told how to call through a function
pointer.

expr: remove unused parameters from 'Exdisc_t.convertf'

I guess when I committed 17479ab6569d40a4778870f712226aa7916f3ca3 it was not
obvious to me that this is entirely part of private internal interfaces and the
callback signature involved is not exposed publicly.

expr: remove dereferencing and parens around calling 'Exdisc_t.reff'

The compiler does not need to be explicitly told how to call through a function
pointer.

expr: remove further unused parameters from 'Exdisc_t.reff'

I guess when I committed ef3ed2a98a46d0641d936c62fde78e719cef98f8 it was not
obvious to me that this is entirely part of private internal interfaces and the
callback signature involved is not exposed publicly.

expr: remove now unused 'disc' parameter from 'Exdisc_t.reff'

expr: replace 'exdump' SFIO buffer with an 'agxbuf'

Unfortunately there is no easy way to stage this. We need to do it all at once,
updating every use of `deparse` and all `exdump`’s internals.

This change improves locality – it is more obvious to both users and the
compiler that the contents of this temporary buffer does not need to be retained
beyond calls to `exerror`. This is a small step towards deprecating SFIO.

Gitlab: #1873

expr: inline private state into 'Excc_t'

Now that `Excc_t` is encapsulated in excc.c and no longer considered an
extensible struct, we can stop playing macro tricks to give it extra fields and
just express these more naturally.

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/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

smyrna toolboxcallbacks: squash -Wunused-parameter warnings

These functions are used as Glade callbacks, so need to conform to particular
type signatures.

smyrna topviewsettings: squash -Wunused-parameter warnings

These functions are used as Glade callbacks, so need to conform to particular
type signatures.

smyrna load_settings_from_graph: remove unused 'g' parameter

tclpkg tclGdSizeCmd: squash an -Wunused-parameter warning

This parameter cannot easily be removed because this function is used as a
callback and needs to conform to a particular type signature.

Merge branch 'smattr/d5eaf8b2-870d-4ef0-9557-51428f498a1b' into 'main'

misc simplification

See merge request graphviz/graphviz!2768