remove unused lib/glcomp files

remove unused lib/common files

remove unused plugin/core files

remove unused smyrna files

remove unused gvpack_builtins.c

remove unused sfexit.c

379b9f58511cbb7f1ef7b6b00865e44070cbad57 removed all references to sfexit.c but
did not remove the file itself.

Merge branch 'smattr/59c0681a-e63f-44ca-b903-04fb073e174a' into 'master'

some clean up of string handling

See merge request graphviz/graphviz!1833

swap sprintf for snprintf where straightforward

sprintf has consistently been in the top ten root causes of vulnerabilities in
low level applications and is generally considered unsafe for any use in modern
code bases. This commit removes the instances of it that are straightforward to
convert to snprintf. Related to #1950.

remove an unnecessary intermediate buffer

remove an unnecessary intermediate buffer

remove some commented out code

move short lived arrays to the stack

It is fine to allocate 100 bytes on the stack instead of the heap in these
situations.

remove some commented out code

remove some commented out code

remove an unnecessary intermediate buffer

remove some commented out code

remove some errant comments

move a heap-allocated smyrna buffer to the stack

It is fine to allocate 256 bytes on the stack here.

remove redundant lefty conditional

rephrase extremely visually confusing lefty comparison

rephrase unnecessarily cryptic string comparisons in lefty

remove some unnecessary intermediate buffers in lefty

convert sprintf calls in xdot to snprintf

Related to #1950.

Merge branch 'smattr/3DB6A16B-DE0E-4004-A579-D109134E7310' into 'master'

remove unused hpglgen.c

See merge request graphviz/graphviz!1830

remove unused hpglgen.c

Merge branch 'smattr/26d4fb27-7a4a-4114-9dff-541dec3edc65' into 'master'

some gvmap clean up

See merge request graphviz/graphviz!1831

remove unused plot_dot_map argument to plot_dot_map

remove unused poly_point_map parameter to get_country_graph

remove unused function plot_processing_polygons

mark plot_dot_polygons static

This is not used outside make_map.c.

mark processing_one_poly static

This is not used outside make_map.c

remove unused close argument from processing_polygon

remove unused close argument to dot_one_poly

mark dot_one_poly static

This is unused outside of make_map.c

remove unused parameter to dot_polygon

remove unused parameter to plot_dot_labels

mark plot_dot_labels static

This is not used outside make_map.c

remove unused parameters from plot_dot_edges

remove commented out code in gvmap

remove plot_*_map prototypes

These functions are not implemented.

squash some -Wmissing-prototypes warnings in gvmap

remove unused label_sizes argument to improve_contiguity

remove intermediate string construction altogether in dot_polygon

There's no need for this and we can achieve it in one (well two) shot.

The better way to do this would be with an in-memory expanding buffer (like
open_memstream), but this change is at least an improvement over the previous
code.

abbreviate computation of len_swidth in gvmap

C99 semantics allow us to avoid using an intermediate buffer for this operation.

remove redundant swidth computation

The values being computed here were already computed in a previous block and
have not changed since. There was clearly copy-pasted wholesale from the code
above it.

Merge branch 'smattr/FD7CAB8E-F5F4-476A-83DE-B2F650DD87C6' into 'master'

fix: remove BROWSER #define from Windows MS Build compilation

Closes #1954

See merge request graphviz/graphviz!1828

fix: remove BROWSER #define from Windows MS Build compilation

This is only used by two plugins, glitz and xlib, and it’s unclear whether
either are intended to run on Windows. The code path using BROWSER is also only
enabled when the build environment has sys/types.h and unistd.h, neither of
which typically exist on Windows. In any event, xdg-open is not the way to open
a browser on Windows. It is better that we exclude this and fail at compile time
than build plugins that fail in unpredictable ways at runtime. Closes #1954.

Merge branch 'smattr/6992578B-3535-4D79-B858-D75963CAD598' into 'master'

Start 2.47 development series

See merge request graphviz/graphviz!1842

Start 2.47 development series

Merge branch 'smattr/2438B2D8-0DA8-4113-BB34-552A50EC9C91' into 'master'

Stable Release 2.47.0

See merge request graphviz/graphviz!1835

Stable Release 2.47.0

Merge branch 'smattr/DE0345C1-564D-44F6-82A4-8369EFE0151A' into 'master'

fix: correctly recognize Windows packages in deploy script

Closes #1955

See merge request graphviz/graphviz!1834

fix: correctly recognize Windows packages in deploy script

239f6301789b603202d500f7ae58e800354c1524 identified Windows packages containing
the word “Windows” but the package filenames actually have this in lower case.
Closes #1955.

Merge branch 'smattr/F122BD95-99D1-4293-A809-6EF587D2D8ED' into 'master'

deploy: generate SHA256 checksums in addition to MD5, and also apply to macOS and Windows packages

Closes #1955

See merge request graphviz/graphviz!1827

generate checksums for macOS and Windows packages during deployment

This is in addition to the previously generated portable source checksum. For
now, it is assumed we don't need checksums for the fined grained Linux packages.
Closes #1955.

generate a SHA256 checksum of the release tarball in addition to MD5

Related to #1955.

extract checksumming in deploy script into a function

Related to #1955.

Merge branch 'swaldhoer/simple-python-fixes'

use main convention
use argparse to work with command line arguments
sort imports alphabetically
fix whitespace inconsistencies
fix some pylint errors
fix formatting

Merge branch 'smattr/8FF1769C-58CB-4DD6-891E-3ACEE75C40A0' into 'master'

some libgvpr cleanup

See merge request graphviz/graphviz!1813

avoid some use of literal 0 for a pointer

We can write this clearer as NULL.

