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

add a test case for #56

add a test case for #1880

add a test case for #1856

add a test case for #1990

add a test case for #14

add a test case for #1408

Merge branch 'smattr/d00934a1-c2db-4d93-b033-bfda973477bf' into 'main'

more clean up in lib/sparse

See merge request graphviz/graphviz!2005

remove unused SparseMatrix_scaled_by_vector

remove unused lib/sparse macros

remove unused dump_coordinates

remove unused SparseMatrix_read_dot

remove prototype for unimplemented function

This function was removed in 32ad37113a52a6818b19c6ab0554bed224da2a1d, but its
prototype was left behind.

remove unused makeDotGraph

remove unused convert_edge_labels_to_nodes

remove unused assign_random_edge_color

remove unused edgelist_export

remove unused hue2rgb_real

Merge branch 'smattr/4776D4AA-9B85-4010-BF10-ED5243311BE9' into 'main'

clean up in lib/cgraph/io.c

See merge request graphviz/graphviz!2012

Update CHANGELOG.md

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

Merge branch 'fix-path-of-gvc.def' into 'main'

Fix a path of gvc.def specified in libgvc_la_LDFLAGS (WITH_WIN32)

See merge request graphviz/graphviz!2017

Fix a path of gvc.def specified in libgvc_la_LDFLAGS (WITH_WIN32)

remove some unnecessary bracketing in memiofread

remove unnecessary cast in memiofread

A void* implicitly coerces to an rdr_t*.

remove unnecessary cast in ioflush

The function fflush takes a FILE* to which void* implicitly coerces.

remove unnecessary cast in ioputstr

The function fputs takes a FILE* as its second argument, to which void*
implicitly coerces.

squash a -Wsign-conversion warning in iofread

The preceding call to fgets can only have returned at most bufsize - 1
characters, so it is safe to downcast the size_t result of strlen to an int.

remove unnecessary cast in iofread

The function fgets takes a FILE* as its third argument, to which void*
implicitly coerces.

Merge branch 'smattr/5a012c5d-e9c4-4395-97b1-ef472d03fc83' into 'main'

clean up in cmd/gvmap

See merge request graphviz/graphviz!2014

fix some comment typos

remove some unnecessary bracketing

remove dynamic allocation capability of power_method for eigs

The behavior to dynamically allocate memory for eigs was not documented (unlike
the parameter eigv). However, the only caller was not using this anyway. This
commit removes it to ease ongoing maintenance.