Smyrna GUI menu callbacks: remove excess space, commented out code, brackets

on_gvprbuttonsave_clicked: remove unused local 'charcnt'

change_cursor: remove unused local 'w'

mGraphPropertiesSlot: remove unused local 'respond'

remove unnecessary execv prototype

This is unused in Graphviz. Related to #1998.

remove unused O_TEMPORARY

Related to #1998.

replace sfio SF_MAXLONG with libc LONG_MAX

Related to #1998.

remove unused SF_MAXINT

Related to #1998.

replace sfio SF_MAXCHAR with libc UCHAR_MAX

Related to #1998.

Merge branch 'smattr/4151f512-15b8-4cea-b0e4-86d7ad442341' into 'main'

more char* → const char* in libcgraph

Closes #634

See merge request graphviz/graphviz!2100

agsafeset: take const pointers for 'value' and 'def' arguments

Neither of these are modified by this function. Closes #634. I believe this is
the last Cgraph function currently able to receive a modification like this.

agset: take a const pointer for 'value' argument that is not modified

Related to #634.

agattrsym: remove a pointless local

remove some use of open coded NULL when calling agattr

agattr: take a const pointer for 'value' argument that is not modified

Related to #634.

global_def: remove 'dclfun' argument

This function is only ever called with `agattr` as this parameter. Removing this
takes care of the sole use of `agattr` with a function pointer, clearing the way
to modify the type signature of `agattr`. Related to #634.

setattr: take a const pointer for 'value' argument that is not modified

Related to #634.

agxset: take a const pointer for 'value' argument that is not modified

Related to #634.

inline and remove ag_xset macro

This did not seem to be adding any value.

agnewsym: take const pointers for string arguments that are not modified

Related to #634.

Merge branch 'smattr/0432D029-37BD-4A2A-B61B-DA05F137EFE1' into 'main'

Remove DATE file and variables

See merge request graphviz/graphviz!2095

autotools: remove some commented out lines

autotools: do not write DATE file

This is not used by CI or other parts of the build system.

do not export DATE file from portable-source CI job

No other CI jobs use this file.

cmake: remove ability to override GRAPHVIZ_VERSION_BUILD

Commit 705ced5fa22ba13f239adfe1ba45713553c1fe05 added the ability to override
the Graphviz version from the command line when configuring the CMake build
system. This is not currently in use and does not seem desirable to maintain
going forwards.

Merge branch 'smattr/dfe2d6e6-3d41-4797-b1b9-3c59049efcf9' into 'main'

remove Fmt_t.tmp and some s/boolean/bool modernization

See merge request graphviz/graphviz!2096

edge_in_box: return a C99 bool

remove unused neato_closest

overlap_bale: return a C99 bool

overlap_node: return a C99 bool

gvevent.c: remove unnecessary casts

overlap_edge: return a C99 bool

swap some unnecessary prototypes for a #include

utils.h: fix missing include

This header references agxbuf objects in function prototypes.

overlap_bezier: use a C99 bool return type

overlap_arrow: use C99 bool return type to simplify

overlap_arrow: remove unused 'flag' parameter

arrow_bb: remove unused 'flag' parameter

squash warnings for unused parameters in Arrowtypes functions

These all need to conform to a common prototype, so many have parameters that
they do not need to use.

simplify iteration over Arrowtypes array

The size of this array can be computed at compile-time, allowing the compiler to
unroll loops over its content.

mark Arrowtypes as const

arrows.c: remove commented out code

remove 'tmp' member from 'Fmt_t'

As of merge 9053aaba8f7791983b493c56b5443bf602cc529b and
f0adc7da80a22230174f4afcec12151e3ae28fad, this member is no longer used. Related
to #1998.

Merge branch 'smattr/1408a71c-89e3-4b1a-8aa9-c088fe4f01bd' into 'main'

fix failed Exshort_t type discrimination

Closes #1799

See merge request graphviz/graphviz!2088

remove unused ERROR_catalog

remove Exshort_t type

This type discrimination did not seem to make much sense (why select between int
and short based on the sizes of long and int?), but more problematically was not
working as intended. This header does not include limits.h, so both LONG_MAX and
INT_MAX were undefined and this expression defaulted to 0 regardless of the
sizes of these types.

