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

fix: consistently treat entry indices as uint64_t in tclhandle

This code intermingled uint64_t and int values for representing indices into an
array. Making this consistent removes a huge number of -Wsign-conversion and
-Wsign-compare compiler warnings, as well as more serious problem where printing
code assumed "%lu" was the format code to print an index. This is not true when
uint64_t is not the same type as unsigned long and would have most probably
resulted in stack corruption.

Merge branch 'smattr/b3c0b645-f677-4b03-b68f-18e05373d6ef' into 'main'

more clean up and warning squashing in plugin/core

See merge request graphviz/graphviz!1914

squash -Wunused-parameter warning in pic_polygon

squash -Wunused-parameter warnings in pic_bezier

squash two -Wsign-compare warnings in pic_string

This is the equivalent of 4b33b8c3ad41e03f441959a58dc3e5d65ffc12bd and
337190b7b9b35c114f13043fa4541a210e61925a which we need to apply for a third time
because this function has been copied and pasted. Related to #2051.

squash -Wsign-conversion warnings in pic_string

This is the equivalent of 5a5aa9c606db77f444d744a11de61eb75de2b6ea and
be2de1fec0e5279a54a16e66cd96cdb620029a30 which we need to apply for a third time
because this function has been copied and pasted. Related to #2051.

remove an open coded fmax

fold printing of some constants into the format string itself

There's no need to suffer the overhead of dynamically serializing these values
whose representation we know at compile time.

do pov_begin_graph computation entirely on doubles

This code was triggering a number of -Wfloat-conversion compiler warnings, and
there is really no need to truncate values during computation. We can do the
entire thing on doubles with better accuracy, though the final result is
truncated to three decimal places.

remove unnecessary calls to fabsf

fix some casts to incorrect target type

Removes two -Wsign-conversion warnings.

remove use of %d to emit unsigneds in VML plugin

squash -Wunused-parameter in tkgen_begin_edge

squash -Wunused-parameter warning in tkgen_begin_node

squash 8 -Wconversion warnings

remove unused parameters to svg_rgradstyle

remove commented out code

Merge branch 'smattr/b06d6e22-33f7-4371-ba03-b58b72af0dce' into 'main'

more modernization and removal of manual memory management in plugin/visio

See merge request graphviz/graphviz!1913

replace use of NULL in the Visio plugin with more modern nullptr

more precisely express inheritance relationship in Path classes

This gives the compiler a greater ability to understand the programmer’s
intentions and diagnose misuse of these classes.

remove manual memory management of _points in Path class

remove the use of %d to print unsigned integers in the Visio plugin

remove Visual Studio options files

Quoting the Microsoft docs,¹ “This file should not be checked in to source
code control.”

  ¹ https://docs.microsoft.com/en-us/visualstudio/extensibility/internals/solution-user-options-dot-suo-file?view=vs-2019

remove the use of C-style casts to convert void* to Visio::Render*

C-style casts are unnecessarily strong to use here. A reinterpret_cast gives the
compiler a better ability to warn about misuse.

Merge 'jfriesne/jaf-fix-uninitialized-memory-read-in-call-try' into main

add a changelog entry for prior commit

Fix for uninitialized-memory-read in call_tri() that would occur after delaunay_tri() returned NULL and never set numberofedges to anything

Merge branch 'smattr/d0423bd6-65fb-4d10-abf7-905a337bab9e' into 'main'

more warning squashing in cmd/smyrna

See merge request graphviz/graphviz!1910

remove unused display function

fix type of variable used for interacting with getopt

Fixes some -Wsign-conversion and -Wsign-compare warnings.

remove unused modefile parameter to load_mouse_actions

remove commented out code

fix some incorrect function return type definitions

Squashes a number of -Wold-style-declaration warnings.

squash -Wunused-parameter warnings in outfn

squash -Wunused-parameter warnings in on_dlgOpenGraph_btnOK_clicked

squash -Wunused-parameter warning in dlgOpenGraph_OK_Clicked

squash -Wunused-parameter warnings in save_graph_clicked

abbreviate an increment