smyrna: use an 'unsigned' for 'topviewcache.edge_id'

Squashes 4 -Wsign-conversion warnings. Note that this also changes the sentinel
from `-1` to `UINT_MAX`.

smyrna: use an 'unsigned' for 'topviewcache.node_id'

Squashes 4 -Wsign-conversion warnings. Note that this also changes the sentinel
from `-1` to `UINT_MAX`.

smyrna renderSelectedNodes: squash -Wfloat-conversion in call to 'glprintfglut'

smyrna: remove unnecessary casts of 'defaultnodealpha'

smyrna parseArgs: remove shadowing of 'view' global

smyrna: use a 'size_t' for 'ViewInfo.mouse_action_count'

Squashes two -Wsign-conversion warnings.

smyrna load_mouse_actions: fix comment typo

smyrna load_mouse_actions: remove shadowing of 'line' global

smyrna mTestgvpr: use a 'size_t' for 'argc'

This squashes a -Wsign-conversion warning. We need to cast back to `int` for
calling `gvpr`, but this is acceptable as the number of arguments cannot
practically exceed `INT_MAX`.

smyrna load_graph_properties: remove unused 'graph' parameter

Merge branch 'smattr/1e70f4b0-5e30-43aa-ac19-5736454aeb2c' into 'main'

more compiler warning squashing

See merge request graphviz/graphviz!2832

gvpr: squash -Wundef warning for 'DEBUG'

gvpr TFA_Advance: squash warnings in calling ctype.h functions

Some standard libraries implement the ctype.h functions as macros in a way that
triggers compiler warnings if the parameter is not int-typed.

gvpr TFA_Advance: squash -Wconversion warning from 'tolower' return

GCC is overly pedantic about the return type of the ctype.h functions.

gvpr assignable: make it clearer that argument to 'TFA_Advance' fits in a char

Squashes a -Wconversion warning.

gvpr getval: squash a -Wswitch-default warning

This switch is exhaustive.

osage layout: remove shadowing of 'p'

sfdpgen: remove shadowing of 'UNMATCHED'

These constants were added in 1b847b5abf50d2ab0701523b90a20306d0ee5528 and it is
unclear why, given they are unused. They shadow a different constant in
general.h, causing compiler warnings.

Merge branch 'smattr/gitlab-2272' into 'main'

Revert "cgraph: rewrite scanner to use an agxbuf"

Closes #2272

See merge request graphviz/graphviz!2831

Revert "cgraph: rewrite scanner to use an agxbuf"

This reverts commit 2d7852a049b729fb0db71c5a8ef2366215ae2035. This commit
introduced a bug where the string buffer was assumed to be uninitialized when
beginning parsing a string, but it may not be if a previously unterminated
string was processed.

Github: fixes #2272
Reported-by: Rob Hart <robhart@google.com>

add a test case for #2272

Merge branch 'add-test-case-for-issue-372' into 'main'

Add edge and node overlap test case for issue #372

See merge request graphviz/graphviz!2824

tests: add new test_edge_node_overlap_simple test

Upcoming commit series will add additional such tests so most of the
code is in a separate utilities file and is somewhat more generalized
than what is strictly necessary for this first simple test.

tests: SVGAnalyzer: add SVGRect::intersection()

This will be used in an upcoming commit the determine overlap between
edges and nodes.

tests: SVGAnalyzer: GraphvizEdge: add retrieval of outline bounding box

This will be used in an upcoming commit that determines the overlap
between the outline bounding box of a node and an edge.

tests: SVGAnalyzer: GraphvizNode: add retrieval of outline bounding box

This will be used in an upcoming commit that determines the overlap
between the outline bounding box of a node and an edge.

tests: SVGAnalyzer: SVGElement: add retrieval of outline bounding box

This will be used in an upcoming commit that determines the overlap
between the outline bounding box of a node and an edge.

tests: SVGAnalyzer: add SVGElement::miter_point()

This will be used in an upcoming commit to calculate the outline
bounding box, i.e., the bounding box when taking stroke width into
account.

Merge branch 'smattr/a1e8ee3e-5d71-47c0-8eb0-8ed4dea82aa0' into 'main'

label: use cgraph wrappers for allocation

See merge request graphviz/graphviz!2826

label doxlabel: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

label xlnew: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

label xlhdxload: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

label: remove unused memory.h include

correct spelling in comment

Merge branch 'smattr/e10a9b96-cc44-484e-afb0-09213bcdf3dd' into 'main'

plugins: use cgraph wrappers for allocation

See merge request graphviz/graphviz!2823

common: remove now unused 'N_GGNEW'

GD plugin vrml_polygon: fix unchecked allocation failure

GD plugin vrml_begin_job: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

core plugin: remove unused memory.h include

core plugin insert: fix unchecked allocation failure

core plugin write_edges: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

Merge branch 'extend-svg-analyzer-to-handle-graphviz-penwidth-pencolor-and-fillcolor-attributes' into 'main'

Extend the SVG analyzer to handle Graphviz penwidth, pencolor and fillcolor attributes

See merge request graphviz/graphviz!2808

