gvusershape.c: squash -Wmissing-prototypes warnings

gvusershape_find: fix mismatch between prototype and implementation

usershape_close: squash -Wunused-parameter warnings

usershape_close: remove unnecessary cast

get_int_msb_first: squash a -Wsign-conversion warning

jpeg_size: mark 'standalone_markers' const

jpeg_size: remove abuse of 'strchr' to search a byte array

Using `memchr` instead avoids the need to cast the array, shrinks the array
itself by avoiding the need for a NUL terminator, and makes it more obvious to
the compiler it can inline and unroll the whole thing.

jpeg_size: remove use of 'junk' variable

This code was reading big endian integers into this variable that was then
ignored as a way of skipping fields. It is simpler and more efficient to just
fast forward the file handle over those bytes. This assumes the file handle is
to something seekable (e.g. not a FIFO), but the handle was already seeked
elsewhere in this function, so this assumption already existed.

mingle usage: remove always-true if branch

This function is only ever called with `eval` non-negative.

Merge branch 'smattr/209CD786-B836-43BD-A488-535300F228D0' into 'main'

cmd/smyrna: more clean up

See merge request graphviz/graphviz!2288

smyrna menucallbacks: squash -Wunused-parameter warnings

smyrna: remove unused 'mHelp'

smyrna: remove unused engine slot functions

smyrna: remove unused 'show_close_nosavedlg'

smyrna: remove unused 'get_gtktextview_text'

smyrna: remove unused 'update_graph_properties'

smyrna: remove unused 'on_attrSearchBtn_clicked'

smyrna: remove unused 'attr_label_motion'

smyrna: make 'attr_list_new' static

This function is not used outside of its containing file.

Merge branch 'smattr/7e500215-5167-44d7-90c2-9e79ec4686dd' into 'main'

cmd/tools warning squashing and clean up

See merge request graphviz/graphviz!2283

gvcolor: remove unnecessary parens

sccmap: remove unnecessary parens

bcomps: remove unnecessary parens

bcomps getName: use size_t for buffer lengths

Squashes some -Wconversion warnings.

bcomps blockName: use size_t for buffer lengths

Squashes some -Wconversion warnings.

gvpack redoBBk: use size_t for buffer lengths

Squashes some -Wconverion warnings.

gvpack freef: squash -Wunused-parameter warnings

gvpack init: use a size_t to squash some -Wconversion warnings

sccmap.c: use libc support to define 'INF'

Squashes a -Wconversion warning.

sccmap.c: return an unsigned type from 'visit'

Squashes a -Wsign-conversion warning.

The `val` member of `Agnodeinfo_t` is unsigned and all uses of this function
store the return value in an unsigned. So it seems this function always should
have been declared as returning an unsigned.

sccmap.c: use unsigned types consistently in sccstate

Squashes a -Wconversion warning.

sccmap.c: remove 'INLINE' alternatives

Modern compilers are easily capable of inlining these functions by themselves.

sccmap.c: use another unsigned type to squash -Wconversion warnings

The `val` member of `Agnodeinfo_t` is unsigned and all uses of this function
store the return value in an unsigned. So it seems this function always should
have been declared as returning an unsigned.

sccmap.c: use an unsigned type to squash some -Wconversion warnings

The `val` member of `Agnodeinfo_t` is unsigned and all uses of this function
store the return value in an unsigned. So it seems this function always should
have been declared as returning an unsigned.

gvcolor.c: remove an unnecessary cast

gvcolor.c: remove duplicated #include

gvcolor.c: use a size_t to squash a -Wsign-conversion warning

Merge branch 'cross' into 'main'

cmd/dot: don't run dot -c when cross-compiling

See merge request graphviz/graphviz!2281

CHANGELOG.md entry for the prior commit

cmd/dot: don't run dot -c when cross-compiling

We probably can't execute the binary we've cross compiled.

Merge branch 'smattr/5E1B768C-337D-4ED1-9693-E6A6B1DFE934' into 'main'

cgraph man page formatting fix

See merge request graphviz/graphviz!2280

add a CHANGELOG entry for the prior commit

Correct boldness of agnxtsubg in cgraph.3

Merge branch 'smattr/6c18e731-115d-4167-92e7-bf96cbbe151e' into 'main'

gmlparse.y: fix all compiler warnings

See merge request graphviz/graphviz!2278

gmlparse.y: remove unnecessary parens

mkGraph: remove shadowing of 'G' parameter

addNodeLabelGraphics: remove unused 'xb' parameter

gmlparse.y: remove commented out code

free_attr: squash -Wunused-parameter warnings

gmlparse.y: consistently use unsigned short types for 'kind'

Squashes a -Wconversion warning.

