remove unused SETCLOEXEC and FD_CLOEXEC

The last use of these was removed in f552c0dece1435773de62334dee3310d1728e22f.

remove unused ESPIPE

ESPIPE has never been used in sfio.

remove unused ENOMEM

The last use of this in sfio was removed in
bb04aad8c6e033b6fef9cecd424a4d02758d661a.

remove cbrt configuration check

Graphviz is built with C99 which requires cbrt.

remove unlink configuration check and Windows shim

Nothing in Graphviz uses unlink.

remove remove/unlink declarations in sfio

The last use of these was removed in 751ab539a2531b5459ef709946ea0642ce2c44e8.

remove fork/vfork detection

As of the previous commit, nothing uses these configuration variables.

remove unused sfio fork/vfork declarations

The last usage of fork in this library was removed in
f552c0dece1435773de62334dee3310d1728e22f.

remove unnecessary errno declaration

The errno variable/macro is part of ISO C.

remove unused MAXSHORT

remove unused MAXLONG

Merge branch 'smattr/77f377e7-e733-48ca-8003-6de107b48820' into 'main'

test cases for #2089

See merge request graphviz/graphviz!2023

test cases for #2089

remove prototype for unimplemented agrelabel

I do not know why this function was described as “scary.” However, it seems it
was so scary no one in Graphviz’ history ever implemented it.

Merge branch 'smattr/259855E7-1FFB-4A36-8116-EA81D402CFCD' into 'main'

fix macOS CI

See merge request graphviz/graphviz!2030

remove Courier.gv graph reference test

This test compares the rendering of a graph using the Courier font against
reference examples. However the rendering of Courier is not stable across
operating system releases. At present, this test is spuriously failing due to a
macOS upgrade. But in principle, an upgrade of any platform can break it. To
avoid testing this unstable behavior, we remove this test.

remove unused font_tests.txt

This seems to be a subset of tests covered elsewhere. This file is also
currently unused.

Update README.md -- queries go to Graphviz forum

Should keep bug tracker full of bugs, and give people with other queries a better chance at getting them answered (there are more people on the forum).

Merge branch 'smattr/94045bf0-ac99-4edb-acba-84c65c67d983' into 'main'

clean up in lib/vpsc

See merge request graphviz/graphviz!2020

remove unused Rectangle {x|y}Border

use a range-based for-loop to simplify remapOutConstraints

use a range-based for-loop to simplify remapInConstraints

rephrase accumulate loop in Blocks::cost using a range-based for-loop

rephrase delete loop in Blocks::~Blocks to a range-based for-loop

remove unnecessary clear of Blocks set

The object is being destructed, so the underlying set is automatically cleared.

remove unused getSplitCnt

remove unused splitIncVPSC

remove duplicate solve_VPSC.h in VPSC noinst_HEADERS list

remove unused newVPSC

Merge branch 'smattr/49e76f9c-dbd6-42d8-bcc1-cbe0a523441e' into 'main'

clean up in lib/sparse

See merge request graphviz/graphviz!2019

remove unused StringVector

remove unused IntegerVector

remove unused SparseMatrix_pseudo_diameter_weighted

remove unused SparseMatrix_pseudo_diameter_unweighted

remove pattern_symmetric_only parameter to SparseMatrix_symmetrize_nodiag

This function is only called with FALSE for this argument, so we can propagate
it through and remove the parameter.

remove duplicate palette_grey_to_red

remove another unused MAXINT

remove unused MAXINT

remove unused vector_subtract_from

remove unused vector_add_to

Merge branch 'smattr/584ca143-13f9-4b18-aecc-1195d5de7644' into 'main'

clean up in lib/common

See merge request graphviz/graphviz!2018

reflow some routespl.c code for readability

remove dead code in get_centroid

The preceding return statement ensures none of the latter part of the function
is reachable. This was introduced in 05674ae147c14ec4ec74b2b6771e04e0c82209b8
and seemingly never noticed.

This commit also drops now-unused variables, as well as the static qualifier
from sum. It is not clear to me what the purpose of making sum static was. It is
a small struct and both its fields are overwritten immediately when entering
this function, so there is no value to retaining it across calls or
BSS-allocating it.

remove a micro-optimization in checkpath

Self-stores like this are effectively free on a modern CPU. The branch guarding
this store is more costly than the store itself.

use C99 booleans to clarify some ambiguous types in routespl.c

remove unnecessary bracketing in lib/common/routespl.c

remove casts of vec_get’s return value

The vec_get function returns a void* that implicitly coerces to other pointer
types.

condense some compound assignments

remove __CYCLE_CENTROID and conditional code for when it is undefined

This was unconditionally defined, so remove the alternative that is always
disabled.

rephrase open coded fmax/fmin in checkpath

_routesplines: remove unnecessary cast

pp->data is a void*, which implicitly coerces to a edge_t*.

remove commented out code in lib/common

remove DONTFIXPATH guards

These guards were present in the very first revision of Graphviz, but nothing
has ever defined them.

remove NOTNOW-guarded code

Nothing in the build defines this.

Merge branch 'fix-typos-GD_LIBS-to-GDLIB_LIBS' into 'main'

Fix a typo GD_LIBS to GDLIB_LIBS in tclpkg/tcldot/Makefile.am

See merge request graphviz/graphviz!2022

add a CHANGELOG.md entry for the GTK→GDLIB change

This fix and the GD_LIBS→GDLIB_LIBS fix are both addressing problems that were
introduced in 3fcf096870bffa7d21fceecb56ba26e925073ecb.

move GD_LIBS typo fix to Fixes section of CHANGELOG.md

realign configure.ac after removal of $with_gd conditional

