remove redefining bool to int in Smyrna

This appears to have no effect, as C99 bools are not used anywhere in Smyrna or
in the headers included by viewport.h. However this is a trap waiting to cause
problems if/when C99 bools are used in these files.

Merge branch 'smattr/610A0333-73CC-4DE3-9188-DAC1B96D3F2B' into 'main'

port windows/bin/build.ps1 to Python

See merge request graphviz/graphviz!1941

remove now-unused build.ps1

remove now-unused setup-and-build.ps1

call Python script for Windows CI compilation instead of PowerShell

Following the previous commit, ci/build_windows.py provides a drop in
replacement for the PowerShell script previously used for compilation.

port windows/bin/build.ps1 to Python

This is a first step to moving away from PowerShell for CI scripts. PowerShell
has too many error prone characteristics and there is not enough knowledge about
it amongst the Graphviz maintainers.

Merge branch 'smattr/8B3F2F18-8D38-4668-AAD4-F7EAEFDDB151' into 'main'

Pylint warning squashing

See merge request graphviz/graphviz!1938

remove duplicate shutil import

squash a wrong-import-position Pylint warning

disable Pylint warning about use of the global statement

While this may be considered a code smell in larger Python code bases, the use
of global in the Graphviz scripts is reasonably well motivated.

disable Pylint warning about too many local variables

This is not a significant issue in Graphviz scripts.

port generate_configuration_table to argparse

Removes a Pylint warning about using the deprecated optparse library.

remove unused --verbose argument to configuration summarizer

disable invalid-name Pylint warning

Pylint has a very narrow idea of acceptable names. Things it does not like, such
as single letter variable names, are not really a problem in the Graphviz
scripts.

rename generate-configuration-table.py to be importable

Silences a Pylint warning.

disable Pylint warning for subprocess.run without check argument

Graphviz generally have a good use case for calling subprocess.run without
passing check.

disable f-string in logging messages Pylint warning

This is generally considered error prone because you can include non-pure
operations in logging functions that your code accidentally depends on. However
the Graphviz scripts are simple enough that this is not a significant issue. The
use of f-strings pervasively makes for more consistent and readable scripts.

Merge branch 'stdio-standard-macros' into 'main'

sfio: remove singular reference to `_BLD_stdio`

See merge request graphviz/graphviz!1943

sfio: remove singular reference to `_BLD_stdio`

Remove the singular reference to `_BLD_stdio`.  This is part of a series
to remove unused build macros to unify on the CMake standard `*_EXPORTS`
macros.

Merge branch 'smattr/39EB89E6-4BE4-4737-AE77-5C2B31922DBB' into 'main'

remove unused tmsize10.clo

Closes #2061

See merge request graphviz/graphviz!1937

remove unused tmsize10.clo

Fixes #2061.

Merge branch 'smattr/ac55eb33-3e00-4d16-8daf-b70d0fec6825' into 'main'

remove unused md5 implementation in Smyrna

Closes #2050

See merge request graphviz/graphviz!1936

remove now-unused md5 implementation in Smyrna

Closes #2050.

remove unused get_md5_key

Related to #2050.

remove unused fill_key function

Related to #2050.

remove now-unused orig_key field in Smyrna’s Viewinfo struct

Related to #2050.

remove dead stores to orig_key field in Smyrna

This field is never read from. Related to #2050.

remove unused final_key field from Smyrna’s Viewinfo struct

Related to #2050.

Merge branch 'smattr/2bfe1f77-53cc-4eda-bc00-454cd8893455' into 'main'

remove use of unsigned ints for interacting with getopt

See merge request graphviz/graphviz!1935

gc: fix type of variable used for interacting with getopt

Similar to the prior commits. Squashes a -Wsign-compare warning and a
-Wsign-conversion warning.

gvmap: fix type of variables used for interacting with getopt

Similar to the prior commit. Squashes two -Wsign-compare warnings and two
-Wsign-conversion warnings.

edgepaint: fix type of variable for interacting with getopt

