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

squash -Wunused-parameter warnings in on_consoleincbtn_clicked

abbreviate a decrement

squash -Wunused-parameter warnings in on_consoledecbtn_clicked

squash -Wunused-parameter warnings in on_hideconsolebtn_clicked

squash -Wunused-parameter warnings in on_clearconsolebtn_clicked

squash -Wunused-parameter warnings in frmObjectBtnCancel_clicked

squash -Wunused-parameter warnings in frmObjectBtnOK_clicked

squash -Wunused-parameter warning in frmObject_set_scroll

squash -Wunused-parameter warning in attr_widgets_modifiedSlot

squash -Wunused-parameter warnings in on_dlgOpenGraph_btncancel_clicked

squash -Wunused-parameter warnings in on_newNode

remove useless memory leak

It is unclear what this code was attempting to achieve. It allocated memory and
then immediately lost the pointers to this memory.

squash -Wunused-parameter warning in btn_fdp_clicked

squash -Wunused-parameter warning in btn_circo_clicked

squash -Wunused-parameter warning in btn_twopi_clicked

squash -Wunused-parameter warning in btn_neato_clicked

squash -Wunused-parameter warning in btn_dot_clicked

squash -Wunused-parameter warnings in btn_clicked

squash -Wunused-parameter warnings in remove_graph_clicked

squash -Wunused-parameter warnings in save_as_graph_clicked

squash -Wunused-parameter warnings in open_graph_clicked

squash -Wunused-parameter warnings in new_graph_clicked

remove curse word from comment

There's no need for this in the code base.

squash -Wunused-parameter warning in motion_notify_event

remove unused key parameter to appmouse_key_release

remove no-op functions appmouse_left_click, appmouse_right_click

This squashes some -Wunused-parameter warnings.

squash -Wunused parameter warnings in scroll_event

squash -Wunused-parameter warnings in key_release_event

squash -Wunused-parameter warnings in key_press_event

remove some excess white space