tests: SVGAnalyzer: add support for retrieving the Graphviz edge fillcolor

tests: SVGAnalyzer: add support for retrieving the Graphviz node fillcolor

tests: test_utilities: declare unmodified parameter const

tests: SVGAnalyzer: add handling of the 'fill-opacity' attribute

tests: SVGAnalyzer: add support for retrieving the Graphviz edge color

tests: SVGAnalyzer: add support for retrieving the Graphviz node color

tests: SVGAnalyzer: factor out rgb_to_hex() and add support for opacity

tests: SVGAnalyzer: clarify comment

tests: SVGAnalyzer: add handling of the 'stroke-opacity' attribute

tests: SVGAnalyzer: add support for retrieving the Graphviz edge penwidth

tests: SVGAnalyzer: add support for retrieving the Graphviz node penwidth

tests: SVGAnalyzer: add handling of the 'stroke-width' attribute

tests: SVGAnalyzer: add 're_create_and_verify_svg' method and use in test_svg_analyzer

Upcoming commits will extend the SVG analyzer's ability to handle new
SVG attributes and add new test cases verifying its ability to
re-create correct SVG containing those attributes. This change will
facilitate the creation of those tests and keep the code DRY.

tests: SVGAnalyzer: extend 'make_from_dot' method and use in test_svg_analyzer

This change adds support to the SVGAnalyzer for saving a
copy of the Graphviz version and build ID which are needed to exactly
re-create the SVG document.

Upcoming commits will extend the SVG analyzer's ability to handle new
SVG attributes and add new test cases verifying its ability to
re-create correct SVG containing those attributes. This change will
facilitate the creation of those tests and keep the code DRY.

tests: SVGAnalyzer: save a copy of the original SVG and give access to it

This is needed to be able to verify the SVG re-creating capablility of
the SVGAnalyzer.

Upcoming commits will extend the SVG analyzer's ability to handle new
SVG attributes and add new test cases verifying its ability to
re-create correct SVG containing those attributes. This change will
facilitate the creation of those tests and keep the code DRY.

The copying is necessary since the original SVG is modified by the XML
parser in destructive mode. See
http://rapidxml.sourceforge.net/manual.html#namespacerapidxml_1destructive_non_destructive
and we don't want to use the non-destructive mode since no entity
reference translation is done then and we want to avoid having to do
that ourselves.

Merge branch 'smattr/6be69cc6-9ab8-4a2a-ae79-087aec1d9525' into 'main'

ortho: use cgraph wrappers for allocation

See merge request graphviz/graphviz!2821

ortho mkMaze: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho mkMazeGraph: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho PQgen: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho partition: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho monotonate_trapezoids: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho mkStack: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho make_graph: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho createSGraph: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho initSEdges: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho construct_trapezoids: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho orthoEdges: use cgraph wrappers for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho attachOrthoEdges: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho extractVChans: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho extractHChans: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho addChan: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

ortho convertSPtoRoute: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

Merge branch 'smattr/gitlab-2270' into 'main'

Revert "gvc auto_output_filename: avoid 'strdup' when constructing file name"

Closes #2270

See merge request graphviz/graphviz!2822

Revert "gvc auto_output_filename: avoid 'strdup' when constructing file name"

This reverts commit 4291cc769a3eeef8b1c171e5479194733a4da6cd. This commit that
was intended to be a functional no-op actually altered the generated output
filename to remove `.` characters.

Github: xflr6/graphviz #178, NixOS/nixpkgs #188175
Gitlab: fixes #2270
Reported-by: Sebastian Bank <sebastian.bank@uni-leipzig.de>

add a test case for #2270

Merge branch 'smattr/1fac7330-9d40-4de2-be68-925e698f5285' into 'main'

smyrna: migration to cgraph wrappers for allocation

See merge request graphviz/graphviz!2820

tests: SVGAnalyzer: add 'make_from_dot' static method and use in test_rankdir

Upcoming commits will add several new test cases doing exactly
that. This change will keep those tests short and DRY.

smyrna: remove unused memory.h includes

smyrna load_attributes: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna main: use cgraph wrapper for allocations

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna smyrnaPath: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna initFocus: use cgraph wrapper for allocations

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna positionAllItems: use cgraph wrapper for allocations

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna makeXDotSpline: use cgraph wrapper for allocations

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna mTestgvpr: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna new_viewport_camera: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna new_viewport_camera: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna prepare_topological_fisheye: use cgraph wrapper for allocations

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna makeGraph: use cgraph wrapper for allocations

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna attr_list_add: fix unchecked allocation failure

smyrna attr_list_new: fix unchecked allocation failures

smyrna new_attr_ref: fix unchecked allocation failure

smyrna new_attr: fix unchecked allocation failure

smyrna create_color_theme: use cgraph wrapper for allocations

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna init_viewport: use cgraph wrapper for allocations

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.

smyrna add_graph_to_viewport: fix unchecked allocation failure

smyrna load_mouse_actions: fix unchecked allocation failure

smyrna add_selpoly: fix unchecked allocation failure