]> granicus.if.org Git - graphviz/log
graphviz
2 years agoGTK plugin: squash -Wsign-compare, -Wsign-conversion warnings for width/height
Matthew Fernandez [Mon, 5 Sep 2022 17:22:42 +0000 (10:22 -0700)]
GTK plugin: squash -Wsign-compare, -Wsign-conversion warnings for width/height

Graphviz deals with these values as unsigned, but the GTK APIs deal with them as
signed. Nevertheless we expect both sides to only ever deal in non-negative
values.

2 years agoGTK plugin attr_value_edited_cb: remove shadowing of global 'path'
Matthew Fernandez [Mon, 5 Sep 2022 17:13:56 +0000 (10:13 -0700)]
GTK plugin attr_value_edited_cb: remove shadowing of global 'path'

2 years agoGTK plugin: remove unnecessary 'g_object_get_data' casts
Matthew Fernandez [Mon, 5 Sep 2022 17:12:20 +0000 (10:12 -0700)]
GTK plugin: remove unnecessary 'g_object_get_data' casts

2 years agoGTK plugin: squash -Wunused-parameter warnings in callbacks
Matthew Fernandez [Mon, 5 Sep 2022 17:09:11 +0000 (10:09 -0700)]
GTK plugin: squash -Wunused-parameter warnings in callbacks

These functions are used as Glade callbacks, so need to conform to particular
type signatures.

2 years agoGTK plugin load_store_with_attrs: remove unused 'job' parameter
Matthew Fernandez [Mon, 5 Sep 2022 17:02:52 +0000 (10:02 -0700)]
GTK plugin load_store_with_attrs: remove unused 'job' parameter

2 years agoMerge branch 'smattr/6c968265-3d0c-4a22-b74e-4ceb14eb6dd2' into 'main'
Matthew Fernandez [Sat, 10 Sep 2022 15:38:30 +0000 (15:38 +0000)]
Merge branch 'smattr/6c968265-3d0c-4a22-b74e-4ceb14eb6dd2' into 'main'

Stable release 6.0.0

See merge request graphviz/graphviz!2825

2 years agoStable release 6.0.0
Matthew Fernandez [Mon, 5 Sep 2022 15:33:27 +0000 (08:33 -0700)]
Stable release 6.0.0

2 years agoMerge branch 'smattr/e4241336-151a-41aa-8ffa-ec73b00d6f93' into 'main'
Matthew Fernandez [Sat, 10 Sep 2022 01:05:05 +0000 (01:05 +0000)]
Merge branch 'smattr/e4241336-151a-41aa-8ffa-ec73b00d6f93' into 'main'

smyrna warning squashing

See merge request graphviz/graphviz!2830

2 years agosmyrna: use an 'unsigned' for 'topviewcache.edgelabel_id'
Matthew Fernandez [Wed, 7 Sep 2022 04:53:40 +0000 (21:53 -0700)]
smyrna: use an 'unsigned' for 'topviewcache.edgelabel_id'

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

2 years agosmyrna: use an 'unsigned' for 'topviewcache.nodelabel_id'
Matthew Fernandez [Wed, 7 Sep 2022 04:49:51 +0000 (21:49 -0700)]
smyrna: use an 'unsigned' for 'topviewcache.nodelabel_id'

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

2 years agosmyrna: use an 'unsigned' for 'topviewcache.seledge_id'
Matthew Fernandez [Wed, 7 Sep 2022 04:45:53 +0000 (21:45 -0700)]
smyrna: use an 'unsigned' for 'topviewcache.seledge_id'

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

2 years agosmyrna: use an 'unsigned' for 'topviewcache.selnode_id'
Matthew Fernandez [Wed, 7 Sep 2022 04:42:00 +0000 (21:42 -0700)]
smyrna: use an 'unsigned' for 'topviewcache.selnode_id'

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

2 years agosmyrna: use an 'unsigned' for 'topviewcache.edge_id'
Matthew Fernandez [Wed, 7 Sep 2022 04:37:59 +0000 (21:37 -0700)]
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`.

2 years agosmyrna: use an 'unsigned' for 'topviewcache.node_id'
Matthew Fernandez [Wed, 7 Sep 2022 04:30:51 +0000 (21:30 -0700)]
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`.

