Merge branch 'smattr/70deb7de-4445-488a-be9f-b5f73e1f8d77' into 'main'

fix xdot JSON output to be valid JSON

Closes #1958

See merge request graphviz/graphviz!2295

fix xdot JSON output to be valid JSON

The output of libxdot’s JSON functions included identifiers as non-strings,
something that is not valid JSON. This change results in identifiers like `e`
now being emitted as a string `"e"`.

Fixes #1958.

Merge branch 'kf/gvcdef' into 'main'

gvc.def: Don't override export name

See merge request graphviz/graphviz!2290

Merge branch 'smattr/03501BAF-AD18-474D-B48E-D7A8F81262F3' into 'main'

fail deployment if a release exceeds 50 artifacts

See merge request graphviz/graphviz!2284

fail deployment if a release exceeds 50 artifacts

Following commit 0155cabbb7b22494b3d5da1425059387d8236cf2, the number of
artifacts produced during packaging is down to 40. We expect this count to rise
as the CMake build system continues to be worked on, operating systems move in
and out of support cycles, and the Cygwin/Msys builds are tweaked. This change
introduced a safe guard to fail deployment if the number of artifacts grows to
exceed 50 again.

Related to #2147.

Merge branch 'smattr/e936b86f-3ac0-4234-98eb-79f6d6e43a16' into 'main'

further gvusershape.c clean up

See merge request graphviz/graphviz!2287

jpeg_size: use C99 'true' in loop condition

jpeg_size: use a C99 bool for 'saw_bb' local

svg_size: use C99 bools for local flag variables

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.

gvc.def: Don't override export name

As reported in https://gitlab.com/graphviz/graphviz/-/issues/2156,
the export name override breaks the win32/mingw32 autoconf build.
By not specifying the export name, each toolchain should use its
appropriate default, which should at least work, even if it's
not consistent across toolchains.

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