CI: avoid BOM in COLLECTION file in windows build jobs

CI: avoid BOM in GRAPHVIZ_VERSION file in windows build jobs

Towards https://gitlab.com/graphviz/graphviz/-/issues/2133.

Merge branch 'smattr/05f4cdeb-0e09-4cfc-ac55-d8822807d4a5' into 'main'

libcommon: provide more generic XML escaping

See merge request graphviz/graphviz!2198

common: remove 'xml_url_string'

As of the previous commit, all calls to `xml_url_string` have been replaced with
calls to `xml_escape`.

Related to #1868.

replace 'gvputs(… xml_url_string(…))' with 'xml_escape(…)'

Similar to previous `xml_string` and `xml_string0` related changes, this avoids
dynamic allocations and makes the affected code thread safe.

Related to #1868.

common: remove 'xml_string0'

All calls of this function have now been refactored to use `xml_escape`.

Related to #1868.

common: remove 'xml_string'

All calls of this function have now been refactored to use `xml_escape`.

Related to #1868.

gv2gml emitAttr: replace 'fprintf(… xml_string(…))' with 'xml_escape(…)'

Similar to the prior commits, this avoids dynamic allocation and improves thread
safety. This is intended to be a functional no-op.

Related to #1868.

pango_textlayout: replace 'agxbput(… xml_string0(…))' with 'xml_escape(…)'

Similar to the prior commit, this avoids dynamic allocating – well, some,
`axbput` itself is still dynamically allocating – and improves thread safety.
This is intended to be a functional no-op.

Related to #1868.

replace 'gvputs(… xml_string0(…))' with 'xml_escape(…)'

This is, again, a functional no-op. But it has the effect of removing dynamic
allocations and making this code thread-safe.

Related to #1868.

replace 'gvprintf(… xml_string(…))' with 'gvputs_xml'

A first demonstration of how to use the `xml_escape` functionality introduced in
preceding commits. Note that, while this is intended to be a non-functional
change, this does side step intermediate buffering in `xml_string` and escaped
output is now passed directly to `gvputs`.

Related to #1868.

Visio plugin: replace redeclaration of 'xml_string' with a #include

There is no need for this duplicate prototype, or to lie about its type
signature.

Related to #1868.

implement a 'gvputs' alternative that does XML-escaping

This will be used in an upcoming commit to replace `gvputs` when used in
combination with `xml_string`.

Related to #1868.

plugin core: swap duplicated prototypes for missing includes

This file is calling functions `xml_string` `xml_url_string` without including
the header that prototypes them, utils.h. There seems to be no reason to avoid
this and locally re-prototype these functions.

Related to #1868.

expose a primitive 'xml_escape' for generic XML escaping

This function is intended to be a flexible XML-escaper that subsumes
`xml_string`, `xml_string0`, and `xml_url_string`. Unlike the preceding named
functions, calling `xml_escape` is thread-safe and does not mandate dynamic
allocation.

Related to #1868.

xml_url_string: call xml_core instead of repeating logic

Related to #1868.

xml_core: make '-' and ' ' escaping configurable

This is a non-functional change. It is preparation to allow `xml_url_string` to
use `xml_core`.

Related to #1868.

xml_string0: outline the inner loop body functionality

The motivation for this change is to move towards consolidating XML-escaping
functionality. The introduced function, `xml_core`, is intended to be usable by
`xml_url_string` and other Graphviz XML escaping functions in future.

This aim of consolidation hopefully also explains why this change looks more
complicated than might be expected. It is intended to be functionally a no-op.
But `xml_core` takes a generalized callback and state because it is imagined a
future caller may want to pass in, e.g. `gvputs` as their target sink for
XML-escaped data. That is, future changes may side step the static buffer usage
and allocation that happens in `xml_string0` and instead pump bytes directly
into a final target file.

Related to #1868

xml_isentity: take a const pointer instead of a mutable one

This function does not modify the pointed to data of its parameter, and this
will simplify some upcoming changes.

Related to #1868.

xml_isentity: outline [a-zA-Z] check

This has no functional effect, but makes the code slightly more readable.

xml_isentity: use ctype.h functions for readability

This has no functional impact, but just makes the code simpler to read and less
error prone.

Merge branch 'smattr/949303BC-D384-42CF-A449-3969EA5F0175' into 'main'

upgrade Pylint and remove 'whitelist' terminology

See merge request graphviz/graphviz!2201

switch to inclusive terminology 'allow-list' instead of 'whitelist' in Pylint

