represent route array lengths as size_t

This fixes one -Wsign-conversion warning, though introduces a -Wconversion
warning. We should be able to remove the latter as we continue cleaning up the
use of int for sizes.

remove some commented out code

squash a -Wsign-conversion warning

squash a -Wmissing-prototypes warning

squash some -Wunused-parameter warnings

squash two -Wswitch-default warnings

squash a -Wswitch-default warning

squash a -Wswitch-default warning

squash some -Wunused-parameter warnings

use a stronger type to squash a -Wconversion warning

remove unused parameter route_list from assignTracks

remove unused parameter nrtes from assignTracks

Merge branch 'smattr/CC513531-8BFC-4587-8816-4E41C4672694' into 'main'

some clean up of sfextern.c

See merge request graphviz/graphviz!1888

swap some use of hand rolled NIL for the standard NULL

remove unnecessary global initialization

This structure is default-0-initialized.

Merge branch 'smattr/5D993CF4-B158-4D08-AEFE-FC86452B0FF9' into 'main'

Move back to 2.47 development series

See merge request graphviz/graphviz!1904

Move back to 2.47 development series

Merge branch 'smattr/5689080F-D576-43E6-99D5-B96A4E4D6D98' into 'main'

Stable release 2.47.1

See merge request graphviz/graphviz!1897

Stable release 2.47.1

Merge branch 'fopen' into 'main'

Output more useful error message on fopen failure

See merge request graphviz/graphviz!1901

Output more useful error message on fopen failure

This should help with bugs like
https://gitlab.com/graphviz/graphviz.gitlab.io/-/issues/86

Merge branch 'smattr/556EAFC0-B0EF-4141-AB81-47E2C24419F9' into 'main'

squash C++ warnings

See merge request graphviz/graphviz!1881

remove an unused variable

Squashes a -Wunused-but-set-variable warning.

squash -Wunused-parameter warnings

squash -Wunused-parameter warnings

LASI plugin: remove use of endl

This is generally considered an anti-pattern in C++ these days.

LASI plugin: remove unnecessary use of a pointer

LASI plugin: use job context instead of globals

The LASI plugin was using two global variables, making it impossible to process
multiple graphs with it at the same time. This change refactors it to use the
jobs context as other rendering plugins do.

delete commented out code

remove useless code

This was added in the initial commit of VPSC,
03c20a55c921ac07599be2ae19ae34209ec0d02e. The comment was incorrect. This does
*not* limit iteration because the variable type is unsigned. That is, it is
always >= 0. This was caught by enabling CXXFLAGS+=-Wtype-limits.

squash a -Wparentheses warning

squash a -Wmisleading-indentation warning

Merge branch 'smattr/1F78CB9B-13C5-4885-BAFA-8CB840D952FB' into 'main'

general clean up

See merge request graphviz/graphviz!1886

push a definition closer to its use

remove unused Info variable

remove unused SFIO Version

abbreviate some arithmetic

remove some unnecessary bracketing

remove some unnecessary casts

These pointers coerce implicitly.

remove some commented out code

mark Origin as static const

This variable is not modified or used outside this file.

Merge branch 'smattr/30a4bc3c-4d5a-4130-8a78-60712ab9805e' into 'main'

replace a use of sfstropen with malloc+vsnprintf

See merge request graphviz/graphviz!1884

replace a use of sfstropen with malloc+vsnprintf

This is another step towards removing uses of sfstropen in the code base.
Related to #1873.

add missing #include

Merge branch 'smattr/19cb473f-0226-41d2-9f69-0fc1b45513ef' into 'main'

remove another use of setjmp/longjmp

See merge request graphviz/graphviz!1883

remove now-unused jmp_buf

Related to #1801.

return NULL from deriveGraph to indicate failure

Related to #1801.

anticipate a NULL return from deriveGraph

Related to #1801.

anticipate failure from layout

Related to #1801.

add a return value to layout

Related to #1801.

anticipate failure from fdpLayout

Related to #1801.

add a return value to fdpLayout

Related to #1801.

delete commented out code

Merge remote-tracking branch 'origin/smattr/1EBF65AE-6292-4A55-B9E5-A01E28911B08' into main

allow macOS CI tasks to run immediately