Similar to 49fd8f62d8103d6e40a33922d2492a5ec3e3e76e, this location was using an
unsigned int for some reason. This change squashes a -Wsign-compare and a
-Wsign-conversion warning.

Merge branch 'smattr/BB1ACA26-949E-46F0-AD76-06C877563ABC' into 'main'

some cleanup of cmd/tools/dijkstra.c

See merge request graphviz/graphviz!1934

remove some unnecessary bracketing

remove unused struct name

use an unsigned type for indexing in dijkstra

mark dijkstra function static

This is unused outside of this file.

squash some -Wsign-conversion warnings

squash some -Wunused-parameter warnings

use stronger types for dijkstra command line flags

free some allocated memory prior to exiting dijkstra

This is not strictly necessary, but aids debugging with tools like Valgrind.

remove the use of 0 for pointers in dijkstra

use calloc instead of malloc for some arrays in dijkstra

remove use of HUGE

HUGE_VAL is standard C, portable across environments.

remove some commented out code

Merge branch 'smattr/6d5c1473-c70e-45ad-81b8-5a6386ba2986' into 'main'

add a test case to keep .vcxproj and .vcxproj.filters sources in sync

See merge request graphviz/graphviz!1932

add a test case to keep .vcxproj and .vcxproj.filters sources in sync

Inspired by reviewing !1925, I realized there are few safeguards for those of us
editing the .vcxproj files while not on Windows. This commit introduces a basic
seatbelt to try to keep these files in sync. In future we could extend this to
check for file existence and other consistency.

Merge branch 'tred' into 'main'

GVC: export 'gvToolTred'

Closes #2057

See merge request graphviz/graphviz!1925

GVC: export 'gvToolTred'

Merge branch 'close-the-door' into 'main'

xdot: remove unused Windows.h header

See merge request graphviz/graphviz!1930

xdot: remove unused Windows.h header

This header is unused and does not need to be included.  Although for
most uses this does not impact anything, attempting to use this in a
Swift program causes problems due to the library being pulled into a
modular build.  This removal allows the use of xdot in Swift.

Merge branch 'smattr/3C4AE1AF-408F-4CB2-9F54-0457CE6C3F52' into 'main'

remove unused lib/inkpot

Closes #2054

See merge request graphviz/graphviz!1929

remove some non-existent paths from MSVC include directories

The reference directories were previously removed.

remove unused lib/inkpot

Closes #2054.

Merge branch 'smattr/AB2A899C-BF99-4303-82F9-79AA1D1078F8' into 'main'

try to work around `lneato -?` flakiness on Windows

See merge request graphviz/graphviz!1928

try to work around `lneato -?` flakiness on Windows

Related to #1934.

Merge branch 'smattr/880181C2-61D1-421D-9FA4-B15E4A19C741' into 'main'

remove /etc/redhat-release path in build script

See merge request graphviz/graphviz!1921

remove /etc/redhat-release path in build script

Every distro Graphviz CI now runs on appears to have an /etc/os-release, so this
code is unused.

Merge branch 'smattr/3E2C5588-8462-437B-9A7B-2ADA9D63769C' into 'main'

clean up "#xxxxxx" color printing

See merge request graphviz/graphviz!1918

abbreviate r2hex in lib/sparse

This is a shorter way of writing the same thing.

simplify color printing in lib/sparse

Equivalent of the previous commit, but applied to another function which is a
duplicate.

simplify color printing in mm2gv

This is a shorter, more efficient way to accomplish the same thing.

Merge branch 'smattr/747D40FC-E73B-4D36-A74A-99425E25AD8A' into 'main'

remove assumption in Windows builds that python3.exe exists

Closes #2049

See merge request graphviz/graphviz!1920

remove python3.exe copying in Windows CI

With the prior couple of changes, neither the Windows build nor the test suite
relies on a python3.exe binary. So we can simplify Windows CI by removing these
steps, leaving only a python.exe binary. This is more reflective of the typical
environment a user will have too. Related to #2049.

call Python as `python` instead of `python3` in MSBuild files