Pylint commit 12beaa0d3fa050b68dff092520966c63d2f0e437 added this alternative
spelling for this option, in line with the aim of using more inclusive
terminology. This made it into Pylint 2.7.3. As of the prior commit, Graphviz is
upgraded to Pylint 2.7.3 and can use this new spelling.

upgrade Pylint 2.7.1 → 2.7.3

Commit a9c626e7f6a81f0949b1e4134e17c8968f13256b cherry-picked some upcoming
Pylint changes related to inclusive terminology. This and the remainder of the
changes relevant to Graphviz (the `extension-pkg-allow-list` option) made their
way into Pylint 2.7.3. By upgrading to 2.7.3 “whitelist” terminology can be
removed from Graphviz’ Pylint configuration.

Note that a9c626e7f6a81f0949b1e4134e17c8968f13256b incorrectly quotes Pylint
commit hashes from PR3961¹ that never made their way into the main Pylint branch
as-is but were rebased. The corresponding commit hashes are:

  * 850c442dbda8d332049971d5c38f82b86da44f1d became
    d19c7733717ae9ad0a527233c2950473dce3ffcf
  * 12a76469782fab5c23d78039c4bfb37e3331945a became
    12beaa0d3fa050b68dff092520966c63d2f0e437

¹ https://github.com/PyCQA/pylint/pull/3961

Merge branch 'smattr/383BFACC-1BDD-4992-84FB-12ECD80D55FF' into 'main'

remove explicit support for PHP5 SWIG bindings in the Autotools build system

See merge request graphviz/graphviz!2200

remove explicit support for PHP5 SWIG bindings in the Autotools build system

Older versions of SWIG can generate bindings for a variety of PHP versions.
Periodically SWIG bumps the version of PHP targeted by the default flag,
`-php`:¹

  SWIG version       `-php` effect   PHP5 support?   PHP7 support?
  ================================================================
  <1.3.26            `-php4`         no              no
  ≥1.3.26, <1.3.37   `-php4`         yes             no
  ≥1.3.37, <3.0.11   `-php5`         yes             no
  ≥3.0.11, <4.0.0    `-php5`         yes             yes
  ≥4.0.0             `-php7`         yes             yes

This commit changes the behavior under the Autotools build system:

  SWIG version       old behavior     new behavior
  ================================================
  <1.3.26            target PHP4  ==  target PHP4
  ≥1.3.26, <1.3.37   target PHP5  →   target PHP4
  ≥1.3.37, <3.0.11   target PHP5  ==  target PHP5
  ≥3.0.11, <4.0.0    target PHP7  ==  target PHP7
  ≥4.0.0             target PHP7  ==  target PHP7

It seems reasonable to consider the first three rows of the above table
irrelevant. The last version of PHP5, 5.6, was EOLed on 2018-12-31,² so any
users in rows 1-3 of the table have been unsupported for some time. In practice,
only users in row 2 are affected by this change, who are using a version of SWIG
released prior to 2009-01-12.³

In future, it may make sense to drop build system support for the `-php` flag
altogether and *only* support `-php7`, given that using `-php` on an older
version of SWIG that defaults to PHP5 probably produces uncompilable code
anyway.

¹ http://www.swig.org/Doc4.0/Php.html
² https://www.php.net/eol.php
³ https://github.com/swig/swig/releases

Merge branch 'smattr/F4CD811A-5132-47F8-A5D3-2208237B457B' into 'main'

more smyrna dead code removal

See merge request graphviz/graphviz!2199

smyrna: remove 'btnToolMove_clicked' no-op

smyrna: remove 'btnToolCursor_clicked' no-op

smyrna: remove unused 'btnToolFindEdge_clicked'

smyrna: remove unused 'btnToolDeleteEdge_clicked'

smyrna: remove unused 'btnToolAddEdge_clicked'

smyrna: remove unused 'btnToolFindNode_clicked'

smyrna: remove unused 'btnToolDeleteNode_clicked'

smyrna: remove unused 'btnToolAddNode_clicked'

smyrna: remove prototypes for unimplemented toolbox functions

The implementations of these were all removed in
02e266e74b1cf217c9bc8555e9fd4608c2d3e91a.

smyrna: remove commented out code in smyrnadefs.h

smyrna: remove unused 'guibuffer'

The last use of this was removed in b7687bc4b8964f5257e61a248270960572435532.

Merge branch 'smattr/1B7E88AD-ABB7-4CC6-B676-7A7D653FB628' into 'main'

fix: upgrade Pytest 6.2.2 → 6.2.4

See merge request graphviz/graphviz!2202

fix: upgrade Pytest 6.2.2 → 6.2.4

