Merge branch 'fix-include-warning' into 'main'

plugin/gdiplus: Fix warning with header file name

See merge request graphviz/graphviz!2079

plugin/gdiplus: Fix warning with header file name

This fixes the following warning with clang in mingw
warning: non-portable path to file '<gdiplus.h>'; specified path differs in case from file name on disk [-Wnonportable-include-path]

Merge branch 'smattr/B40D0BFD-1F7E-4DF2-8043-2612346DBEE9' into 'main'

gcalloc: fix: do not abort on calloc(0, 0) returning NULL

See merge request graphviz/graphviz!2078

gcalloc: fix: do not abort on calloc(0, x) or calloc(x, 0) returning NULL

Commit 43a37bedd934bde50a6441766b85dfc9648abda9 added a `calloc` wrapper but
mistakenly did not account for it being valid for `calloc` to return `NULL` when
the input arguments are 0. This is unlikely to have caused a problem with Glibc
whose allocator typically returns non-null pointers even for 0 allocations.
However assuming a 0 allocation may return a null pointer is more portable.

Merge branch 'smattr/D1A6A66B-15B4-449F-BB71-088F95E12676' into 'main'

fix an #if _WIN32

See merge request graphviz/graphviz!2073

fix an #if _WIN32

This worked out correctly, as _WIN32 is defined to 1 on Windows and is undefined
elsewhere. But it triggered -Wundef compiler warnings on non-Windows platforms.

Merge branch 'smattr/2D62A146-06BC-4160-B15F-B8F495E8589C' into 'main'

squash some Pylint warnings

See merge request graphviz/graphviz!2080

disable a wrong-import-position Pylint warning around run_c

This import has to come out of order because it only works due to prior sys.path
tweaks.

disable Pylint’s too-many-lines warning

By the nature of how Graphviz makes use of Python, some test sources are long
files of independent functions. It does not make sense to split these up and
having long files is not an impediment to maintaining these test cases.

address some missing docstring Pylint warnings

Merge branch 'smattr/90bb42c1-3025-449a-8300-7ec7776f7c33' into 'main'

make agsetfile argument a const char*

See merge request graphviz/graphviz!2072

CFrmSettings::createLayout: remove now-unnecessary cast of a agsetfile parameter

Related to #634.

agsetfile: take a const char pointer instead of a mutable one

This function does not modify the contents of the argument passed to it. Related
to #634.

cgraph: use a const char pointer for storing input filename

Related to #634.

chkNum: use a const char pointer for storing input file name

Related to #634.

Merge branch 'smattr/51C10EB6-96A3-4442-9E10-9CE23E07AD2F' into 'main'

minor no-op clean up

See merge request graphviz/graphviz!2076

remove conditional inlining of mydist

Modern compilers mostly ignore `inline` and make their own decisions about
whether to inline a function or not.

remove unused Windows gettxt redirect

Merge branch 'smattr/2A41855C-DA22-4790-B27C-C1FF9B6108BC' into 'main'

install Python3 to a known path in Windows CI

Closes #2085

See merge request graphviz/graphviz!2067

install Python3 to a known path in Windows CI

This avoids a latent problem wherein Python is installed to a directory
containing its version number and the follow on `Path` manipulation was hard
coded to Python 3.9. This would have broken as soon as Python 3.10 became
available through Chocolatey. Closes #2085. Note, I still do not understand why
the documented technique Chocolatey recommends of `refreshenv` does not work.

Merge branch 'smattr/b672cf7c-7615-4629-b14d-4760b3a76c03' into 'main'

remove vmresize

See merge request graphviz/graphviz!2060

remove return type from assignable

This function only ever returns 1 and no calls check its return value.

remove unnecessary casts of disc->user

This pointer implicitly coerces.

reflow some text in lib/gvpr

xyOf: use size_t for string length

Squashes a -Wsign-conversion warning.

bbOf: use size_t for string length

Squashes a -Wsign-conversion warning.

xyOf: use a C99 bool argument to more strongly indicate a type

bbOf: use a C99 bool argument to more strongly indicate a type

remove some unnecessary forward declarations

remove now-unused vmresize

remove now-unused vmnewof

scformat: use vmalloc instead of vmresize when processing '['

Though it is hard to see, the previous code bottomed out in a call to vmresize.
After allocating, it zeroes the allocated region. So vmresize (itself calling
realloc) signals an unnecessary constraint to the allocator that previous data
in this memory must be preserved.

To simplify this, this change converts the sequence to a vmfree of the prior
memory and then a new vmalloc, more clearly divorcing the new buffer from the
old one.

simplify exalloc

This macro eventually bottoms out on a call to vmresize with NULL. We can
simplify this to directly calling vmalloc instead.

exstralloc: call vmalloc instead of vmresize