Avoid cases where `$with_libgd = yes` in spite of `$use_gd != Yes`

Such cases would have occurred when
the first `test "x$use_gd" = "x"` [1] is true
and then the third `test "x$use_gd" = "x"` [2] is false.

[1]: Just before `PKG_CHECK_MODULES([GDLIB], [gdlib >= 2.0.33],[`
[2]: Just before `# see if we can use the external gd lib`

Remove a redundant `if`-conditional, which is always true in its scope

Fix a typo `GDK` to `[GDLIB]` in a call `PKG_CHECK_MODULES()`

Remove a redundant arg `$GD_LIBS`

It might be a typo of `$GDLIB_LIBS`,
and, if so, it is redundant, since `$LDFLAGS` contains `$GDLIB_LIBS`.

Update CHANGELOG.md

For https://gitlab.com/graphviz/graphviz/-/merge_requests/2022

Fix a typo GD_LIBS to GDLIB_LIBS in tclpkg/tcldot/Makefile.am

Merge branch 'smattr/EFF6337A-A391-409A-9B11-F11C26A76481' into 'main'

CMake: when finding Bison, require ≥3.0

Closes #1916

See merge request graphviz/graphviz!2008

CMake: when finding Bison, require ≥3.0

Commit 4a13c9598ad3d950a251e5d3e5be8d05c59c6214 made the minimum Bison baseline
3.0. This change teaches CMake ths constraint, so using a version of Bison <3.0
now results in a clearer error during configuration rather than a cryptic
failure during build. Closes #1916.

Merge branch 'smattr/5c6b4c6f-b22b-49a6-8607-7a4b5caa120a' into 'main'

clean up in lib/edgepaint

See merge request graphviz/graphviz!2007

remove color_diff and color_diff_sum computation from node_distinct_coloring

Neither of the callers of this function use the results returned in these output
parameters.

remove commented out code in node_distinct_coloring.c

remove some variable shadowing in node_distinct_coloring_internal2

Squashes two -Wshadow warnings.

remove unused x parameter to splines_intersect

remove unused color_blend_rgb

remove unused color_blend_rgbstring

return errors from node_distinct_coloring via the return value, rather than flag

This makes things simpler for readers, callers, as well as the compiler.

remove commented out code in edge_distinct_coloring.c

remove unused lab_gamut_from_file

fix: incorrect failure diagnosis in lab_gamut_quadtree call

This function can only fail due to exhausting memory, not due to the lab_gamut
file failing to be opened. As per the previous commit, this file is no longer
relevant as the gamut information is in the source itself.

remove unused gamut_file parameter to lab_gamut_quadtree

Commit 5649b9fa3dd7699750cfd69753ab4d54d29e469f added the gamut data in the
source code itself, so it does not need to be read in from a file.

Merge branch 'smattr/2738CB52-612E-47E9-8950-4DA41DAB9A7D' into 'main'

some clean up around usage of zmalloc

See merge request graphviz/graphviz!2010

remove some commented out code in gvusershape_open

remove unnecessary cast in gvusershape_open

The result of find_user_shape, shape_desc*, implicitly coerces to void*.

remove redundant error check in gvusershape_open

The zmalloc function calls exit() on failure, so there is not need to check its
return value for out-of-memory.

remove unnecessary cast in gvjobs_next

The result of zmalloc, void*, implicitly coerces to all other pointer types.

remove redundant error check in gvNEWcontext

The zmalloc function calls exit() on failure, so there is not need to check its
return value for out-of-memory.

remove redundant error check in push_obj_state

The zmalloc function calls exit() on failure, so there is not need to check its
return value for out-of-memory.

Merge branch 'smattr/3FE7CC37-7FE8-4BD6-9C4E-C4A17C4ACB68' into 'main'

introduce branch-tuning support

See merge request graphviz/graphviz!2011

mark verbose reporting in gvParseArgs as unlikely

This indicates to the compiler it should predict the condition of this branch as
false in its optimization choices.

add macros for branch-tuning optimization

Merge branch 'smattr/D9E1B66F-05C1-4156-960B-F48BF3FA06EC' into 'main'

add a better way of indicating unreachable code

See merge request graphviz/graphviz!2009

fix some comment typos

swap assertion in agdelrec for a call to UNREACHABLE

This removes some -Wstring-conversion warnings when building with Clang, due to
the expression `!"unreachable"`. It also teaches the compiler it can more
aggressively optimize this switch.

swap assertion in setattr for a call to UNREACHABLE

This removes some -Wstring-conversion warnings when building with Clang, due to
the expression `!"unreachable"`. It also teaches the compiler it can more
aggressively optimize this switch.

add a macro for optimizing unreachable code

Merge branch 'smattr/C266706C-CF93-4076-8F45-297A35F08DDC' into 'main'

only enable -Wtrampolines and -Wlogical-op for compilers that support them

Closes #2066

See merge request graphviz/graphviz!2004

only enable -Wtrampolines and -Wlogical-op for compilers that support them

This fixes an issue where these flags were enabled for Clang, despite it not
supporting them. This did not fail the build, but produced a lot of noisy
warning output about unsupported flags. Closes #2066.

Merge branch 'smattr/d7229a4a-c7dd-4aaa-bbd2-016b3839b8e1' into 'main'

test cases for ortho routing bugs and typo fix in eqEndSeg

Closes #2047

See merge request graphviz/graphviz!2006

remove excess bracketing in eqEndSeg

fix typo in eqEndSeg

This fixes a typo which was causing this function to check equality incorrectly.
Unfortunately this seems to have no effect on any open ortho-related bugs.
Closes #2047.

add a test case for #144

add a test case for #1658