expr: [nfc] remove unnecessary parens and dereference of 'getf'

The C compiler knows how to dereference and call through a function pointer
without needing this manual instruction.

Merge branch 'smattr/1197F5FA-7D54-45A7-961E-EC4BB50BCFE2' into 'main'

cgraph: fix some minor bugs

See merge request graphviz/graphviz!2475

cgraph: fix: handle allocation failures in 'agcanon' and friends

There are a number of cgraph API functions that use an internally managed buffer
to save the caller from having to allocate space themselves. Failure to expand
this buffer was being silently ignored, resulting in messy crashes when memory
was exhausted. These failures are now checked for and `agcanon` and `agcanonStr`
return `NULL` on allocation failure. `agwrite` returns `EOF` on allocation
failure.

agwrite: fix: ignore out of range 'linelength'

This code was accepting large negative numbers and then converting them to
positive numbers that were applied as the line length limit. This seems clearly
unintended. This rephrasing now ignores any out of range value set for
`linelength`.

agwrite: [nfc] separate assignment from conditional

Squashes an MSVC “warning C4706: assignment within conditional expression”
warning.

Merge branch 'smattr/D7055808-9A1D-4DCD-90A7-E0484002CF03' into 'main'

Start 3.0 development series

See merge request graphviz/graphviz!2478

Start 3.0 development series

Merge branch 'smattr/5DA70CF4-980D-4F81-AEF7-058695CA2E40' into 'main'

Stable Release 3.0.0

See merge request graphviz/graphviz!2461

Stable Release 3.0.0

Merge branch 'smattr/51EA14FF-2973-4E74-9B56-9E7681B60DBB' into 'main'

add a test case for #191

See merge request graphviz/graphviz!2474

add a test case for #191

Merge branch 'smattr/6BECC6DB-0939-436D-839C-A2159C2C25A3' into 'main'

agnameof: [nfc] separate assignments from conditionals

See merge request graphviz/graphviz!2473

agnameof: [nfc] separate assignments from conditionals

Squashes a number of MSVC “C4706: assignment within conditional expression”
warnings.

Merge branch 'fix-warnings' into 'main'

fix freeTreeList

See merge request graphviz/graphviz!2449

fix freeTreeList

Merge branch 'smattr/0A79D579-4787-4FF1-AC4B-BE5FFA581FE8' into 'main'

cgraph: [nfc] remove unused return value from 'agxbmore'

See merge request graphviz/graphviz!2471

cgraph: [nfc] remove unused return value from 'agxbmore'

Merge branch 'smattr/C7E834B5-E093-4BCD-B39E-39DDFA0CE179' into 'main'

lib/pathplan/shortest.c: [nfc] some clean up and warning squashing

See merge request graphviz/graphviz!2469

lib/pathplan/shortest.c: [nfc] separate assignments from conditionals

Squashes a number of MSVC “C4706: assignment within conditional expression”
warnings.

pathplan connecttris: [nfc] remove unnecessary use of the comma operator

lib/pathplan/shortest.c: [nfc] reflow some code for readability

lib/pathplan/shortest.c: [nfc] remove unnecessary parens

Merge branch 'smattr/9662124d-546f-46ec-86b2-59fdac3d6e8a' into 'main'

various test cases for #2185

See merge request graphviz/graphviz!2463

various test cases for #2185

Merge branch 'graphml2gv' into 'main'

tools: remove unused arguments

See merge request graphviz/graphviz!2470

Merge branch 'smattr/139c540a-38bd-48cf-806c-10652625b583' into 'main'

add a test case for #2193

See merge request graphviz/graphviz!2462

add a test case for #2193

tests: recognize 'canon' as a textual format

Gitlab: related to #2193

Merge branch 'smattr/39320B11-3CE5-4D7B-9C8E-17152F849788' into 'main'

cdt: [nfc] some clean up

See merge request graphviz/graphviz!2466

cdt: [nfc] remove unnecessary dereference and parens on function pointers

The compiler knows how to call through a function pointer without requiring the
programmer to manually dereference it.

cdt: [nfc] remove some unnecessary casts of 'void*' during assignment

cdt: [nfc] remove an unnecessary cast of an argument to '_DTKEY'

cdt: [nfc] remove an unnecessary cast of the return value of '_DTOBJ'

cdt: [nfc] remove unnecessary casts of arguments to 'searchf'

cdt: [nfc] remove unnecessary casts of arguments to 'eventf'

