sparse SingleLinkedList_new_int: fix unchecked allocation failure

sparse SingleLinkedList_new: 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-2332' into 'main'

Autotools: remove legacy `python` discovery

See merge request graphviz/graphviz!3022

Autotools: remove legacy `python` discovery

When removing Python 2 support, the discovery paths that checked for the Python
interpreter as `python` were retained to support older distros in the Red Hat
ecosystem where Python 3 goes by the name `python`. All currently supported
distros use the name `python3`, so this code is no longer needed.

Gitlab: related to #2332

Merge branch 'smattr/baf8fad8-8d84-44e2-8ff6-1ac1ee98a0e2' into 'main'

sfdpgen: remove unused control options

See merge request graphviz/graphviz!3021

sfdpgen: remove unused 'maximal_independent_vertex_set'

sfdpgen: remove unused 'maximal_independent_vertex_set_RS'

sfdpgen: remove unused 'maximal_independent_edge_set'

sfdpgen: remove unused 'maximal_independent_edge_set_heavest_edge_pernode'

sfdpgen: remove unused 'maximal_independent_edge_set_heavest_edge_pernode_leaves_first'

sfdpgen: remove unused 'maximal_independent_edge_set_heavest_cluster_pernode_leaves_first'

sfdpgen: remove unused 'maximal_independent_edge_set_heavest_edge_pernode_scaled'

sfdpgen: remove unused 'D' parameter threaded through functions

sfdpgen: remove 'coarsen_scheme' control parameter that is always set the same

sfdpgen: remove 'randomize' control parameter that is always true

sparse Multilevel_MQ_Clustering_establish: remove unnecessary cast

sparse get_mq: remove unnecessary cast

sparse SparseMatrix_import_dot: remove unnecessary cast

neatogen OverlapSmoother_new: remove unnecessary casts

neatogen ideal_distance_avoid_overlap: remove useless return statement

neatogen fdpAdjust: remove unused return value

gvmap get_polygon_solids: remove commented out code

gvmap get_polygon_solids: remove unnecessary casts

gvmap get_poly_lines: remove unnecessary cast

gvmap plot_dot_polygons: remove unnecessary cast

gvmap improve_contiguity: remove unnecessary cast

sparse SparseMatrix: use a 'size_t' for 'size'

Squashes 20 -Wsign-conversion, -Wconversion warnings.

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

remove gv.php include from PHP demo

Closes #2322

See merge request graphviz/graphviz!3024

remove gv.php include from PHP demo

As of SWIG 4.1.0, PHP source is no longer generated when SWIG is targeting PHP.

Gitlab: closes #2322
Reported-by: Petr Sumbera

Merge branch 'smattr/0f210128-9f5a-42e9-b2cd-9a2a16a12a7c' into 'main'

CI: remove Fedora 35 jobs

See merge request graphviz/graphviz!3013

CI: remove Fedora 35 jobs

Fedora 35 has been EOLed since 2022-12-13¹.

¹ https://docs.fedoraproject.org/en-US/releases/eol/

Merge branch 'smattr/4fc2dbeb-1d26-463e-9258-c6bbfda1b586' into 'main'

smyrna: remove more unused code

See merge request graphviz/graphviz!3012

Merge branch 'smattr/441b433a-bf42-4224-b269-29de6bd0e153' into 'main'

neatogen: fix memory leak

See merge request graphviz/graphviz!3006

neatogen: push queue allocation into 'bfs_bounded'

Similar to the previous commit, the callee does not care about the contents of
the queue on entry and the caller does not care about the contents of the queue
on exit. Note that in this case we need to add an extra parameter because
`bfs_bounded` did not have the queue size already.

neatogen: push queue allocation into 'bfs'

Callers of `bfs` were constructing `Queue` objects and passing them into `bfs`.
But in all of these cases neither the callee nor the caller care about the
contents of the queue. This appears to have been an optimization to hoist
allocation of this object outside loops in callers. This is nowhere close to the
most expensive operation these locations are performing. And replicating
operations like this led to opportunities for errors like that fixed in the
prior commit. It seems a win to undo this for readability.

neatogen embed_graph: fix memory leak of queue

Every call to `mkQueue` is intended to be paired with a call to `freeQueue`.
This function was missing its call to the free function.

neatogen embed_graph: remove free of output parameter

The only call into this function passes `NULL` for the `coords` parameter.
Additionally it did not make much sense to free this pointer on behalf of the
caller. This function is not designed to be called in a loop, so if the caller
wants their passed-in parameter freed they should do it themselves prior to
calling.

Merge branch 'smattr/9d358411-5cf8-4ba0-90f6-97c5708027dc' into 'main'

also allow macOS to fail the long chain test

See merge request graphviz/graphviz!3011

smyrna: remove unused 'dlgOpenGraph_OK_Clicked'

The last use of this was removed in 7d1170f8d5208c296db891f5d93bbb158b1c1b56.

smyrna: remove unused 'on_newNode'

This has never been used.

smyrna set_combobox_widget: remove no-op casts

smyrna: remove unused 'nodesize'

smyrna appmouse_up: remove no-op branches

smyrna: remove unused 'prevX', 'prevY'

These have never been used.

gvedit: remove unused 'initFiles'

smyrna: remove disabled test mode code

smyrna: remove unused top view user mode constants

The last use of these was removed in 34d079565882fbffdf58fe61d0f493274438a699.

smyrna: remove unused 'DEFAULT_FISHEYE_MAGNIFIER_RADIUS'

The last use of this was removed in 95730136091ba690f74d1ef802ca42b3cb4ed71c.

smyrna: remove unused magnifier constants

The last use of these was removed in
6baa97a6e98923364f9d1b9a613f2b1aed87fb1b.