Closes #1799.

replace use of Exshort_t type with int

This squashes a number of -Wconversion warnings where int values are written
into these fields. Related to #1799.

remove pad_1 member of Exnode_s

This kind of compile assistance is unnecessary. Compilers can infer the
necessary padding by themselves. Related to #1799.

remove pad_2 member of Exnode_s

This padding seems unnecessary. Nothing relies on the size of this struct.
Related to #1799.

remove %{JAVASCRIPT} build warning during rpmbuild on Fedora

This code was commented out in 05f7c2f6f911c92fcaeafc0a02f6d19537386e0d.
rpmbuild does not like the presence of macros in comments:

  warning: Macro expanded in comment on line 100: %{JAVASCRIPT}

Merge branch 'smattr/3ef81d11-6c0d-45af-8447-2824072f5f40' into 'main'

remove some use of sfputc, sfputr

See merge request graphviz/graphviz!2093

exsub: remove use of sfio temporary buffer

This removes yet another use of `ex->tmp` within exeval.c code. It is replaced
with a dynamically allocated buffer from vmalloc. This not only reduces reliance
on sfio, but is likely more efficient as data is written directly into a vmalloc
buffer rather than being written to an sfio buffer and then copied to a vmalloc
buffer. Related to #1998.

replace: explicitly brace conditionals

Makes this function a little easier to read.

replace: re-indent body

Upcoming changes refactor parts of this function and its current chaotic indent
style make these changes very error prone to apply. It is hoped that this
reformatting makes the upcoming change safer.

strgrpmatch: reflow some text

Revert "remove now-unused vmresize"

This reverts commit ab322d219b7dae03fe5f7114dfd0e1d2fc7d3b5d. A `realloc`
analogue appears to be the simplest way of removing sfputr/sfputc usage in
exeval.c. Related to #1998.

exsub: use a C99 bool for 'global' parameter to be clearer

gvprmain.c: remove unnecessary casts

sfwrite: remove unnecessary casts

Merge branch 'smattr/a7c9e340-7b78-40cd-947b-40e8fc6a5e45' into 'main'

more warning squashing and clean up

See merge request graphviz/graphviz!2094

pathcat: return a const char* instead of char*

Squashes a -Wcast-qual warning, and is generally more honest.

pathcanon: remove some int-as-char type confusion

Removes a -Wconversion warning.

remove PRESERVE_TRAILING_SLASH and DONT_PRESERVE_TRAILING_SLASH

Neither of these are defined anywhere in the build system(s).

fmquote: squash -Wconversion warnings

These operations appear safe and deliberate to me.

fmtquote: remove reuse of an int variable

Squashes 6 -Wsign-conversion/-Wconversion warnings.

sfwrite: squash a -Wsign-conversion warning

The value of `r` here is known between `0` and `sizeof(buf)`.

sfwr: remove unnecessary cast of 'write' argument

The `write` function takes a `const void*`, which `buf` already is.

sfoutput: squash a -Wsign-conversion warning

This value is guaranteed positive in this branch.

remove disabled _lib_wctype-guarded code

remove _hdr_wchar-guarded code

Nothing in the build system(s) define this.

remove isxdigit shim

This function is defined everywhere as far back as ISO C89.

remove _lib_fnmatch-guarded code

Nothing in the build system(s) define this.

Merge branch 'smattr/4aa37954-37b6-4d19-a3b0-c8554069c3d3' into 'main'

more const-ing of API char* arguments

See merge request graphviz/graphviz!2092

rec_apply: take 'preorder' parameter as a C99 bool for clarity

agapply: remove useless comment

This comment is indeed useless. It is clear it is *an* entry point by virtue of
being part of the public API.

aginit: take a const char* instead of char* for record name

This function does not modify its argument. No changelog entry because this is
already covered by the existing #634 one for the upcoming release.

agdelrec: take a const char* instead of char* for record name

This function does not modify its argument. No changelog entry because this is
already covered by the existing #634 one for the upcoming release.

set_data: use a bool for 'mtf'