The macOS tasks were, by default, blocking on the portable source generation.
However these tasks do not use the portable source or any prior stage’s
artifacts. Now that we have moved to shared macOS runners without caching, the
macOS CI tasks are the limiting factor on CI runtime. Cutting this dependency
should meaningfully impact this.

propagate COLLECTION from macOS CI build tasks to macOS CI test tasks

This is more straightforward and reliable than trying to keep multiple places
where we manually set the collection coherent.

fix: propagate VERSION from macOS build tasks to macOS CI test tasks

The macOS test tasks in CI were picking up the VERSION from the portable source
task, despite their build tasks using a different process to derive VERSION.
This change ensures they get the VERSION file their build task created.

Merge branch 'fix-strcpy-param-overlap' into 'main'

Fix AddressSanitizer: strcpy-param-overlap in gvconfig_libdir

Closes #1994

See merge request graphviz/graphviz!1887

Fix AddressSanitizer: strcpy-param-overlap in gvconfig_libdir

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

Merge branch 'smattr/19C32E28-7A45-406A-A700-B74AB173D7FE' into 'main'

remove inlining of flip_rec_box

See merge request graphviz/graphviz!1880

remove inlining of flip_rec_box

With a modern compiler, the inline keyword is more or less ignored and the
compiler makes a (usually better) decision itself.

Merge branch 'smattr/1DFCCF93-D5FA-4443-A701-9C65CC090280' into 'main'

fix computation of minimum graph rank on large graphs

See merge request graphviz/graphviz!1879

fix computation of minimum graph rank on large graphs

This code was using MAXSHORT as the maximum (unrealistically large) rank. That
is, start out with this value and assume all nodes will rank below this so we
can progressively step down. This was likely correct when the code was first
written. However, 97cb5fffdcaa97eadb76a9b82a026d084a6a94d9 altered the type used
for storing ranks from short to int to support larger graphs. That is, nodes in
a graph are now anticipated to have ranks in excess of 32767 (on e.g. x86). This
change did not update these algorithms to compensate.

The effect of this is that any graph whose nodes were all ranked >32767 would be
incorrectly judged to have a minimum rank of 32767. The present change fixes
this by realigning this to the limit of the new ranking type.

Note that this removes the last use MAXSHORT in the code base. Unfortunately we
cannot easily remove its definition without breaking API because the containing
header lib/common/arith.h is shipped.

Merge branch 'smattr/BC4B3197-1AC8-4F9E-BCB8-CDF5BFE64A0D' into 'main'

use smart pointers to clean up Visio::Graphics

See merge request graphviz/graphviz!1878

optimize some size()>0 checks

The implementation of empty() is typically more efficient.

use C++11 in-class member initialization to remove the need for an explicit ctor

This is an equivalent way of saying the same thing.

remove some explicit constructor calls

These constructors are all called implicitly already.

use range-based for loops to clean up some manual iteration

Note that in once instance this modifies a loop that was skipping the first
element to not skip it. But this is a no-op the compiler should see through.

store Graphics as managed pointers in Visio::Render instead of raw pointers

This lets us clean up some manual handling of these and stop worrying about
memory management. Note that Visio::Render::ClearGraphicsAndTexts touched in
this commit is effectively a hand written destructor for this class. Presumably
it was implemented in a time before destructors were standardized. We can likely
move the other collections affected here to using smart pointers too and remove
this function entirely eventually.

take a reference instead of a pointer in Visio::Render::PrintEdgeShape

Similar to the previous commits, this function assumes its input pointer is
non-null, so we may as well use the more convenient C++ syntax for this. This
will also ease some upcoming changes to convert the underlying raw pointers to
smart pointers.

take a reference instead of a pointer in Visio::Render::PrintInnerShape

Similar to the previous commit, this function assumes its input pointer is
non-null, so we may as well use the more convenient C++ syntax for this. This
will also ease some upcoming changes to convert the underlying raw pointers to
smart pointers.

take a reference instead of a pointer in Visio::Render::PrintOuterShape

This function assumes its input pointer is non-null, so we may as well use the
more convenient C++ syntax for this. This will also ease some upcoming changes
to convert the underlying raw pointers to smart pointers.

remove const qualifier on Visio::Graphics