cdt: [nfc] remove unnecessary casts of the return value of 'memoryf'

cdt: [nfc] remove unnecessary casts of the arguments to 'memoryf'

cdt dtvsearch: [nfc] squash unused variable warnings from MSVC

Merge branch 'smattr/864EBF21-0976-4521-B6B5-7C21A337FD00' into 'main'

Windows: remove 'sed' as a setup build utility

See merge request graphviz/graphviz!2465

tools: remove unused argument in setGlobalEdgeAttr

fix warning:
cmd/tools/graphml2gv.c: In function ‘setGlobalEdgeAttr’:
cmd/tools/graphml2gv.c:348:71: warning: unused parameter ‘ud’ [-Wunused-parameter]
  348 | setGlobalEdgeAttr(Agraph_t * g, char *name, char *value, userdata_t * ud)
      |

tools: remove unused argument in setGlobalNodeAttr

cmd/tools/graphml2gv.c: In function ‘setGlobalNodeAttr’:
cmd/tools/graphml2gv.c:298:71: warning: unused parameter ‘ud’ [-Wunused-parameter]
  298 | setGlobalNodeAttr(Agraph_t * g, char *name, char *value, userdata_t * ud)
      |                                                          ~~~~~~~~~~~~~^~

Windows: remove 'sed' as a setup build utility

Gitlab: #2115

Windows: remove usage of 'sed' in setup-build-utilities script

This results in `$PATH` potentially having duplicated trailing entries, but this
causes no harm and preventing it does not seem worth a `sed` dependency.

Gitlab: #2115

Merge branch 'aglasterr' into 'main'

lib/cgraph: use return value of 'fread' in aglasterr

See merge request graphviz/graphviz!2468

lib/cgraph: use return value of 'fread' in aglasterr

fix warning:
agerror.c: In function ‘aglasterr’:
agerror.c:49:5: warning: ignoring return value of ‘fread’, declared with attribute warn_unused_result [-Wunused-result]
   49 |     fread(buf, sizeof(char), len, agerrout);

Merge branch 'fix' into 'main'

refactor circularLayout

See merge request graphviz/graphviz!2467

refactor circularLayout

cut function cleanup, eliminate warning:
./lib/circogen/circular.c: In function ‘cleanup’:
./lib/circogen/circular.c|56 col 50| warning: unused parameter ‘sp’ [-Wunused-parameter]
||    56 | static void cleanup(block_t * root, circ_state * sp)
||       |                                     ~~~~~~~~~~~~~^~

Merge branch 'smattr/46A5B7F0-9139-42B6-90B3-E35965040565' into 'main'

lib/common: [nfc] remove 'nnames' micro-optimization and 'nToName'

See merge request graphviz/graphviz!2453

lib/common: [nfc] replace 'nToName' with 'itos'

This code was attempting to avoid dynamic string construction with low numbered
IDs. This is unnecessary when using a modern compiler that understands functions
like `snprintf` as built-ins and can achieve better optimization without this
trick.

Apart from de-duplicating code, this has the effect of making this code thread
safe where it was not before.

Merge branch 'fix-C++-API-not-usable-after-install' into 'main'

fix C++ API not usable after install

Closes #2196

See merge request graphviz/graphviz!2464

add a CHANGELOG entry for the C++ API not usable after install fix

gvc++: include "AGraph.h" instead of <AGraph.h> in GVLayout.h

This makes it possible to use GVLayout.h from the install directory
without the need to specify both "include" and "include/graphviz" as
include directories to the compiler.

Fixes https://gitlab.com/graphviz/graphviz/-/issues/2196.

gvc++: include "gvc.h" instead of <gvc.h> in GVContext.h

This makes it possible to use GVContext.h from the install directory
without the need to specify both "include" and "include/graphviz" as
include directories to the compiler.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2196.

cgraph++: include "cgraph.h" instead of <cgraph.h> in AGraph.h

This makes it possible to use AGraph.h from the install directory
without the need to specify both "include" and "include/graphviz" as
include directories to the compiler.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2196.

gvc++: include AGraph.h without the cgraph++ directory prefix in GVLayout.h

This makes it possible to use GVLayout.h from the install directory
where AGraph.h is not located under this prefix.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2196.

gvc++: include gvc.h without the gvc directory prefix in GVContext.h

This makes it possible to use GVContext.h from the install directory
where gvc.h is not located under this prefix.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2196.