When passing an input pointer of NULL, vmresize is equivalent to vmalloc. So we
can shortcut the whole thing by just calling vmalloc directly.

remove p argument to exstralloc

This function is only ever called with NULL for this argument.

Merge branch 'smattr/3f6d98e5-32f1-4219-aaf4-6e595331f922' into 'main'

remove sfsprintf

See merge request graphviz/graphviz!2066

remove now-unused sfio functionality for printing to strings

Related to #1998.

trace: replace sfsprintf calls with snprintf

Related to #1998.

getdyn: replace sfsprintf call with snprintf

Related to #1998.

evaldyn: replace sfsprintf call with snprintf

Related to #1998.

lexname: replace sfsprintf calls with snprintf

Related to #1998.

lexname: fix latent buffer overflow

Buffers of length MAXNAME are printed into, including in a case where the
printed string is "(EXTERNAL:%d)". This needs a maximum of 23 bytes, not 16
bytes as was previously used. This overflow looks impossible to actually trigger
because I believe this code path is only used in the case of a bug in the lexer
itself. Hence no changelog entry for this.

This issue was exposed when moving sfsprintf calls to snsprintf, as the compiler
understands the semantics of the latter and knows how to warn about detectable
overflows. Related to #1998.

exccopen: replace sfsprintf call with snprintf

Related to #1998.

exopname: replace sfsprintf call with snprintf

Related to #1998.

pathfind: replace sfsprintf calls with snprintf

Related to #1998.

Merge branch 'smattr/fbbdaf80-7180-4b89-8e4d-2f12839b234f' into 'main'

[nfc] remove mode argument to openFile

See merge request graphviz/graphviz!2065

spine: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for spine.

doc/libgraph/Makefile: remove unused TEXS variable

It is not clear to me what the purpose of this variable is/was. This Makefile
was added in adf9f9ad70662c0ca291feea1eb51df113c7d281. Even in that revision,
the variable was unused and refers to the non-existent file graph.tex.

doc sccmap: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for the copy of sccmap in doc/.

unflatten: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for unflatten.

sccmap: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for sccmap.

gvpack: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for gvpack.

gvgen: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for gvgen.

gv2gml: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for gv2gml.

graphml2gv: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for graphml2gv.

gml2gv: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for gml2gv.

cvtgxl: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for cvtgxl.

mingle: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for mingle.

gvmap: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for gvmap.

cluster: [nfc] remove mode argument to openFile

This function is only ever called with mode "w". This is the equivalent of
21a6f2e2f7e8a377977c7bdad1356591a25d516f for cluster.

Merge branch 'smattr/73792e61-f73a-4119-9604-12749cd2bf7e' into 'main'

more warning squashing, around strlen usage

See merge request graphviz/graphviz!2074

use size_t when handling the length of some directory lists

Squashes five -Wsign-compare/-Wconversion warnings.

findPath: remove some unnecessary brackets

Fgets: use size_t instead of int for buffer length

Squashes three -Wconversion warnings.

use size_t when dealing with string lengths of XML tokens

Squashes a number of -Wconversion warnings.

exsubstr: remove some unnecessary bracketing

prformat: use a size_t when dealing with string lengths

Squashes three -Wconversion warnings.

strip_dir: replace ad hoc boolean with C99 bool

strip_dir: rewite character loop using a size_t

Squashes a -Wconversion warning.

strip_dir: remove an unnecessary cast

&s[i+1] is already a char*.

track rdr_t offset and extent as size_t instead of int

Provides more flexibility for larger data and better interoperates with strlen,
squashing a -Wconversion warning.

Merge branch 'smattr/0E77B8A2-8B78-48BB-9CFC-4D40C6A734B0' into 'main'

ccomps: some clean up

See merge request graphviz/graphviz!2075

ccomps: use size_t when dealing with buffer sizes in chkGraphName

Squashes a -Wconversion and a -Wsign-conversion warning.

ccomps: use a size_t when dealing with buffer sizes in getBuf

Squashes a -Wsign-conversion and four -Wconversion warnings.

ccomps: mark getopt switch as exhaustive

Squashes a -Wswitch-default warning.

ccomps: use a size_t when dealing with character offsets in split

Squashes a -Wconversion and a -Wsign-conversion warning.

ccomps: abort on all allocation failures

None of the allocations in ccomps could tolerate failure. So this change makes
them all call wrappers that cleanly abort in the event of out-of-memory. Note
that this also fixes an issue where ccomps incorrectly identified itself as `gc`
in one of the previous failure messages.

ccomps: remove an unnecessary code path in getBuf

When len is 0, buf is NULL and thus calling realloc is equivalent to calling
malloc.

Merge branch 'smattr/728D79B1-E42D-4105-811D-08FA2A361B6F' into 'main'

