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

Split generation of colors.html in Makefile

Allows us to separately generate then check in
the constituent color html files as includes in the graphviz.gitlab.io
repo.

Which would let us remove the colors.html template from this repo.

Merge branch 'lang-2' into 'master'

Remove lang.html.j2: it's migrated repos

See merge request graphviz/graphviz!1820

Merge branch 'smattr/A6912F38-21BB-46AC-AE57-DF0CA1990112' into 'master'

strlen-related clean up and warning squashing

See merge request graphviz/graphviz!1795

remove commented out execOp

remove unnecessary bracketing

remove use of int as size_t in gprintf

remove use of int as size_t in pathAppend

This removes three -Wsign-conversion and -Wconversion warnings.

use stronger typing for pathAppend

This removes one -Wsign-conversion warning.

remove commented out code in make_map.c

remove commented out #include

remove intermediate buffer in dot_polygon

This buffer was only used to calculate how many bytes were necessary to
allocate. The semantics of snprintf allow us to achieve this without the buffer.

remove use of int as size_t in dot_polygon

This removes four -Wconversion warnings.

remove use of %d to print size_t

Now that we build with C99, %zu is available.

remove some unnecessary casts

remove some unnecessary bracketing

remove use of int as size_t in colorxlate

This removes one -Wconversion and two -Wsign-conversion warnings.

remove use of int as size_t in fullColor

This removes two -Wconversion and one -Wsign-conversion warnings.

remove use of int as size_t in canontoken

This removes two -Wconversion and one -Wsign-conversion warnings.

remove strncmp micro-optimization in rindexOf

Compilers can do this trick themselves if they decide there is speed to be
gained from it.

remove use of int as size_t in rindexOf

This removes two -Wconversion and one -Wsign-conversion warnings.

rewrite indexOf

There is a C standard library function that does almost all of this for us. This
removes two -Wconversion and one -Wsign-conversion warnings.

remove commented out strdup

This is part of “Extensions to the C Library Part II: Dynamic Allocation
Functions, ISO/IEC TR 24731-2:2010.” It is also slated for inclusion in ISO C23.
In practice, all modern libc implementations provide it.

Merge branch 'smattr/487954c2-c50c-4d99-a3d0-a01379415823' into 'master'

port rtest2 to pytest and re-enable it

See merge request graphviz/graphviz!1794

remove now-legacy rtest2 directory

consolidate ported rtest2 into tests/

port rtest2 to pytest

Only two of the reference tests pass, but this is at least the first step
towards resurrecting these.

decompress test_reference

Git data is already compressed. We do not gain anything by storing compressed
files within the repository.

Merge branch 'color-template' into 'master'

Use python template to generate colors.html

See merge request graphviz/graphviz!1819

Merge branch 'w3data' into 'master'

Remove w3data.js from this repo

See merge request graphviz/graphviz!1818

Merge branch 'lang' into 'master'

Generate lang.html with a template

See merge request graphviz/graphviz!1817

Merge branch 'no_json' into 'master'

Remove graphviz_json_schema.json copy

See merge request graphviz/graphviz!1816

Merge branch 'no-a_' into 'master'

Remove doc/info/a_{forward,nohead}.dot

See merge request graphviz/graphviz!1815

Remove lang.html.j2: it's migrated repos

In https://gitlab.com/graphviz/graphviz.gitlab.io/-/merge_requests/268,
I added a copy of lang.html.j2 to graphviz.gitlab.io.

Now there's no need to keep two copies.

Instead we only generate the insides of lang.html: an EBNF HTML diagram,
grammar.html

Use python template to generate colors.html

One step closer to migrating this to the other repo as a jekyll file.

Confirmed with diff that the only differences are extra blank lines,
nothing important.

Merge branch 'no-html' into 'master'

Inline html.{1,2,3} files into shapes.html template

See merge request graphviz/graphviz!1814

Remove w3data.js from this repo

It's been migrated fully to graphviz.gitlab.io docs repo.

No more references in this repo.

Generate lang.html with a template

