gvmap: remove dynamic allocation for line color

Following the prior type signature changes, there is no need to dynamically
allocate this.

plot_dot_map: take a const pointer for a string that is not modified

plot_dot_polygons: take a const pointer for a string that is not modified

dot_one_poly: take a const pointer for a string that is not modified

dot_polygon: take a const pointer for a string that is not modified

polytess.c: remove unused return values

Render_Contour2: allocate vertices as a single block

Rather than an array of pointers and then individual allocations for each
vertex, we can simply allocate a single block ahead of time. This decreases
memory requirements and increases cache locality. Note that this does nothing
about the fairly egregious memory leak in this code.

Render_Contour2: fix incorrect allocation calculation

It is clear from surrounding code that this was attempting to allocate an array
of GLdouble *pointers*, not an array of GLdoubles. It is unlikely this would
have caused any noticeable issues as sizeof(GLdouble) is typically greater than
sizeof(GLdouble*), but nevertheless the calculation was incorrect. This change
not only corrects this, but uses the more appropriate calloc instead of malloc
for array allocation.

attr_value_edited_cb: remove unused 'row' variable

Merge branch 'smattr/d9bdea2a-6c61-49fb-988e-2b4f61ee06dd' into 'main'

fix path to which CMake install man pages

Closes #1936

See merge request graphviz/graphviz!2102

fix path to which CMake install man pages

The CMake build system was setup to unconditionally install man pages to man3.
This is only correct for libraries, whose man pages have the extension .3.
Binaries have man pages with the extension .1 and were intended to be installed
in man1. Now both go to the correct man subdirectory. Fixes #1936.

Merge branch 'smattr/F6BCF86C-44CC-4894-94B5-E3DE88B8EB93' into 'main'

merge disto_notes.txt into DEVELOPERS.md

See merge request graphviz/graphviz!2104

merge disto_notes.txt into DEVELOPERS.md

Most of this was out of date, but the SUSE entry may still be relevant.

Merge branch 'smattr/DF8696C5-E2D8-45B4-9AC1-96857719F6C3' into 'main'

remove unused chars.tcl script

See merge request graphviz/graphviz!2103

remove unused chars.tcl script

The FontWidth table this script generates appears to have never been used.

Merge branch 'smattr/A84C7010-2B56-4AE4-87A8-1C503E97EB27' into 'main'

remove open coded NULL in agread calls

See merge request graphviz/graphviz!2099

remove open coded NULL in agread calls

Merge branch 'smattr/d6bb3f1d-b287-4d7f-9362-88e271f35b45' into 'main'

more sfio removal, some Smyrna clean up

See merge request graphviz/graphviz!2098

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.