use more explicit types for some booleans

Now that we build with C99, we can be more expressive with types.

reflow some lines

remove some unnecessary bracketing

undo another strcmp micro-optimization

replace another sfstropen with agxbuf

Related to #1873.

replace another sfstropen in GVPR with an agxbuf

Related to #1873.

remove a use of sfstropen in GVPR

Related to #1873.

Merge branch 'smattr/37ce32ea-c880-4551-bbc7-91d0a3ccc596' into 'master'

rewrite agxbputc as a function

Closes #1814

See merge request graphviz/graphviz!1812

rewrite agxbputc as a function

This solves an issue where the macro implementation did not sufficiently bracket
its arguments. Fixes #1814.

Merge branch 'smattr/F8CE1A6E-745B-4768-839C-5004B9EECA6E' into 'master'

remove yet more strcmp micro-optimizations

See merge request graphviz/graphviz!1805

remove commented out M_PI

remove another strcmp micro-optimization

remove no-longer-used Strcmp

swap Strcmp calls for strcmp

Strcmp optimized strcmp by unrolling the first iteration of a byte comparison
loop. This is no longer an optimization. In modern compilers/libc, (1) strcmp is
generally no longer a byte-by-byte comparison loop and (2) the compiler knows
strcmp as a built-in and can do this kind of unrolling itself. As a result, the
compiler actually has to do *more* work to reverse Strcmp back into strcmp and
thus determine it can use, e.g., SIMD optimizations.

This is essentially the next step of f5a326cb061866bb3beaa33c3e5169efe7fc134d.

Merge branch 'smattr/d509f519-e63e-4df1-ab2b-6daac754ff43' into 'master'

remove an intermediate buffer in libxdot’s jsonString

See merge request graphviz/graphviz!1800

remove intermediate buffer in libxdot

The function jsonString was trying to optimize processing by constructing a
string in an intermediate agxbuf before passing this buffer to the caller’s
print callback. An implicit assumption is that the callback is significantly
more expensive than outputting to the intermediate buffer. This is not true. The
only callbacks ever passed to this function are agxbprint itself and fputs,
which does efficient buffering. Both of these are at least as efficient as
agxbuf, making the trip through the intermediate buffer actually a
*de-optimization*.

remove irrelevant comment

JSON encoding needs no special UTF-8 handling.

add a test case for xdot's JSON output

Related to #1958.

refactor existing test cases that compile C to call common code

helper test function for running C code

helper test function for compiling C code

add a test program for proving jsonXDot

Related to #1958.

Merge branch 'master' into 'master'

Revert "Revert "Build docker images with buildkit""

Closes #1963

See merge request graphviz/graphviz!1829

Revert "Revert "Build docker images with buildkit""

This reverts commit 6cdd95b8776d54eb1a474e1d90f7f3b7e85a7e01.

I'm now confident I can make Docker buildkit caching work after
observing it work on graphviz.gitlab.io. repo.

I now understand why I wasn't seeing caching before:
- I was pushing to my own fork of graphviz
- I didn't push to the master branch
- So I didn't build a `live` tag
- So I didn't have anything to cache from

Merge branch 'smattr/7DFF32C2-8BCC-4FDB-8174-28981F4494E5' into 'master'

removes Windows strdup redirection

See merge request graphviz/graphviz!1797

removes Windows strdup redirection

Microsoft already provides a shim that does this [0].

  [0]: https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/strdup-wcsdup?view=msvc-160

Merge branch 'arrows' into 'master'

Move arrows.html generation into jinja2 templates

See merge request graphviz/graphviz!1826

Merge branch 'shapes-no-table' into 'master'

Generate shapes responsively: reflow small screens

See merge request graphviz/graphviz!1825

Move arrows.html generation into jinja2 templates.

In preparation for moving this to Jekyll repo.

Generate shapes responsively: reflow small screens

Don't use a fixed table of width 4, reflow to the available space.

This was a bit of a pain to get the CSS right. This stackoverflow answer
was very helpful: https://stackoverflow.com/a/14155983/171898

This is towards moving shapes.html into graphviz.gitlab.io repo. Simpler
for loops that jekyll can handle.

Generate shapes responsively: reflow small screens

Don't use a fixed table of width 4, reflow to the available space.

This was a bit of a pain to get the CSS right. This stackoverflow answer
was very helpful: https://stackoverflow.com/a/14155983/171898

This is towards moving shapes.html into graphviz.gitlab.io repo. Simpler
for loops that jekyll can handle.

Merge branch 'no-color' into 'master'

Remove colors.html.j2, it's migrated to docs repo

Closes #1967 and #1968

See merge request graphviz/graphviz!1824

Merge branch 'awk-color' into 'master'

Fix single-line color generation

Closes #1967

See merge request graphviz/graphviz!1823

Remove colors.html.j2, it's migrated to docs repo

Moved to graphviz.gitlab.io in
https://gitlab.com/graphviz/graphviz.gitlab.io/-/merge_requests/269

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

Fix single-line color generation

Looks like awk has actually added a `-s` flag in the last ten years, breaking this code! Amazing.

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

Fix single-line color generation

Looks like awk has actually added a `-s` flag in the last ten years, breaking this code! Amazing.

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

Merge branch 'color-template2' into 'master'

Close output file descriptor only after done writing

Closes #1965

See merge request graphviz/graphviz!1822

Close output file descriptor only after done writing

Previously we would close the output as soon as we wrote a single line.

This didn't account for continuation lines, where $1 == "".

The subsequent printfs would be to a closed file.

Fixes #1965

Merge branch 'color-template2' into 'master'

Split generation of colors.html in Makefile

See merge request graphviz/graphviz!1821