cgraph++: include cgraph.h without the cgraph directory prefix in AGraph.h

This makes it possible to use AGraph.h from the install directory
where cgraph.h is not located under this prefix.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2196.

Merge branch 'smattr/5E2D9BBC-3087-4FC2-8C95-929FC239F0CC' into 'main'

diffimg: handle allocation errors

See merge request graphviz/graphviz!2455

diffimg: handle allocation errors

Previously these would be ignored resulting in crashes when allocations failed.

Merge branch 'smattr/3ECB75F8-D9B2-4B09-9DD4-DCEF26761D98' into 'main'

cgraph: [nfc] guard 'strcasecmp' definition to avoid duplication

See merge request graphviz/graphviz!2456

cgraph: [nfc] guard 'strcasecmp' definition to avoid duplication

When compiling with Microsoft Visual Studio 2019, the compiler emits:

  …\lib\cgraph\strcasecmp.h(12,64): warning C4211: nonstandard extension used:
    redefined extern to static

Investigation reveals libgd defines their own `strcasecmp` shim in gd.h:

  /* VS2012+ disable keyword macroizing unless _ALLOW_KEYWORD_MACROS is set
     We define inline, and strcasecmp if they're missing
  */
  #ifdef _MSC_VER
  #  define _ALLOW_KEYWORD_MACROS
  #  ifndef inline
  #    define inline __inline
  #  endif
  #  ifndef strcasecmp
  #    define strcasecmp _stricmp
  #  endif
  #endif

To avoid this warning, guard Graphviz’ shim so it is only used when the libgd
shim is not in use.

Note that this also explains why MSVC is able to compile diffimg.c despite the
Graphviz shim file not being included.

Merge branch 'smattr/dc60dea5-b999-463f-976a-13c2b7443e26' into 'main'

[nfc] remove 'NOTUSED' and propagate its definition everywhere

Closes #2195

See merge request graphviz/graphviz!2457

[nfc] remove 'NOTUSED' and propagate its definition everywhere

Gitlab: closes #2195

Merge branch 'smattr/B20B33D7-255E-4435-A2DD-403C0F96EC13' into 'main'

MS Build: remove references to non-existent ReadMe.txt files

See merge request graphviz/graphviz!2459

MS Build: remove references to non-existent ReadMe.txt files

Merge branch 'smattr/9e779c9a-a15f-478d-b576-789a6bdba462' into 'main'

fix 'gvpr' usage output

See merge request graphviz/graphviz!2460

fix 'gvpr' usage output

Commit 83eea6075b8f4746d632a792b1d1b97c19797acb altered the calling convention
of `errorv`, but failed to take into account that the local variable `s` was
being used for something else prior to its use as the first unpacked parameter.
Following this commit, the first parameter was being discarded as `s` was
overwritten with `error_info.id` or `id`.

This fix restores the previous behavior.

add a test case demonstrating a GVPR regression

Merge branch 'smattr/45757D7F-6696-4553-985F-16E4C839E43D' into 'main'

tclpkg vgpanecmd: remove acceptance of ambiguous abbreviated commands

Closes #1961

See merge request graphviz/graphviz!2458

tclpkg vgpanecmd: remove acceptance of ambiguous abbreviated commands

`vgpanecmd` accepted abbreviations of commands, e.g. `li` for list. However
these abbreviations were accepted as first match with no disambiguation. So `r`
would always be interpreted as `rotate`, not `remove`. This behavior appears to
be an unintentional side effect of commands being incrementally introduced over
time without this kind of prefix-matching overlap being taken into account,
though this is mostly a guess as all this code was implemented prior to the
first Git commit.

Gitlab: fixes #1961

Merge branch 'smattr/294959A0-076C-47FA-AF24-40225D33D7A0' into 'main'

CI: remove '$OS_ID'

See merge request graphviz/graphviz!2454

CI: [nfc] remove writing of OS_ID file

As of the prior commit, nothing uses this.

CI: [nfc] remove setting of '$OS_ID'

As of the prior commit, nothing uses this variable.

CI: remove the use of '$OS_ID' during testing

This commit alters the CI tests to now pull OS information from /etc/os-release
instead of from an ad hoc environment variable. This makes test case execution
independent from which/whether environment variables have been exported. The
intent with this is to reduce the likelihood of an issue like that fixed in
50db0c5e8da293d816d8b5c4620f2fa2d311d37a being reintroduced.