Python 3.10 was released on 2021-10-04 with what appears to be a breaking
change.¹ Graphviz’ Windows CI tasks (intentionally) install an unpinned
version of Python 3, so they get whatever the latest Python 3 version available
on Chocolatey is. Python 3.10 became available in Chocolatey on 2021-10-05 and
Windows CI tasks immediately started failing. The root cause is that 3.10
exposes a latent bug in Pytest.² This bug is reportedly fixed in Pytest 6.2.4,
so upgrading should repair Windows CI.

¹ https://bugs.python.org/issue43798
² https://github.com/pytest-dev/pytest/issues/8539

Revert "plugin/core: autotools: make awk crlf agnostic when generating ps.h"

This reverts commit 50e7d9768b9c02287a0d68e64233c7f713d1f4b8.

In standard awk, only the first character in RS is used, so regular expressions don't work. This means the resulting ps.h in plugins/core contains an illegal C string constant.

Merge branch 'smattr/420913E5-02E6-430D-A210-0DD8BC3F7315' into 'main'

more smyrna dead code removal

See merge request graphviz/graphviz!2196

smyrna: remove unused 'frmObjectBtnCancel_clicked'

smyrna: remove unused 'on_dlgOpenGraph_btncancel_clicked'

smyrna: remove unused 'frmObjectBtnOK_clicked'

smyrna: remove unused 'on_dlgOpenGraph_btnOK_clicked'

smyrna: remove unused 'attr_widgets_modifiedSlot'

smyrna: remove 'attr_widgets_modified' that is never read

smyrna: remove unused gui.c globals

smyrna: remove unused 'Color_Widget'

smyrna: remove past deleted prototypes

These were all removed in 7bfe3c0d255829ff9a837ff85e3c14dd63636771.

smyrna: remove unused 'get_attribute_string_value_from_widget'

smyrna: remove prototypes of deleted functions

These functions were removed in af92d6ce857bfa235d6fe771088074a9979b7172.

Merge branch 'smattr/EE8B63B0-BCF9-4965-BF06-D885D15A5F4C' into 'main'

fix GVPR incorrect interpretation of color strings

Closes #1956

See merge request graphviz/graphviz!2197

fix GVPR incorrect interpretation of color strings

The logical operators here were pretty clearly meant to be `&&` not `||`. But
this kind of micro-optimization is unnecessary on modern machines anyway, so we
can just rewrite this into something that is more obvious for both readers and
the compiler.

It would have been nice to add a provoking test case for this as well, but it is
not clear to me how exactly to reach this path. This bug has been open for quite
a while, so it seems simpler to apply the obvious fix and move on.

Fixes #1956.

Merge branch 'smattr/EFCF3C3B-5099-47C0-8C68-B580D736A900' into 'main'

more smyrna clean up

See merge request graphviz/graphviz!2195

smyrna: mark 'cb_special_key_up' as static

This function is not used outside of its containing file.

smyrna: mark 'cb_special_key' as static

This function is not used outside of its containing file.

smyrna: mark 'cb_keyboard_up' as static

This function is not used outside of its containing file.

smyrna: mark 'cb_keyboard' as static

This function is not used outside of its containing file.

smyrna: remove useless 'cb_mouseover' callback

smyrna: remove useless 'cb_mouseentry' callback

smyrna: mark 'cb_drag' as static

This function is not used outside of its containing file.

smyrna: mark 'cb_mouseclick' as static

This function is not used outside of its containing file.

smyrna: mark 'cb_display' as static

This function is not used outside of its containing file.

smyrna cb_reshape: remove some commented out code

smyrna: mark 'cb_reshape' static

This function is not used outside of its containing file.

smyrna scale_coords: remove open coded fmin/fmax

Merge branch 'smattr/3594869f-443d-4168-9358-4195b0666b11' into 'main'

gv2gml: fix attribute escaping

Closes #1276

See merge request graphviz/graphviz!2189

gv2gml: fix attribute escaping

When processing characters in attribute values that should be escaped (e.g.
`"`), `gv2gml` would emit these as-is, resulting in invalid GML output. Such
attribute values are now correctly XML-escaped.

Fixes #1276.

gv2gml: also link to libgvc

Upcoming changes will make use of some functionality from libcommon, which is
consolidated into libgvc.

Related to #1276.

gv2gml: fix: realign MS Build include paths with CMake and Autotools

lib/common: move XML functions to a separate source file