2 years agosmyrna renderSelectedNodes: squash -Wfloat-conversion in call to 'glprintfglut'
Matthew Fernandez [Wed, 7 Sep 2022 04:17:55 +0000 (21:17 -0700)]
smyrna renderSelectedNodes: squash -Wfloat-conversion in call to 'glprintfglut'

2 years agosmyrna: remove unnecessary casts of 'defaultnodealpha'
Matthew Fernandez [Wed, 7 Sep 2022 04:12:58 +0000 (21:12 -0700)]
smyrna: remove unnecessary casts of 'defaultnodealpha'

2 years agosmyrna parseArgs: remove shadowing of 'view' global
Matthew Fernandez [Wed, 7 Sep 2022 04:10:37 +0000 (21:10 -0700)]
smyrna parseArgs: remove shadowing of 'view' global

2 years agosmyrna: use a 'size_t' for 'ViewInfo.mouse_action_count'
Matthew Fernandez [Wed, 7 Sep 2022 04:08:34 +0000 (21:08 -0700)]
smyrna: use a 'size_t' for 'ViewInfo.mouse_action_count'

Squashes two -Wsign-conversion warnings.

2 years agosmyrna load_mouse_actions: fix comment typo
Matthew Fernandez [Wed, 7 Sep 2022 04:03:30 +0000 (21:03 -0700)]
smyrna load_mouse_actions: fix comment typo

2 years agosmyrna load_mouse_actions: remove shadowing of 'line' global
Matthew Fernandez [Wed, 7 Sep 2022 04:02:58 +0000 (21:02 -0700)]
smyrna load_mouse_actions: remove shadowing of 'line' global