This removes an assumption from these files that Python 3 is available as
`python3`. On a Windows system, it is typically python.exe and users have to
create their own python3.exe if they want to call it under this name. Related to
#2049.

use CMake support to locate Python 3 before calling it

This change means (1) the CMake build system will fail less cryptically when
Python 3 is unavailable and (2) the CMake build system is now able to locate and
use Python 3 even when it is not called `python3`. Related to #2049.

remove test suite assumption that Python can be run as `python3`

On Windows, python3.exe does not typically exist. The python.exe binary is
Python 3 and is intended to be called as just this. The CI setup currently works
around this by copying python.exe to python3.exe. This change removes any
dependency on python3 existing, which should ease life for both CI and Windows
users wanting to run the test suite. Closes #2049.

remove unused cmake-build_and_test.sh script

The CMake builds are driven from build.sh, not from this script.

Merge branch 'independence-inn' into 'main'

build: use CMake to drive the position independence

See merge request graphviz/graphviz!1923

build: use CMake to drive the position independence

This replaces the use of `-fPIC` with `CMAKE_POSITION_INDEPENDENT_CODE`
which is used by CMake to initialise the `POSITION_INDEPENDENT_CODE`
property on targets, which allows the portable way to indicate the
compiler-independent spelling for `-fPIC`.  This was previously unset on
MSVC because MSVC does not support the `-fPIC` option and Windows by
default is PIC by means of the base relocation of the binary.

Merge branch 'smattr/8AC007D7-AEDE-49D4-800C-266AB9198BDD' into 'main'

squash some Pylint warnings

See merge request graphviz/graphviz!1919

remove an unused variable

remove unused parameter to judge()

disable Pylint too-few-public-methods

What kind of a warning is this? A class with even a single method isn’t a
mistake. It’s likely abstraction or indicating something semantic to the reader.

remove an unnecessary Python class

suppress an import-position Pylint warning

squash multiple-imports Pyint warning

remove an unnecessary class in a Python file

replace tabs in one Python file

squash no-else-after-return Pylint warnings

disable Pylint too-many-branches and too-many-statements

Pylint’s definition of “too many” is quite low, especially for the kind of glue
code we use Python for.

squash Pylint warnings for multiple statements on a line

disable Pylint redefined-builtin warning

We do not care about redefining things like `input`.

squash wrong-import-order Pylint warnings for run_c

fix a numbe of Pylint import-order warnings

remove a Python == comparison against None

replace exit() with sys.exit() in Python files

remove unnecessary semi-colons in Python files

fix some Pylint lines-too-long warnings

remove unused imports in Python files

remove trailing newlines from Python files

remove trailing whitespace in Python files

Merge branch 'smattr/AD0827FA-EDD0-4329-BA29-B4A977515179' into 'main'

add sprintf-like helper

See merge request graphviz/graphviz!1916

replace a use of snprintf pattern with gv_sprint

This commit serves as an example of how to use the functionality added in the
previous commit. Related to #1950.

add sprintf-like helper

The function sprintf is generally considered unsafe to use in modern C. In the
Graphviz code base, its uses are slowly being replaced with safer alternatives
(#1950). However, the alternative code is frequently overly verbose and
counter-intuitive. This commit adds (hopefully) convenient alternative functions
that can be used to replace sprintf uses. The gv_sprint function is similar to
the GNU asprintf function with a different calling convention.

The tests added alongside follow the same strategy as used for testing
lib/vmalloc.

Merge branch 'smattr/E31C3F18-8D47-4280-873E-857FC13A19AB' into 'main'

clean up and warning squashing in tclpkg/

See merge request graphviz/graphviz!1915

remove unnecessary bracketing and space in tclhandle.c

remove unused ISSPACE

compute tclhandleEntryAlignment at compile time

There is no need to defer this to runtime. Making this a compile-time constant
allows the compiler to constant-fold computations involving it, producing more
efficient code.

Note, this is not really even the correct notion of platform alignment. The
alignment of a type is not guaranteed to be the same as its size. Nevertheless,
we leave this alone for now.

remove useless cast