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

hsv2rgb: squash a -Wswitch-default warning

The preceding logic makes the rest of the switch exhaustive.

gvrender_beziercurve: realign 'filled' parameter type

This appears to have been a mistake, in that all the other `gvrender_*`
functions take an `int` for the `filled` parameter. This does not appear to have
caused any visible effects, as the values passed to this function always fit in
a `boolean`. But the compiler complained that it thought unintended value
truncation was occurring.

Squashes 3 -Wconversion compiler warnings.

Merge branch 'smattr/4AE39E22-FD66-458A-81E4-F54395DA6F4D' into 'main'

remove 'real' and 'REAL' abstractions

See merge request graphviz/graphviz!2274

gvmap: remove unused 'REAL' definitions

remove 'real' abstraction

This was not configurable and always mapped to the type `double`. There were
also implicit assumptions that `real` and `double` were interchangeable. This
alias is removed to avoid future confusion.

The diff appears large, but it is just removing two #defines of `real` and
propagating their definition (`double`) to all usage sites.

Merge branch 'smattr/D885CA52-BAA6-4360-A822-28B64F491590' into 'main'

fix assertion typo that overwrote matrix value type and stop disabling assertions

See merge request graphviz/graphviz!2275

general.h: stop disabling assertions when 'DEBUG' is undefined

The effect of this code was that #including general.h prior to assert.h would
disable assertions in your code. Not #including general.h or #including it after
assert.h would allow assertions to be controlled through the standard `NDEBUG`
mechanism. It is not clear why suppressing assertions like this was ever
desirable.

This is basically another variant of commit
68bcbbd4ddef4960c7ea6884dda10854b0e012e0.

jacobi: fix assertion typo that overwrote matrix value type

Comparing to the surrounding functions in this file, this was clearly meant to
be checking the type of the matrix not setting it. This was benign as, due to a
separate problem, assertions are disabled in this file, making this a no-op. The
next commit will re-enable assertions, which caused this issue to be discovered
through a compiler error.