document parseXDotFOn in the libxdot API

Related to #1957.

document statistics function in the libxdot AP

Related to #1957.

fix: realign libxdot man page description of xop_kind

This enum was changed in 2ebad94d317711522edaff778825fadda08f7584 but the man
page was not updated. Related to #1957.

fix typo in libxdot man page

Merge branch 'output-md' into 'master'

Extract html_grammar.html to independent make rule

See merge request graphviz/graphviz!1853

Extract html_grammar.html to independent make rule

This lets us build it separately and include it in the Jekyll site.

Merge branch 'smattr/52520E52-65F9-46FF-AFB5-92DBFB4D18DB' into 'master'

pylintrc: backport inclusive language change from Pylint

See merge request graphviz/graphviz!1838

pylintrc: backport inclusive language change from Pylint

Pylint commit 850c442dbda8d332049971d5c38f82b86da44f1d modified the pylintrc
template to remove the use of the problematic term “blacklist.” This commit
extracts the relevant parts of that and applies it to our code base, removing
the term in the Graphviz tree as well.

When upgrading to a version of Pylint that includes commit
12a76469782fab5c23d78039c4bfb37e3331945a, we should also swap our usage of
extension-pkg-whitelist for extension-pkg-allow-list.

Merge branch 'arrows-quote-close' into 'master'

Arrows docs: Add attribute quoting and tag closing

See merge request graphviz/graphviz!1852

Merge branch 'arrows-delete-template' into 'master'

Remove arrows.html templating

See merge request graphviz/graphviz!1851

docs bugfix: recreate arrow_table.html

Don't just append.

Arrows docs: Add attribute quoting and tag closing

This is to appease the more picky Markdown HTML parser.

Merge branch 'colors-no-nbsp' into 'master'

Replace non-breaking spaces with CSS class

See merge request graphviz/graphviz!1850

Merge branch 'colors-no-a' into 'master'

mkcolors.awk: add HTML quotes and close TR tag

See merge request graphviz/graphviz!1849

Remove arrows.html templating

It has been migrated to the other repo in:

https://gitlab.com/graphviz/graphviz.gitlab.io/-/merge_requests/289.

Merge branch 'mkcolors-md' into 'master'

mkcolors.awk: add HTML quotes and close TR tag

See merge request graphviz/graphviz!1848

Replace non-breaking spaces with CSS class

This is simpler and allows easier control of width styling.

mkcolors.awk: Remove anchor tag from colors

This prevents styling the color as a link (with underline).

mkcolors.awk: add HTML quotes and close TR tag

This makes the HTML well-formed enough to include inside a markdown file

Without these changes, the kramdown markdown editor struggles.

Merge branch 'smattr/fb1d8a1c-fbe1-4a18-80c1-afb32a72f94b' into 'master'

compress CIELAB gamut lookup table

Closes #1974

See merge request graphviz/graphviz!1837

compress CIELAB gamut lookup table

The lab_gamut_data array is a lookup table (LUT) for some approximation of the
visible gamut of the CIELAB color space.¹ We have previously had extensive
problems compiling this file on Windows due to memory usage of MSVC.
0927817d203d61305da2b019f244744c10b588e6 finally brought things under control by
removing all struct usage which somehow was a factor.

This commit further reduces our requirements and guards against future such
situations. The LUT now, instead of listing every valid CIELAB value, lists
ranges in the CIELAB space. Some numbers:

                                   old     new
                                  --------------
    file size                     8.3MB   149KB
    LUT size                      2.4MB    43KB
    gcc-8.3 -O3 -c lab_gamut.c    5.8sec  0.8sec
    max lab_gamut() malloc         18MB   151MB

If necessary, we could reduce the dynamic allocation performed by lab_gamut() by
doing a pre-scan to calculate how many actual CIELAB values we need to store for
a given call.

  ¹ I say “some approximation” because all my attempts to distill a formula for
    deriving this table have failed. The closest is to enumerate the CIELAB
    space, translate each color into Standard RGB and then check whether the
    resulting value is within (0,0,0) - (255,255,255). However, this still is
    not quite exact. I expect the machine on which this LUT was originally
    generated uses a differently floating point representation or precision than
    any that I have available.

remove incorrect legacy comment

fix: missing value in CIELAB lookup table

This lookup table contains an approximation of the visible gamut of the CIELAB
color space.¹ While scaling to, e.g., Standard RGB is not linear, it is
continuous. Therefore it follows that if (L=5,a=22,b=6) is visible and
(L=5,a=22,b=8) is visible, (L=5,a=22,b=7) is also visible. This was presumably
an accidental deletion committed in pre-version-control days. Fixes #1974.

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

Merge branch 'smattr/11A3B453-C39B-4B15-91B1-3312A8B52698' into 'master'

track metrics from pylint job in CI

See merge request graphviz/graphviz!1836

track metrics from pylint job in CI

This replicates the metric tracking from some other CI jobs for the pylint task.
This should make the number of pylint warnings appear in the metrics section of
MRs, allowing us to track whether MRs increase or reduce the number of pylint
warnings.

Merge branch 'smattr/D5D52AEF-8F8A-4CFE-B22D-889A8D2F960D' into 'master'

remove unused files

See merge request graphviz/graphviz!1832

remove unused layout_similarity.c

remove unused lib/circogen files

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