This is a bit more complex, but it does let us see the entire doc in one
template.

One step closer to being able to generate this in jekyll and have a
single source of truth for this doc.

Remove graphviz_json_schema.json copy

This file is used in doc/infosrc/ to generate an HTML table. It's not
used here in this directory, it's just a copy.

This is unused here, not part of the html_DATA Makefile in this
directory.

Remove doc/info/a_{forward,nohead}.dot

These seem completely unreferenced.

Inline html.{1,2,3} files into shapes.html template

shapes.html is the only place these files are used.

Moving them into a template is one step towards being able to build this
file in the other graphviz repo (and having a single source of truth for
this doc)

Merge branch 'no-buildkit' into 'master'

Revert "Build docker images with buildkit"

Closes #1963

See merge request graphviz/graphviz!1811

Merge branch 'command' into 'master'

Delete command.html: moved to graphviz.gitlab.io

See merge request graphviz/graphviz!1810

Merge branch 'mv-tcl' into 'master'

Move doc/info/*.tcl build scripts to doc/infosrc

See merge request graphviz/graphviz!1807

Revert "Build docker images with buildkit"

This reverts commit e8c5df0ad943cf2ab1c657165900928c3f6f7d3b.

Fixes #1963

Merge branch 'formats2' into 'master'

Delete mkoutput.py

See merge request graphviz/graphviz!1809

Merge branch 'formats' into 'master'

Adapt mkoutput.py to generate YAML for jekyll

See merge request graphviz/graphviz!1808

Delete command.html: moved to graphviz.gitlab.io

Delete mkoutput.py

Output generation has moved to graphviz.gitlab.io repo.

Towards #1962.

Adapt mkoutput.py to generate YAML for jekyll

This is towards #1962. The output yaml-front-matter HTML can be used as input to jekyll.

Merge branch 'buildkit' into 'master'

Build docker images with buildkit

See merge request graphviz/graphviz!1804

Move .tcl build scripts to infosrc

This is where all the other build scripts live.

And it's where the input to these files lives.

Merge branch 'smattr/6efacd71-1ee2-422c-a843-1765f184956a' into 'master'

fix buffer overflow in unflatten

See merge request graphviz/graphviz!1791

Remove doc/info/*.gv

These files are not referenced by any HTML.

Remove doc/info/*.png

There are no references to any of these files left.

They're all hosted now over at the graphviz.gitlab.io main docs repo.

No need for this duplication.

Build docker images with buildkit

Buildkit is the newer, faster way to build docker images.

For one, buildkit doesn't send the entire repo as build context.

Remove gifs from doc/info

These are now hosted over at graphviz.gitlab.io repo -- the main docs repo.

more strongly type some boolean variables

remove an open coded NULL

remove unnecessary bracketing in unflatten

swap an sprintf for an snprintf

Towards #1950.

fix buffer overflow in unflatten

GCC 8.3.0 with -Wformat-overflow is capable of noticing that the sprintf in this
function can overflow the target buffer.

Merge branch 'smattr/BF97B15D-EDF2-4F74-B79D-E3C0E7E2124F' into 'master'

fix GVPR's ability to comprehend absolute paths on Windows

Closes #1780

See merge request graphviz/graphviz!1790

Merge branch 'redir' into 'master'

Redirect doc files to graphviz.org

See merge request graphviz/graphviz!1802

fix: detect absolute paths correctly in GVPR on Windows

This code was incorrectly depending on WIN32_DLL which was not defined when the
source file was built. Rather than addressing this directly, we make gvpr comply
more accurately with its man page:

  If progfile contains a slash character, the name is taken as the pathname of
  the file.

We now look for a path separator, backslash on Windows and forward slash on
anything else. Fixes #1780.

test case for #1780

Redirect more duplicate docs to graphviz site

I've deleted schema.html instead of redirecting because it's not
intended to be shown as a top-level page on its own, but rather included
in another page as a subtree.

Towards https://gitlab.com/graphviz/graphviz.gitlab.io/-/issues/39