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.

inline tol and maxit from power_method into power.c as constants

The function power_method is only called in a single location. There it is
called with constants for tol and maxit. So it is simpler to remove these as
parameters and just specify them directly next to power_method.

remove now-unused improve_antibandwidth_by_swapping_cheap

remove unnecessary do_swapping parameter to country_graph_coloring_internal

This function is only ever called with do_swapping == DO_SWAPPING.

remove an open coded NULL in cluster.c

make cluster usage string a constant array

Minor clarity change that makes it more obvious to readers and the compiler that
this string is a constant that can be inlined at its usage site.

gvmap: uniformly treat clock() results as clock_t or double

The return type of clock(), clock_t, is an unspecified real type. The usual way
to deal with it is to upcast it to a double when doing computation or printing,
as is done in this commit. Squashes 8 -Wconversion warnings.

remove a useless return statement

mark clusterGraph as static

This function is not used outside of its containing file.

Merge branch 'smattr/4BAD66AE-4F6D-4F40-96A2-755E25EF6F5A' into 'main'

some clean up of lib/sparse/general.c

See merge request graphviz/graphviz!2015

remove unused vector_take

remove unused validQ_int_string

remove unused digitsQ

remove unused excute_system_command3

Similar to the previous commit, this function seems to have a typoed name and
unsafe behavior.

remove unused excute_system_command

This function appears to have a typo in its name and also do something quite
unsafe.

remove unused vector_percentile

remove unused vector_median

remove ascending parameter to vector_sort_int

The only calls to this uses ascending, so having this configurable was adding
unnecessary maintenance burden.

remove unused vector_sort_real

remove ascending parameter from vector_ordering

All calls to this use ascending, so having this configurable was adding
unnecessary maintenance burden.

Merge branch 'smattr/1b923de5-43ac-4445-adfe-fd9b1512bd2b' into 'main'

cleanup configure.ac

See merge request graphviz/graphviz!2002

remove legacy advice to build on Fedora 8

Fedora 8 was EOLed in 2009. Compiling Graphviz on it in 2021 is not a journey
likely to end in happiness.

update Graphviz URL in AC_INIT configure setup

This URL was out of date (the current Graphviz URL is https://graphviz.org/),
but the third parameter of AC_INIT is also expected to be the bug tracker, not
the project’s homepage.

remove configure check for strerror

The strerror function is in ISO C, even prior to C99. POSIX defines some
extensions to it, but Graphviz does not rely on any of these. In either case,
there is no need to link against anything beyond libc.

remove commented out Tclsh configure checks

remove commented out D configure checks

remove commented out Go configure checks

remove commented out JavaScript configure checks

remove commented out TCL AC_SUBST configure commands

adjust indentation after prior commit

remove commented out Visio configure checks

remove commented out ANN configure checks

remove GD_GIFANIM configure checks

Nothing in Graphviz depends on this.

remove GD_OPENPOLYGON configure checks

Nothing in Graphviz depends on this.

remove legacy __eprintf dependency check

This check was forcing assertions off if using them depends on __eprintf. It is
unclear to me what the motivation for this is and it was introduced in the
initial revision with no further information. As far as I am aware, this is no
longer relevant. Modern libc is intended to provide everything necessary to
support assert, unless you are building freestanding which Graphviz does not
support.

Merge branch 'smattr/9ca97a2a-a1dc-4631-9310-ded139978324' into 'main'

clean up in lib/sfdpgen

See merge request graphviz/graphviz!2000

replace lib/sfdpgen header guards with more modern #pragma once

remove unused flag argument to uniform_stress_solve

remove unused flag argument to cg

remove unused flag parameter to conjugate_gradient

remove another pow micro-optimization in spring_electrical.c

A slightly different variant of the instances dealt with in the prior commit.

remove pow micro-optimizations in spring_electrical.c

This code was attempting to optimize the case when the second argument to pow()
was 2. However, modern compilers are able to understand such nuances. In fact,
at -O3 recent Clang *undoes* this branching code into an unconditional call to
pow(). In other words, it considers this branch to multiplication to be a
de-optimization.

This change also incidentally removes 14 -Wfloat-equal warnings.

remove unused jd argument to dmean_get