also allow macOS to fail the long chain test

It seems this test can also fail in CI on macOS.

Gitlab: #1710, #1720

Merge branch 'smattr/0ecb9860-22d9-40d2-9f38-d0e434414c10' into 'main'

smyrna: remove some unused code

See merge request graphviz/graphviz!3005

smyrna: remove unused 'defaultnodeshape'

smyrna: remove unused 'filtered_attr_list'

smyrna: remove unused 'magnifier'

smyrna: remove unused 'fisheye_magnifier'

smyrna: remove unused 'FontSize'

smyrna: remove duplicate 'defaultmagnifierwidth' retrieval

The return value of this call was being ignored. It seems accidentally
copy-pasted from the lines below.

smyrna: remove unused 'topviewusermode'

smyrna: remove unused 'edgerendertype'

smyrna set_viewport_settings_from_template: remove shadowing of 'view' global

smyrna get_attribute_value: remove shadowing of 'view' global

smyrna close_graph: remove shadowing of 'view' global

smyrna clear_viewport: remove shadowing of 'view' global

smyrna add_camera_to_viewport: remove shadowing of 'view' global

smyrna new_viewport_camera: remove unused 'view' parameter

smyrna: squash some -Wmissing-prototypes warnings

symrna: squash -Wunused-parameter warnings in data callbacks

smyrna new_attr: remove shadowing of global 'attr'

Note that none of the zero assignments in this function were necessary because
`gv_alloc` zeros memory.

smyrna new_attr_with_ref: remove shadowing of 'attr' global

smyrna new_attr_ref: remove shadowing of global 'attr'

smyrna attr_list_new: remove unused 'g' parameter

common startElement: remove an unnecessary cast

cgraph: fix comment typo

Merge branch 'smattr/ae4b501f-6c51-4f2c-8ccb-b8f433b8685c' into 'main'

common: warning squashing

See merge request graphviz/graphviz!3007

common: remove unnecessary parens

common newItem: 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.

common fillMap: 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.

common cvtAndAppend: use a more appropriate type for 'len'

Squashes a -Wsign-conversion warning.

common scanEntity: use a more appropriate type for 'len'

This addresses a problem where the subtraction of two pointers, `endp - t`, can
in theory exceed the size of an `int`. In practice this cannot occur, but using
a more correct type squashes a -Wconversion and -Wsign-conversion warning in
this code.

common undoClusterEdges: 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.

common undoClusterEdges: remove shadowing of 'elist' global

common gvToggle: squash -Wunused-parameter warning

common late_int: detect and reject int out of range

Squashes a -Wconversion warning.

common new_queue: squash -Wsign-conversion warning

`sz` is known ≥ 2 here.

common new_queue: 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.

Merge branch 'smattr/1b8f5782-a2fc-4215-b3d5-d82c6944966a' into 'main'

remove some dynamic allocations

See merge request graphviz/graphviz!3009

dotgen _dot_splines: remove unnecessary dynamic allocation of 'P'

common init_job_viewport: remove unnecessary 'junk' string allocation

common taper: return 'stroke_t' by value

Saves unnecessary dynamic allocation and caller clean up.

gxl2gv: remove unnecessary dynamic allocation of user data

gv2gxl: remove unnecessary dynamic allocation of state

graphml2gv: remove unnecessary dynamic allocation of user data

dotgen make_flat_adj_edges: remove unnecessary dynamic allocation of 'attrs'

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

fix reading incorrect input in #1710 test case

See merge request graphviz/graphviz!3010

fix reading incorrect input in #1710 test case

A mistake in 632fe0bd1cfc6a4f636db4f85206aff6720bdc6b made this test read from
/dev/null instead of the input file it was supposed to read. Note that this
required some tweak to the skip condition. The Windows platforms on which this
fails seems all over the place and expressing the exact pattern seemed too
complex. For the curious, what we currently see in CI is:

  ┌──────────────┬─────────┬──────────────┬─────────┬────────┐
  │ architecture │ OS      │ build system │ mode    │ result │
  ╞══════════════╪═════════╪══════════════╪═════════╪════════╡
  │ x86          │ Windows │ MS Build     │ debug   │ FAIL   │
  │              │         │              ├─────────┼────────┤
  │              │         │              │ release │ pass   │
  │              │         ├──────────────┼─────────┼────────┤
  │              │         │ CMake        │ debug   │ FAIL   │
  │              │         │              ├─────────┼────────┤
  │              │         │              │ release │ FAIL   │
  │              ├─────────┼──────────────┼─────────┼────────┤
  │              │ MinGW   │ CMake        │ -       │ pass   │
  ├──────────────┼─────────┼──────────────┼─────────┼────────┤
  │ x86-64       │ Windows │ CMake        │ debug   │ FAIL   │
  │              │         │              ├─────────┼────────┤
  │              │         │              │ release │ FAIL   │
  │              ├─────────┼──────────────┼─────────┼────────┤
  │              │ MinGW   │ CMake        │ -       │ FAIL   │
  └──────────────┴─────────┴──────────────┴─────────┴────────┘

I would not be surprised if these results are not stable. It is likely this
failure presents across all platforms, but is dependent on things like Address
Space Layout Randomization to exhibit.

Gitlab: #1710, #1720

Merge branch 'smattr/331f7848-cb40-4753-91cb-c1599c4e3000' into 'main'

Start 7.0.6 development

See merge request graphviz/graphviz!3008

Start 7.0.6 development

Merge branch 'smattr/26ab1e3b-7718-4870-a101-5767137531c5' into 'main'

Stable Release 7.0.5

See merge request graphviz/graphviz!3001

Stable Release 7.0.5

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

CI: disable Cygwin jobs

See merge request graphviz/graphviz!3004