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

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

chmod -x todo.html

No need for this to be executable. It doesn't have a #! line.

Redirect doc files to graphviz.org

The canonical location for these docs is in the graphviz.gitlab.io repo,
and it's extra work to maintain them in both places. The files have
gotten a bit out of sync, and have been migrated to markdown in the
other repo.

Adding the redirects ensures that they point at the latest versions with
the latest changes.

Merge branch 'smattr/18D8E0C0-7D8C-46F2-93F8-78506439944D' into 'master'

remove strcmp micro-optimizations

See merge request graphviz/graphviz!1788

remove strcmp micro-optimizations

Modern compilers can do these tricks themselves, so we are better off writing
more straightforward readable code.

Merge branch 'smattr/F47441CD-2230-40EC-BADB-64374F9272ED' into 'master'

don't clone the history of submodules in Windows CI tasks

See merge request graphviz/graphviz!1787

don't clone the history of submodules in Windows CI tasks

The Windows builds only need the tip of these repositories, so no need to spend
time and bandwidth retrieving their history.

Merge branch 'smattr/BD4AB1A8-8439-48CA-8E01-7E1A407AAB82' into 'master'

remove sed as a Windows build dependency

See merge request graphviz/graphviz!1786

remove use of sed in CMake files

We now no longer need sed in either CMake or Visual Studio, removing it as a
build dependency on Windows.

remove use of sed in Visual Studio files

It should now be possible to build Graphviz using Visual Studio on Windows
without sed installed.

teach version script how to output CPP definitions

remove use of sed hacks in autogen.sh to get version components

add flags to version script to print individual version components

Currently unused.

move patch version adjustment earlier in version script

This has no relevant effect right now, but will be useful for upcoming changes
that need to see the updated patch version when branching on what to output.

Merge branch 'smattr/2EA29827-1F04-461D-8980-B91F4AD999F4' into 'master'

more precise dependencies for CI build tasks

Closes #1959

See merge request graphviz/graphviz!1799

more precise dependencies for CI build tasks

This is an optimization that allows, e.g., the ubuntu18-04-build task to start
as soon as its Docker image is available and the portable source tarball has
been built. Prior to this change, this task would also block on all the other
docker_build_* tasks. Closes #1959.

Merge branch 'smattr/5a886023-3f7d-4966-81ba-dae26ba1ae43' into 'master'

remove ksh as a build/test dependency

See merge request graphviz/graphviz!1764

use the user's default shell for gvmap.sh

This removes a dependency on ksh. It is still possible to run this script under
ksh, but it is also now possible to run it under e.g. Bash.

port gvmap.sh to POSIX-compatible syntax

This retains the ability to run this script with ksh, while also paving the way
for moving to a POSIX shell.

remove unused clean up trap handler in gvmap

Nothing creates the files that this handler tries to delete, so it is not
necessary.

Merge branch 'add-pylint' into 'master'

add pylint job

See merge request graphviz/graphviz!1784

Add Python dependencies as requirements.txt
Add linting of Python scripts

Merge branch 'smattr/0f3b0ef5-6a75-43f6-9d83-bb0784b8282f' into 'master'

unify some Windows and non-Windows code

See merge request graphviz/graphviz!1763

remove manual memory management of rectangle array

This unifies some Windows and non-Windows code, removes some unwise VLAs, and
removes the need to manually deallocate this array, making the code less error
prone. We could go further with removing manual allocations, but for now this is
a minimal net improvement.

unify some array allocation across Windows and non-Windows

Just because GCC/Clang support variable length arrays, does not mean they are a
good idea. Using heap allocated memory for this is safer. This way we also get
to maintain a single code path. This adds some new compiler warnings, but
they're in line with the issues we already know (namely, passing int values as
allocation sizes).

unify some fopen code across Windows and non-Windows

The mode flag 'b' is a no-op on POSIX, so we can use the same code everywhere.

remove bool shims in diffimg

Now that we build with C99, the existence of this support is guaranteed to be
supplied in stdbool.h.

remove commented out _WIN32 block

Merge branch 'ci5' into 'master'

Change Docker context to be the repo root

See merge request graphviz/graphviz!1796

Change Docker context to be the repo root

See https://stackoverflow.com/a/24540011/171898 and
https://docs.docker.com/engine/reference/builder/#usage

The directory passed to docker is the context. Previously we changed
directory into `ci/$IMAGE` then passed `.` as the "context".

Passing a subdir as context prevents any inclusion of files outside the
context, which isn't what we want: we want to include `requirements.txt`
from the root.

So instead, pass the entire repo as "context", and use the "-f" flag to
set the Dockerfile used.

Should help with #1743 and !1784

Merge branch 'smattr/1840627B-C284-4D16-B0FC-2DEB1CA88F30' into 'master'

remove remaining Vim modeline comments

See merge request graphviz/graphviz!1762

remove remaining Vim modeline comments

These have no effect in non-Vim editors or for Vim users that have modeline
disabled. A more modern way to achieve this is clang-format. This commit was
achieved with

  $ find . -type f \( -iname "*.cs" -or -iname "*.y" -or -iname "*.l" -or \
    -iname "*.m" -or -iname "*.c" -or -iname Makeile.am -or -iname "*.cpp" -or \
    -iname "*.h" \) -exec sed -i '' -e ':a' -e 'N' -e '$!ba' -e \
    's/\/\* vim:set [^\n]*\n\n//g' "{}" \;

and then manually collecting some further ones that were missing by this filter.

Merge branch 'jarrodmillman:changelog-typo'

Fix release typo

Merge branch 'ci5' into 'master'

Revert "Revert "Only run macOS CI on graphviz/graphviz repo""

See merge request graphviz/graphviz!1792

Revert "Revert "Only run macOS CI on graphviz/graphviz repo""

This reverts commit 77c3b048f198798791193a028a77bf903a5b325e.

And replaces the variable with something that doesn't sound like a
GitLab predefined environment variable, to avoid surprise.

Towards #1951

https://docs.gitlab.com/ee/ci/variables/predefined_variables.html

Merge branch 'smattr/37F06C45-6BC7-4617-AA19-99B187958B1C' into 'master'

remove a useless conditional #include block

See merge request graphviz/graphviz!1759

remove a useless conditional #include block

Merge branch 'smattr/c92aba1c-f53c-429e-acfd-8c323bdc0654' into 'master'

some clean up of JSON back end

See merge request graphviz/graphviz!1756

squash some -Wunused-parameter warnings

abbreviate some code for comma-separated JSON

remove some unnecessary bracketing

remove commented out alternate write_subgs implementation

Merge branch 'smattr/65a6d0ab-fc93-4d90-8a02-286d0922e43f' into 'master'

remove regex usage and associated RxSpencer dependency

Closes #1919

See merge request graphviz/graphviz!1757

remove RxSpencer dependency

Closes #1919.

remove use of regex in smyrna

This "regex" was only ever used to match literals like "R" and the empty string.
Related to #1919.

remove unused smyrna file

remove use of regex for parsing SVG attributes

Related to #1919.

add a test of user shapes

Merge branch 'smattr/005D38E6-E842-4522-B368-616652003649' into 'master'

fix HTML validation errors

Closes #1861

See merge request graphviz/graphviz!1758

fix HTML validation errors in graphviz.html

Fixes #1861.