The externally-facing functions that take a `move_to_front` parameter cannot
easily be updated to take C99 bools without breaking API. But at least the
internal functions can use C99 bools to be clearer.

objputrec: remove unused 'g' parameter

remove unnecessary casts in rec.c

remove unnecessary bracketing in rec.c

agbindrec: realign parameter names in prototype with definition

The C compiler does not care if these names mismatch, but it is less confusing
to be consistent. This change leaves the obj/arg_obj discrepancy alone under the
justification that "arg_obj" is not any more useful externally-facing and "obj"
internally would conflict with an already existing local.

agbindrec: take a const char* instead of char* for record name

This function does not modify its argument. No changelog entry because this is
already covered by the existing #634 one for the upcoming release.

aggetrec: take a const char* instead of char* for record name

This function does not modify its argument. No changelog entry because this is
already covered by the existing #634 one for the upcoming release.

aggetrec: remove micro-optimization of string comparison

This kind of trick is unnecessary on modern machines.

Merge branch 'dont-wait-for-portable-source-for-cmake-jobs' into 'main'

Reduce latency in CI for CMake jobs + some related cleanup

See merge request graphviz/graphviz!2090

Merge branch 'smattr/16235491-04e0-471e-a116-e92e994517ea' into 'main'

more steps towards removing sfio

See merge request graphviz/graphviz!2089

remove the dependency on the portable-source job for autotools test jobs

This dependency is now obsolete and unnecessary although there is no
latency benefit by removing it.

add archiving of VERSION and COLLECTION to all build jobs

An upcoming commit will remove the dependency to the portable-source
job for all test jobs, so it must be possible the retrieve these files
from the corresponding build job instead. Note that this is used only
for jobs using the portable source which have these files built into
them. Other jobs generate these files as needed.

remove the dependency on the portable-source job for CMake test jobs

The dependency is unnecessary since both the VERSION and COLLECTION
files are now generated in the CMake test jobs. Note that there
probably is no latency benefit by removing it, since the
portable-source job normally is faster than the CMake builds jobs.

add generation of VERSION to all CMake test jobs

An upcoming commit will remove the dependency to the portable-source
job for all CMake test jobs, so this file instead needs to be
generated by these jobs. Note that the COLLECTION file is generated by
the CMake test jobs already before this commit.

remove the dependency on the portable-source job for CMake build jobs

The dependency is now obsolete and this change reduces the latency for
the CMake jobs significantly by not having to wait for the
portable-source job.

don't read the VERSION file for CMake builds in ci/build.sh

The CMake build system generates this file itself and nothing in this
script uses it for the CMake builds.

An upcoming commit will change the CI configuration to start the CMake
builds jobs directly without waiting for the portable-source job and
since that job produces the VERSION file, it will no longer be present
when the CMake builds jobs start.

remove unnecessary casts of parameters to sfio extf functions

These pointers implicitly coerce. Related to #1998.

remove the first parameter of Sffmtext_f

Neither uses of the only field of this type, extf, were using this parameter.
Related to #1998.

restructure "%.*s" handling to avoid sfio dynamic allocation in expr’s printf

Rather than using the temporary sfio buffer in this code, fmt->tmp, we can just
use the original string as-is as it does not need to be modified. Apart from
slightly accelerating this code, this makes yet more progress towards removing
fmt->tmp and sfio. Related to #1998.

remove the use of an sfio temporary buffer in expr’s qualify

We can simply use regular malloc for this short-lived allocation. Related to
#1998.

remove fallback "%?%K" strftime format in expr’s printf

lib/expr’s printf implementation supports a "%t" format code to print a time. If
no format is provided for this, it would fallback to a default of "%?%K".

While working on #1998, one of my intermediate refactorings moved this string
literal into the actual call to strftime. At this point the compiler politely
interjected that neither "%?" nor "%K" are format strings understood by
strftime. So it seems this results in an invalid call to strftime.

To deal with this, we now throw an error when this function is called with no
format string instead of making an invalid strftime call.

Merge branch 'add-svg-analyzer' into 'main'

Add an SVG analyzer and use it in the C++ tests

See merge request graphviz/graphviz!2087

add a few first tests of the rendered SVG