Symbol resolution during linking is done on a per-file basis. Thus any use of a
function from labels.c was requiring all functions in labels.c to have their
symbols resolved. This is probably one of the partial causes of the
proliferation of duplicate XML escaping code (#1868).

This commit splits the XML escaping code into a separate file, allowing it to be
used without also requiring dependencies of unrelated functions in labels.c to
be satisfied. In the specific case motivating this, this will enable `gv2gml` to
call these XML escaping functions without linking against libcdt and libgvc.

For the above explained linking reasons, in future it may be desirable to
further separate library functions at an even finer grained basis than done in
this commit.

Related to #1276.

gv2gml: rename a 'path' parameter to 'cmd'

This has no immediate effect. However an upcoming change will introduce a header
include that brings in another 'path' symbol. So this change pre-emptively
squashes the -Wshadow compiler warning that would otherwise be introduced.

Related to #1276.

gv2gml: rename 'Agnodeinfo_t' type to 'Local_Agnodeinfo_t'

This has no immediate effect, but having a type named `Agnodeinfo_t` prevents
including lib/common/types.h which has its own `Agnodeinfo_t`. We will want to
include this header (transitively) in an upcoming commit.

Related to #1276.

gv2gml: remove unused struct name

lib/common: fix a comment typo

add a test case for invalid GML output

Related to #1276.

Merge branch 'smattr/6E1345C1-7D81-44AE-A450-A03CF02EBEFC' into 'main'

adjust the release process to not require gen_version.py tweaks

See merge request graphviz/graphviz!2186

adjust the release process to not require gen_version.py tweaks

This change has two primary motivations:

  1. Streamlining the release steps and reducing the opportunities for human
     error; and

  2. Improving clarity in the release artifacts.

Following this change, no edits are required to gen_version.py in order to cut
a new release. The Graphviz version – both inter-release and for releases
themselves – is derived exclusively from CHANGELOG.md. The effect of this is
expected to be:

  1. Cleaner release commits. No more regular churn of gen_version.py.

  2. More obvious CHANGELOG.md in the release artifacts. This file will no
     longer have a spurious “[Unreleased]” heading preceding the actual
     changelog for the release.

Merge branch 'smattr/B81C2CF2-04BB-446A-8473-6CC49CC21B5A' into 'main'

smyrna: yet more dead code removal

See merge request graphviz/graphviz!2184

smyrna: remove unused object init functions

smyrna: remove now-unused cbSelectGraph

smyrna: remove unused get_SelectGraph

smyrna: remove unused frmObjectg

smyrna: remove unused frmObjectTypeIndex

smyrna: mark filter_attributes as static

This is not used outside of its containing file.

smyrna: remove unnecessary binarySearch prototype

smyrna: remove unused topviewdefs.h

smyrna: remove commented out code

smyrna: remove duplicate _BB definitions

There was the option to remove the first duplicate or the second in this file. I
checked that none of the intervening includes depended on a `_BB` definition. It
seemed clearer to remove the first and indicate that only Smyrna itself is using
`_BB`.

Merge branch 'smattr/916A53CB-2137-4E39-802A-72DCD4716DD5' into 'main'

toggle Lefty to disabled by default in the Autotools build system

See merge request graphviz/graphviz!2194

debian/rules: remove commented out lines

remove some commented out RHEL build lines

toggle Lefty to disabled by default in the Autotools build system

Based on a discussion that Lefty should be deprecated,¹ this takes the first
step towards removing it. It is hoped that by disabling it by default, this will
expose any downstream consumers who are relying on it being built by default.

If there are no complaints within three months, I think we can then remove build
system support entirely. If another three months pass without noise, I think we
can then remove Lefty sources from Graphviz.

In case the linked discussion is inaccessible when reading this commit message,
the summary from Stephen was:

  Lefty was an early interpreter for graphics programming. It’s from about 1990
  or 91, and we used it as the basis for a scriptable graphviz viewer, dotty.
  It was student code written by Lefteris Koutsofios. I thought there was some
  C code for parsing graphs but I could be wrong.

  Even libast came into the picture much later, but I suppose that was done to
  clean up some of its OS dependencies or to make it more portable or robust in
  some way.

  I’m ok with retiring it.

¹ https://forum.graphviz.org/t/lefty-not-a-graphviz-application/835

explicitly enable Lefty compilation in Autotools CI

Lefty is enabled by default currently, but an upcoming change will toggle this
to default-disabled.

Merge branch 'add-cygwin-autotools-build-to-ci' into 'main'

Add Cygwin autotools build jobs to CI

See merge request graphviz/graphviz!2193

CI: add windows-cygwin-build-using-autogen job

ci/build.sh: autotools: add support building for Cygwin using autogen

ci/cygwin-build.sh: add installation of autoconf2.5 & automake