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.

dot_end_graph: squash -Wswitch-default warning

dot_begin_graph: squash -Wswitch-default warning

write_plain: take a C99 bool for 'extend' parameter

Merge branch 'smattr/72003c76-27a7-431f-95d7-c69e57514be2' into 'main'

lib/common: more warning squashing

See merge request graphviz/graphviz!2273

freeMPair: squash -Wunused-parameter warning

mkMPair: squash -Wunused-parameter warning

cmppair: squash -Wunused-parameter warnings

freePair: squash -Wunused-parameter warnings

estimate_character_width_canonical: squash a -Wsign-conversion warning

font_in_list_permissive: squash -Wsign-conversion warning

The preceding logic guarantees this value will be non-negative.

all_font_metrics_len: squash -Wold-style-declaration warning

textfont_comparf: squash -Wunused-parameter warnings

textfont_freef: squash -Wunused-parameter warnings

textfont_makef: squash -Wunused-parameter warnings