2 years agosmyrna mTestgvpr: use a 'size_t' for 'argc'
Matthew Fernandez [Wed, 7 Sep 2022 03:56:37 +0000 (20:56 -0700)]
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`.

2 years agosmyrna load_graph_properties: remove unused 'graph' parameter
Matthew Fernandez [Wed, 7 Sep 2022 03:49:08 +0000 (20:49 -0700)]
smyrna load_graph_properties: remove unused 'graph' parameter

2 years agoMerge branch 'smattr/1e70f4b0-5e30-43aa-ac19-5736454aeb2c' into 'main'
Matthew Fernandez [Fri, 9 Sep 2022 15:42:08 +0000 (15:42 +0000)]
Merge branch 'smattr/1e70f4b0-5e30-43aa-ac19-5736454aeb2c' into 'main'

more compiler warning squashing

See merge request graphviz/graphviz!2832

2 years agogvpr: squash -Wundef warning for 'DEBUG'
Matthew Fernandez [Thu, 8 Sep 2022 03:57:57 +0000 (20:57 -0700)]
gvpr: squash -Wundef warning for 'DEBUG'

2 years agogvpr TFA_Advance: squash warnings in calling ctype.h functions
Matthew Fernandez [Thu, 8 Sep 2022 03:56:00 +0000 (20:56 -0700)]
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.

2 years agogvpr TFA_Advance: squash -Wconversion warning from 'tolower' return
Matthew Fernandez [Thu, 8 Sep 2022 03:54:51 +0000 (20:54 -0700)]
gvpr TFA_Advance: squash -Wconversion warning from 'tolower' return

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

2 years agogvpr assignable: make it clearer that argument to 'TFA_Advance' fits in a char
Matthew Fernandez [Thu, 8 Sep 2022 03:46:34 +0000 (20:46 -0700)]
gvpr assignable: make it clearer that argument to 'TFA_Advance' fits in a char

Squashes a -Wconversion warning.

2 years agogvpr getval: squash a -Wswitch-default warning
Matthew Fernandez [Thu, 8 Sep 2022 03:40:24 +0000 (20:40 -0700)]
gvpr getval: squash a -Wswitch-default warning

This switch is exhaustive.

2 years agoosage layout: remove shadowing of 'p'
Matthew Fernandez [Thu, 8 Sep 2022 03:36:45 +0000 (20:36 -0700)]
osage layout: remove shadowing of 'p'

2 years agosfdpgen: remove shadowing of 'UNMATCHED'
Matthew Fernandez [Thu, 8 Sep 2022 03:25:58 +0000 (20:25 -0700)]
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.

2 years agoMerge branch 'smattr/gitlab-2272' into 'main'
Matthew Fernandez [Fri, 9 Sep 2022 14:42:04 +0000 (14:42 +0000)]
Merge branch 'smattr/gitlab-2272' into 'main'

Revert "cgraph: rewrite scanner to use an agxbuf"

Closes #2272

See merge request graphviz/graphviz!2831

2 years agoRevert "cgraph: rewrite scanner to use an agxbuf"
Matthew Fernandez [Thu, 8 Sep 2022 03:04:06 +0000 (20:04 -0700)]
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>
2 years agoadd a test case for #2272
Matthew Fernandez [Thu, 8 Sep 2022 02:46:10 +0000 (19:46 -0700)]
add a test case for #2272

2 years agoMerge branch 'add-test-case-for-issue-372' into 'main'
Magnus Jacobsson [Tue, 6 Sep 2022 20:28:48 +0000 (20:28 +0000)]
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

2 years agotests: add new test_edge_node_overlap_simple test
Magnus Jacobsson [Wed, 24 Aug 2022 07:37:07 +0000 (09:37 +0200)]
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.

2 years agotests: SVGAnalyzer: add SVGRect::intersection()
Magnus Jacobsson [Wed, 24 Aug 2022 07:48:19 +0000 (09:48 +0200)]
tests: SVGAnalyzer: add SVGRect::intersection()

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

2 years agotests: SVGAnalyzer: GraphvizEdge: add retrieval of outline bounding box
Magnus Jacobsson [Tue, 23 Aug 2022 17:13:15 +0000 (19:13 +0200)]
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.

2 years agotests: SVGAnalyzer: GraphvizNode: add retrieval of outline bounding box
Magnus Jacobsson [Tue, 23 Aug 2022 17:12:47 +0000 (19:12 +0200)]
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.

2 years agotests: SVGAnalyzer: SVGElement: add retrieval of outline bounding box
Magnus Jacobsson [Tue, 23 Aug 2022 16:14:28 +0000 (18:14 +0200)]
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.

2 years agotests: SVGAnalyzer: add SVGElement::miter_point()
Magnus Jacobsson [Tue, 23 Aug 2022 16:08:02 +0000 (18:08 +0200)]
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.

2 years agoMerge branch 'smattr/a1e8ee3e-5d71-47c0-8eb0-8ed4dea82aa0' into 'main'
Matthew Fernandez [Tue, 6 Sep 2022 15:28:11 +0000 (15:28 +0000)]
Merge branch 'smattr/a1e8ee3e-5d71-47c0-8eb0-8ed4dea82aa0' into 'main'

label: use cgraph wrappers for allocation

See merge request graphviz/graphviz!2826

2 years agolabel doxlabel: use cgraph wrappers for allocation
Matthew Fernandez [Mon, 5 Sep 2022 15:48:51 +0000 (08:48 -0700)]
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.

2 years agolabel xlnew: use cgraph wrapper for allocation
Matthew Fernandez [Mon, 5 Sep 2022 15:48:16 +0000 (08:48 -0700)]
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.

2 years agolabel xlhdxload: use cgraph wrapper for allocation
Matthew Fernandez [Mon, 5 Sep 2022 15:45:38 +0000 (08:45 -0700)]
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.

2 years agolabel: remove unused memory.h include
Matthew Fernandez [Mon, 5 Sep 2022 15:45:21 +0000 (08:45 -0700)]
label: remove unused memory.h include

2 years agocorrect spelling in comment
Magnus Jacobsson [Tue, 6 Sep 2022 06:47:35 +0000 (08:47 +0200)]
correct spelling in comment

2 years agoMerge branch 'smattr/e10a9b96-cc44-484e-afb0-09213bcdf3dd' into 'main'
Matthew Fernandez [Mon, 5 Sep 2022 23:20:54 +0000 (23:20 +0000)]
Merge branch 'smattr/e10a9b96-cc44-484e-afb0-09213bcdf3dd' into 'main'

plugins: use cgraph wrappers for allocation

See merge request graphviz/graphviz!2823

2 years agocommon: remove now unused 'N_GGNEW'
Matthew Fernandez [Sun, 4 Sep 2022 19:37:40 +0000 (12:37 -0700)]
common: remove now unused 'N_GGNEW'

2 years agoGD plugin vrml_polygon: fix unchecked allocation failure
Matthew Fernandez [Sun, 4 Sep 2022 19:36:28 +0000 (12:36 -0700)]
GD plugin vrml_polygon: fix unchecked allocation failure

2 years agoGD plugin vrml_begin_job: use cgraph wrapper for allocation
Matthew Fernandez [Sun, 4 Sep 2022 19:35:51 +0000 (12:35 -0700)]
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.

2 years agocore plugin: remove unused memory.h include
Matthew Fernandez [Sun, 4 Sep 2022 19:35:28 +0000 (12:35 -0700)]
core plugin: remove unused memory.h include

2 years agocore plugin insert: fix unchecked allocation failure
Matthew Fernandez [Sun, 4 Sep 2022 19:34:47 +0000 (12:34 -0700)]
core plugin insert: fix unchecked allocation failure

2 years agocore plugin write_edges: use cgraph wrapper for allocation
Matthew Fernandez [Sun, 4 Sep 2022 19:34:11 +0000 (12:34 -0700)]
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.

2 years agoMerge branch 'extend-svg-analyzer-to-handle-graphviz-penwidth-pencolor-and-fillcolor...
Magnus Jacobsson [Mon, 5 Sep 2022 08:05:24 +0000 (08:05 +0000)]
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

2 years agotests: SVGAnalyzer: add support for retrieving the Graphviz edge fillcolor
Magnus Jacobsson [Tue, 23 Aug 2022 09:25:17 +0000 (11:25 +0200)]
tests: SVGAnalyzer: add support for retrieving the Graphviz edge fillcolor

2 years agotests: SVGAnalyzer: add support for retrieving the Graphviz node fillcolor
Magnus Jacobsson [Sat, 13 Aug 2022 13:53:15 +0000 (15:53 +0200)]
tests: SVGAnalyzer: add support for retrieving the Graphviz node fillcolor

2 years agotests: test_utilities: declare unmodified parameter const
Magnus Jacobsson [Tue, 23 Aug 2022 11:52:45 +0000 (13:52 +0200)]
tests: test_utilities: declare unmodified parameter const

2 years agotests: SVGAnalyzer: add handling of the 'fill-opacity' attribute
Magnus Jacobsson [Wed, 10 Aug 2022 14:40:44 +0000 (16:40 +0200)]
tests: SVGAnalyzer: add handling of the 'fill-opacity' attribute

2 years agotests: SVGAnalyzer: add support for retrieving the Graphviz edge color
Magnus Jacobsson [Tue, 23 Aug 2022 08:55:14 +0000 (10:55 +0200)]
tests: SVGAnalyzer: add support for retrieving the Graphviz edge color

2 years agotests: SVGAnalyzer: add support for retrieving the Graphviz node color
Magnus Jacobsson [Wed, 10 Aug 2022 13:54:00 +0000 (15:54 +0200)]
tests: SVGAnalyzer: add support for retrieving the Graphviz node color

2 years agotests: SVGAnalyzer: factor out rgb_to_hex() and add support for opacity
Magnus Jacobsson [Thu, 11 Aug 2022 18:51:55 +0000 (20:51 +0200)]
tests: SVGAnalyzer: factor out rgb_to_hex() and add support for opacity

2 years agotests: SVGAnalyzer: clarify comment
Magnus Jacobsson [Tue, 23 Aug 2022 11:26:37 +0000 (13:26 +0200)]
tests: SVGAnalyzer: clarify comment

2 years agotests: SVGAnalyzer: add handling of the 'stroke-opacity' attribute
Magnus Jacobsson [Wed, 10 Aug 2022 12:33:02 +0000 (14:33 +0200)]
tests: SVGAnalyzer: add handling of the 'stroke-opacity' attribute

2 years agotests: SVGAnalyzer: add support for retrieving the Graphviz edge penwidth
Magnus Jacobsson [Tue, 23 Aug 2022 08:26:24 +0000 (10:26 +0200)]
tests: SVGAnalyzer: add support for retrieving the Graphviz edge penwidth

2 years agotests: SVGAnalyzer: add support for retrieving the Graphviz node penwidth
Magnus Jacobsson [Wed, 10 Aug 2022 09:26:23 +0000 (11:26 +0200)]
tests: SVGAnalyzer: add support for retrieving the Graphviz node penwidth

2 years agotests: SVGAnalyzer: add handling of the 'stroke-width' attribute
Magnus Jacobsson [Wed, 10 Aug 2022 07:36:08 +0000 (09:36 +0200)]
tests: SVGAnalyzer: add handling of the 'stroke-width' attribute

2 years agotests: SVGAnalyzer: add 're_create_and_verify_svg' method and use in test_svg_analyzer
Magnus Jacobsson [Sat, 20 Aug 2022 16:12:54 +0000 (18:12 +0200)]
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.

2 years agotests: SVGAnalyzer: extend 'make_from_dot' method and use in test_svg_analyzer
Magnus Jacobsson [Sat, 20 Aug 2022 16:12:54 +0000 (18:12 +0200)]
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.

2 years agotests: SVGAnalyzer: save a copy of the original SVG and give access to it
Magnus Jacobsson [Fri, 2 Sep 2022 09:57:40 +0000 (11:57 +0200)]
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.

2 years agoMerge branch 'smattr/6be69cc6-9ab8-4a2a-ae79-087aec1d9525' into 'main'
Matthew Fernandez [Mon, 5 Sep 2022 00:25:34 +0000 (00:25 +0000)]
Merge branch 'smattr/6be69cc6-9ab8-4a2a-ae79-087aec1d9525' into 'main'

ortho: use cgraph wrappers for allocation

See merge request graphviz/graphviz!2821

2 years agoortho mkMaze: use cgraph wrappers for allocation
Matthew Fernandez [Sat, 3 Sep 2022 18:13:54 +0000 (11:13 -0700)]
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.

2 years agoortho mkMazeGraph: use cgraph wrappers for allocation
Matthew Fernandez [Sat, 3 Sep 2022 18:13:16 +0000 (11:13 -0700)]
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.

2 years agoortho PQgen: use cgraph wrappers for allocation
Matthew Fernandez [Sat, 3 Sep 2022 18:10:42 +0000 (11:10 -0700)]
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.

2 years agoortho partition: use cgraph wrappers for allocation
Matthew Fernandez [Sat, 3 Sep 2022 18:09:49 +0000 (11:09 -0700)]
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.

2 years agoortho monotonate_trapezoids: use cgraph wrappers for allocation
Matthew Fernandez [Sat, 3 Sep 2022 18:09:19 +0000 (11:09 -0700)]
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.

2 years agoortho mkStack: use cgraph wrappers for allocation
Matthew Fernandez [Sat, 3 Sep 2022 18:03:48 +0000 (11:03 -0700)]
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.

2 years agoortho make_graph: use cgraph wrappers for allocation
Matthew Fernandez [Sat, 3 Sep 2022 18:03:24 +0000 (11:03 -0700)]
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.

2 years agoortho createSGraph: use cgraph wrappers for allocation
Matthew Fernandez [Sat, 3 Sep 2022 18:02:14 +0000 (11:02 -0700)]
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.

2 years agoortho initSEdges: use cgraph wrappers for allocation
Matthew Fernandez [Sat, 3 Sep 2022 18:01:45 +0000 (11:01 -0700)]
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.

2 years agoortho construct_trapezoids: use cgraph wrapper for allocation
Matthew Fernandez [Sat, 3 Sep 2022 18:00:14 +0000 (11:00 -0700)]
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.

2 years agoortho orthoEdges: use cgraph wrappers for allocation
Matthew Fernandez [Sat, 3 Sep 2022 17:58:35 +0000 (10:58 -0700)]
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.

2 years agoortho attachOrthoEdges: use cgraph wrapper for allocation
Matthew Fernandez [Sat, 3 Sep 2022 17:58:02 +0000 (10:58 -0700)]
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.

2 years agoortho extractVChans: use cgraph wrapper for allocation
Matthew Fernandez [Sat, 3 Sep 2022 17:57:40 +0000 (10:57 -0700)]
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.

2 years agoortho extractHChans: use cgraph wrapper for allocation
Matthew Fernandez [Sat, 3 Sep 2022 17:57:12 +0000 (10:57 -0700)]
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.

2 years agoortho addChan: use cgraph wrapper for allocation
Matthew Fernandez [Sat, 3 Sep 2022 17:56:31 +0000 (10:56 -0700)]
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.

2 years agoortho convertSPtoRoute: use cgraph wrapper for allocation
Matthew Fernandez [Sat, 3 Sep 2022 17:55:41 +0000 (10:55 -0700)]
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.

2 years agoMerge branch 'smattr/gitlab-2270' into 'main'
Matthew Fernandez [Sun, 4 Sep 2022 23:31:49 +0000 (23:31 +0000)]
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

2 years agoRevert "gvc auto_output_filename: avoid 'strdup' when constructing file name"
Matthew Fernandez [Sun, 4 Sep 2022 18:44:01 +0000 (11:44 -0700)]
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>
2 years agoadd a test case for #2270
Matthew Fernandez [Sun, 4 Sep 2022 18:43:38 +0000 (11:43 -0700)]
add a test case for #2270

2 years agoMerge branch 'smattr/1fac7330-9d40-4de2-be68-925e698f5285' into 'main'
Matthew Fernandez [Sun, 4 Sep 2022 18:24:56 +0000 (18:24 +0000)]
Merge branch 'smattr/1fac7330-9d40-4de2-be68-925e698f5285' into 'main'

smyrna: migration to cgraph wrappers for allocation

See merge request graphviz/graphviz!2820

2 years agotests: SVGAnalyzer: add 'make_from_dot' static method and use in test_rankdir
Magnus Jacobsson [Tue, 23 Aug 2022 07:11:02 +0000 (09:11 +0200)]
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.

2 years agosmyrna: remove unused memory.h includes
Matthew Fernandez [Fri, 2 Sep 2022 02:03:36 +0000 (19:03 -0700)]
smyrna: remove unused memory.h includes

2 years agosmyrna load_attributes: use cgraph wrapper for allocation
Matthew Fernandez [Fri, 2 Sep 2022 01:57:58 +0000 (18:57 -0700)]
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.

2 years agosmyrna main: use cgraph wrapper for allocations
Matthew Fernandez [Fri, 2 Sep 2022 01:54:37 +0000 (18:54 -0700)]
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.

2 years agosmyrna smyrnaPath: use cgraph wrapper for allocation
Matthew Fernandez [Fri, 2 Sep 2022 01:53:49 +0000 (18:53 -0700)]
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.

2 years agosmyrna initFocus: use cgraph wrapper for allocations
Matthew Fernandez [Fri, 2 Sep 2022 01:51:33 +0000 (18:51 -0700)]
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.

2 years agosmyrna positionAllItems: use cgraph wrapper for allocations
Matthew Fernandez [Fri, 2 Sep 2022 01:51:09 +0000 (18:51 -0700)]
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.

2 years agosmyrna makeXDotSpline: use cgraph wrapper for allocations
Matthew Fernandez [Fri, 2 Sep 2022 01:48:54 +0000 (18:48 -0700)]
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.

2 years agosmyrna mTestgvpr: use cgraph wrapper for allocation
Matthew Fernandez [Fri, 2 Sep 2022 01:46:04 +0000 (18:46 -0700)]
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.

2 years agosmyrna new_viewport_camera: use cgraph wrapper for allocation
Matthew Fernandez [Fri, 2 Sep 2022 01:44:28 +0000 (18:44 -0700)]
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.