gmlparse.y: consistently use unsigned short types for 'sort'

Squashes a -Wconversion warning.

gmlparse.y: remove unused 'ushort' typedef

gmlparse.y: use more appropriate stack entry count type

Squashes some -Wsign-conversion warnings.

free_graph: squash -Wunused-parameter warnings

free_edge: squash -Wunused-parameter warnings

free_node: squash -Wunused-parameter warnings

Merge branch 'smattr/0111593e-6ce0-4cf7-9b22-f4a39a08236d' into 'main'

lib/gvc warning squashing and minor bug fix

See merge request graphviz/graphviz!2277

get_int_msb_first: use a C99 bool return type

get_int_lsb_first: use a C99 bool return type

gvusershape.c: remove unnecessary #include

stddef.h is unnecessary if stdlib.h is #included.

remove commented out 'tiff_size'

get_int_msb_first: use a more appropriate type to squash -Wsign-compare warning

get_int_lsb_first: fix undefined behavior in little endian decoding

When this function was called with a byte length of 4 (something that happens
often in this file), the last iteration of this loop would shift an int left by
24. On a platform with 32-bit ints (most platforms Graphviz runs on) this shifts
_into_ the sign bit, something that is undefined behavior with respect to the C
standard.

The changes in this commit make the shift well defined and remove a
-Wsign-conversion warning.

get_int_lsb_first: use a more appropriate type to squash -Wsign-compare warning

imagetype: squash -Wsign-compare warning

gvevent_select_current_obj: remove commented out code

gvevent_select_current_obj: squash -Wconversion warnings

gvevent_leave_obj: squash -Wconversion warnings

gvprintf: squash some -Wsign-conversion warnings

`len` is known to be non-zero in this code path, so these casts are safe.

gvwrite: use more conforming type when calling 'deflate'

This is what the zlib API documents `deflate` as returning. Squashes a
-Wsign-conversion warning.

z_file_header: use an unsigned element type and mark 'const'

Squashes a -Wconversion compiler warnings and this data is never modified.

gvwrite_no_z: avoid assumptions on input data type

This is consistent with how `s` is used agnostic to its type within this
function. This helps us remove some casts and do some upcoming simplification.

Merge branch 'smattr/2E26CA6A-60DD-401E-AF6A-A5821A9115F7' into 'main'

smyrna: more clean up

See merge request graphviz/graphviz!2276

smyrna: consistent use of 'static' on 'get_temp_coords'

This function was already declared static earlier in its containing file, but it
is clearer if all mentions of it are marked `static`.

drawTessPolygon: remove unused return value

smyrnaPath: remove use of int-as-size_t

smyrna: make 'smyrnaGlade' static

This variable is not used outside of its containing file.

smyrna: remove unused 'on_attrApplyAllBtn_clicked'

smyrna: remove unused 'print_attr_list'

smyrna: make 'free_attr_list' static

This function is not used outside of its containing file.

smyrna: make 'attr_list_add' static

This function is not used outside of its containing file.

smyrna: make 'new_attr' static

This function is not used outside of its containing file.

smyrna: make 'free_attr' static

This function is not used outside of its containing file.

smyrna: fix comment typos

smyrna: remove C++ 'extern "C"' guards

Smyrna is written entirely in C, so these guards are unused.

Merge branch 'smattr/c9c2aa68-dc61-4ae6-877e-e332899e1938' into 'main'

plugin/core related warning squashing

See merge request graphviz/graphviz!2282

mp_end_edge: squash -Wunused-parameter warning

mp_begin_edge: squash -Wunused-parameter warning

mp_end_node: squash -Wunused-parameter warning

mp_begin_node: squash -Wunused-parameter warning

mp_begin_page: squash -Wunused-parameter warning

fig_end_edge: squash -Wunused-parameter warning

fig_begin_edge: squash -Wunused-parameter warning

fig_end_node: squash -Wunused-parameter warning

fig_begin_node: squash -Wunused-parameter warning

fig_begin_page: squash -Wunused-parameter warning

core_loadimage_xdot: squash -Wunused-parameter warning

xdot_gradient_fillcolor: use double instead of float for 'r1', 'r2'

Squashes a number of -Wfloat-conversion warnings.

xdot_gradient_fillcolor: remove dead store to 'r1'

This variable is overwritten on line 648 without having been read in between.
This code appears copy-pasted from `cairo_gradient_fill` without noticing that
this line is unused here.

get_gradient_points: take 'angle' as a double instead of a float

The originating values used to compute an angle were generally doubles and
computation was done in `get_gradient_points` as if this value was a double
(e.g. calling `sin` and `cos`). Treating it consistently as a double squashes a
number of -Wfloat-conversion compiler warnings and improves precision.