Merge branch 'smattr/4DE02E85-5B6C-45CD-BB3C-4C7183C5DD33' into 'main'

neatogen: [nfc] remove 'DEVDBG' guarded code

See merge request graphviz/graphviz!2452

neatogen: [nfc] remove 'DEVDBG' guarded code

Nothing in the build systems define this, but it is also unclear how to use this
debugging support as the psdbg.c support functions seem to be something internal
that was never committed to the repository.

Merge branch 'smattr/08C6E85C-C8A0-48CB-B812-98567782EBAC' into 'main'

upgrade Pylint dependency

See merge request graphviz/graphviz!2442

upgrade Pylint dependency

Merge branch 'smattr/4F517B52-B8A0-4A70-9E3E-AAB95452CC48' into 'main'

fix: Run feasible_tree() even if maxiter is zero

Closes #1902

See merge request graphviz/graphviz!2448

Move maxiter check after the feasible_tree() call.

Edit from Matt: changelog entry and un-xfailed the test case.

Quoting the description of MR !1688 if it ever ends up lost to the sands of
time:

  If `nslimit1` is zero, then `rank2` is called with `maxiter=0`. It is likely
  the intent here is to at least compute and return an initial feasible solution
  (though not running any network simplex iterations).

Gitlab: fixes #1902

Co-authored-by: Matthew Fernandez <matthew.fernandez@gmail.com>

tests: [nfc] conform with double quotes for strings in #1902

tests: [nfc] use newer 'dot' helper in #1902

tests: [nfc] use more modern 'pathlib' in #1902

Regression test for issue 1902

Edit from Matt: Added `xfail` marker as I have reordered the commits to put this
test case introduction before its fix, as well as aligning with current Pylint
requirements.

Gitlab: #1902

Co-authored-by: Matthew Fernandez <matthew.fernandez@gmail.com>

Merge branch 'smattr/5DE18E14-8822-4DB4-96DF-550AEDEBC599' into 'main'

CI: some minor clean up of steps

See merge request graphviz/graphviz!2451

CI: [nfc] abbreviate metrics file creation

Slightly more efficient and concise.

CI: remove log file deletion steps

The CI environments are transient and start fresh each time, so there is no need
to clean up temporary files.

Merge branch 'smattr/C3D5797C-DD58-4F74-AD17-DE8C39AEFA1D' into 'main'

neatogen: some simplification

See merge request graphviz/graphviz!2447

neatogen: [nfc] brace a single-line if block

neatogen: [nfc] remove some temporaries

These seemed to be making the code less clear rather than more clear.

neatogen max_absf: replace open coded max with a call to 'fmaxf'

neatogen max_abs: replace open coded max with a call to 'fmax'

neatogen: [nfc] remove a no-op cast

neatogen: replace float-casted calls to 'fabs' with calls to 'fabsf'

This is simpler, more efficient, and clearer to the compiler.

neatogen: replace float-casted calls to 'sqrt' with calls to 'sqrtf'

This is simpler, more efficient, and clearer to the compiler.

Revert "work around a bug in GCC: It generates a call to "sqrtf" for "(float)sqrt(d)""

This reverts commit 0dfd6ab3449455907a5a684fb54cb8b54c5befd5.

As of fe3f9411d2c59b463ab1b64eecfd19f2db55d2fc, Graphviz requires a C99
compiler. C99 guarantees `sqrtf` so even if the “bug” discussed in the original
commit still exists, generating a `sqrtf` call is fine on a C99 toolchain.

neatogen: [nfc] brace some single-line if blocks

Revert "more sqrt"

This reverts commit ef77679d9fd51ac7a1a69e5a2635c2280856cd85. This commit was
working around a GCC bug. Quoting its predecessor
0dfd6ab3449455907a5a684fb54cb8b54c5befd5 that gives context:

  work around a bug in GCC: It generates a call to "sqrtf" for "(float)sqrt(d)"
  but sqrtf doesn't exist in libm.a.
  reported by: Aaron Digulla <digulla@hepe.com>

As of fe3f9411d2c59b463ab1b64eecfd19f2db55d2fc, Graphviz requires a C99
compiler. C99 guarantees `sqrtf` so even if this “bug” still exists, generating
a `sqrtf` call is fine on a C99 toolchain.

neatogen: [nfc] reflow some code

This appears to be two copies of the same copy-pasted code.

neatogen: [nfc] rephrase some boolean accumulation

These operations can be written simpler using bitwise-or assignment.