remove Windows overrides of `inline`

See merge request graphviz/graphviz!2063

remove Windows overrides of `inline`

The `undef` part of this macro juggling was typoed as `WIN32_STATIC` in commit
b26b5fc076c1b5d9919ce79c807f5b3921149597, so it never properly undid the
preceding `inline` redirection. However, this redirection is unnecessary anyway.
Contrary to the Microsoft docs,¹ the `inline` keyword seems understood in *both*
C and C++.

¹ https://docs.microsoft.com/en-us/cpp/cpp/inline-functions-cpp?view=msvc-160

Merge branch 'smattr/007BBBEC-441C-49AA-9E7F-31A25ACE5257' into 'main'

replace mkdefs-generated gdefs.h with X macros

Closes #2101

See merge request graphviz/graphviz!2062

replace mkdefs-generated gdefs.h with X macros

Prior to this commit, the gdefs.h header was generated by a C program, mkdefs.c.
There were a number of issues with this approach:

  1. The CMake build system was assuming the compiler to build mkdefs.c and the
     compiler to build Graphviz itself were the same. This is not necessarily
     true when cross-compiling.

  2. Generation under MSBuild seems to have been impractical, so the generated
     header was checked in to the repository under windows/include/gvpr/gdefs.h,
     somewhat defeating the purpose of making it generated.

  3. The CMake build system seems to not have been setup to correctly compile
     mkdefs.c under all circumstances (see #2101).

This change removes any reliance on a host C compiler and instead uses a series
of X macros¹ to achieve the same effect. The values of all generated constants
and the content of generated structures is intended to be unchanged, though some
#defines have been altered to enums. In these cases, there was no advantage to
using a macro and multiple advantages to not using a macro.

This change is affecting a shipped header (gdefs.h) and also removes it from the
list of shipped headers. Installing it appears to have been a mistake as there
is no easy way for end users to use it. The header, fully expanded, still relies
on further expansion of macros that are only defined in expr.h, a header that is
not shipped.

Closes #2101.

¹ https://en.wikipedia.org/wiki/X_Macro

Merge branch 'smattr/F218DB16-9060-4D1D-88E2-DDDBF8A5034D' into 'main'

add Fedora 34 testing to CI

Closes #2059

See merge request graphviz/graphviz!2070

add Fedora 34 testing to CI

Closes #2059.

Merge branch 'smattr/16FA1351-3EE8-4DAB-84C6-E98818C000D1' into 'main'

lib/gvc: [nfc] fix some comment typos

See merge request graphviz/graphviz!2069

lib/gvc: [nfc] fix some comment typos

Merge branch 'smattr/5BC8FF91-4894-488A-8212-6A37F2A35ECC' into 'main'

remove RS HPPA header work arounds

See merge request graphviz/graphviz!2068

rename LS to LSH for consistency with RSH

remove no longer necessary RS undefine

lib/expr: use RSH as a constant for >> instead of RS

A couple of places in the code base are dodging an `RS` symbol apparently
introduced by an HPPA header. It is simpler to just use a symbol that does not
collide at all.

Merge branch 'cmansley:cmansley-main-patch-01536' into main

See merge request graphviz/graphviz!2064

Initialize nPasses in aspect_t struct in SetAspect

In dotLayout, nPasses is decremented in a while loop. Since it may be
uninitialized, this triggers clang's UndefinedBehaviorSanitizer:
signed-integer-overflow. While behavior does not change because the loop also
checks nextIter, which is initialized, this change fixes the sanitizer error.

Merge branch 'smattr/5c73db2b-d34f-4673-9e8a-b67c3070c221' into 'main'

rewrite edgepaint command line parsing using getopt_long

Closes #1971

See merge request graphviz/graphviz!2059

edgepaint: more consistently use EXIT_SUCCESS and EXIT_FAILURE when exiting

These are more portable than relying on numeric values.

edgepaint: remove unused infile

Surprisingly this is configured but never used. Instead all code accesses the
input file(s) through `Files`.

edgepaint: remove mode argument to openFile

In edgepaint, this function is only called to open a file in write mode.

rewrite edgepaint command line parsing using getopt_long

This change makes edgepaint command line parsing code more standard and robust.
It now rejects invalid arguments and takes more standard GNU-style double dash
prefixed options. Note that the old style single dash prefixed options are also
still accepted for compatibility purposes. Closes #1971.

qualify openFile’s arguments as const

The various versions of this function all do more or less the same thing, none
of which involves modifying input parameters. Related to #634, #1971.

qualify knownColorScheme’s argument as const

This function does not modify its parameter. Related to #634, #1971.

qualify color_palettes_Q’s argument as const

This function does not modify its parameter. Related to #634, #1971.