Using a const qualifier on the element type of an STL container is usually an
anti-pattern. In this case, doubly so, because Visio::Render does not treat
elements of such a vector as const.

remove const qualifier from Visio::Render::AddGraphic argument

This pointer is not treated as const internally by Visio::Render. This is a step
towards removing manual memory management of Graphic pointers.

Merge branch 'smattr/72E0571B-88AC-4CBC-AC52-ABB391482FDB' into 'main'

fix: do not recognize "&;" as an XML escape sequence

Closes #797

See merge request graphviz/graphviz!1876

fix: do not recognize "&;" as an XML escape sequence

xml_isentity was incorrectly recognizing "&;" as an escape sequence. Despite
vague wording in the standard, it seems fairly clear that &<name>; is only
intended to be a valid escape sequence when <name> is non-empty. Fixes #797.

Note that unfortunately due to #1868, we need to fix this bug four times in
copy-pasted versions of the same function.

add a test case for #797

Merge branch 'smattr/8D352476-8903-4236-8AE7-1C1DA75CC4B8' into 'main'

remove internal definitions of NULL

See merge request graphviz/graphviz!1875

remove internal definitions of NULL

ISO C guarantees this to exist via stddef.h or stdlib.h.

include stddef.h in some files that are using NULL

These were relying on picking up NULL through one of their other includes. We
are about to remove some internal definitions of NULL that may affect this, so
adding explicit stddef.h includes guards against these upcoming changes.

remove stray line feed characters

remove unhelpful comment

remove some commented out code

Merge branch 'smattr/20D31C39-98CB-43AC-89A4-D37C86FA81E3' into 'main'

clean up lib/twopigen/circle.c

See merge request graphviz/graphviz!1874

swap an unnecessary MAX call for fmax

move some definitions closer to their usage

more precise return type for isLeaf

squash -Wcompare, -Wconversion, -Wsign-convertion warnings

squash some -Wsign-compare warnings

squash a -Wsign-compare and a -Wconversion compiler warnings

remove some commented out code

squash -Wconversion compiler warning

This also squashes a -Wsign-compare and a -Wsign-conversion warning.

Merge branch 'smattr/D2AA2D59-2982-42CC-9CA3-374F58FDC5B2' into 'main'

fix: put HTML bit at the top of the count in reference counted strings

Closes #1984

See merge request graphviz/graphviz!1873

simplify some debugging code

remove unused struct name

use a bitfield instead of shifting and masking for reference count in refstr.c

This performs the same operations, but asks the compiler to generate the
required shifts and masks instead of relying on the error prone process of doing
them manually.

fix: put HTML bit at the top of the count in reference counted strings

lib/cgraph/refstr.c implements reference-counted strings with the structure
refstr_t. Since the initial commit of cgraph,
d7767d4b5c511afa89b46798eea4e2f915930844, HTML_BIT has been (IMHO incorrectly)
set based on unsigned int instead of unsigned long. Presumably this was not an
issue because most machines were 32-bit x86 at the time, where
sizeof(unsigned int) == sizeof(unsigned long).

In the transition to x86-64 machines, this setup became awkward. The reference
count is 8 bytes wide, but HTML_BIT is set to be bit 31. I.e. the bit indicating
that a string is HTML is now in the middle of the reference count. A string with
≥ 2³¹ references, something that is possible on an x86-64 machine, now
accidentally sets the HTML bit. Note that this ecosystem transition also made
the comment about HTML_BIT incorrect, /* msbit of unsigned long */.

d1244c8001e8c681def4c0ff25a91136845c2a75 did a tree-wide replacement of unsigned
long with uint64_t, which further compounded the situation. Now the HTML bit is
in the incorrect position on both 32-bit and 64-bit x86.

The present change fixes this by moving the HTML bit into bit 63 of the refcnt
field. Fixes #1984.

It is questionable whether this even needs to be a bitfield, and not simply a
uint64_t and a bool, but I think we should leave this space optimization in
place for now.

Merge branch 'smattr/f4713040-3ec5-4d22-9ea3-48bdc47c1eae' into 'main'

remove a use of setjmp/longjmp

See merge request graphviz/graphviz!1871

remove -Wshadow warning in conc.c

remove now-unused jmp_buf

Related to #1801.