gdiplus plugin: remove the use of C-style casts to convert void pointers

In C++, C-style casts are a heavy hammer that is generally too powerful for your
needs. It effectively says to the compiler, “shut up, I know what I’m doing” and
impedes its ability to give you useful diagnostics. By using less powerful
reinterpret_casts, we preserve the compiler’s ability to warn about, e.g.,
implicitly converting a const pointer to non-const.

gdiplus plugin: remove some unnecessary casts

Typed pointers implicitly coerce to void* in C++.

gdiplus plugin: remove manual memory management of font pointers

Merge branch 'smattr/74956806-d2f0-4864-b125-77374faa7de5' into 'main'

more clean up and modernization in lib/ortho

See merge request graphviz/graphviz!1905

use a stronger return type for is_parallel

use a stronger return type for rectIntersect

remove unused return type from get_vertex_positions

use a stronger return type for inside_polygon

use C99 bools in _greater_than

use a stronger return type for edge_exists

use a C99 bool for segment_t.is_inserted

Now that we compile with C99, there is no need to use an ambiguous type like int
or the hand rolled boolean values for something like this.

use a C99 bool return type for init_query_structure

Now that we compile with C99, there's no need to use the internal boolean type
for things like this.

use stronger types for comparators in lib/ortho/trapezoid.c

This leads to more readable and obvious code.

remove unused return value from _min

remove unused return value from _max

remove some commented out code

replace header guards in lib/ortho with more modern #pragma once

reflow some code in lib/ortho

remove some unnecessary bracketing in lib/ortho

replace some open coded max/min with fmax/fmin

remove redundant __USE_ISOC99

The build system sets -std=c99 these days, which already achieves this.

Merge branch 'add-ubuntu20-10-cmake-ASan-build-job-to-ci' into 'main'

Add Ubuntu 20.10 CMake ASan build and test jobs to CI

Closes #1881

See merge request graphviz/graphviz!1899

Add ubuntu20-10-cmake-ASan-test job to CI

Add ubuntu20-10-cmake-ASan-build job to CI

Add use_santizers option to CMake build

ensure ID_LIKE is always set on Debian in CI

When running on the native Linux host in Gitlab CI instead of within a Docker
container, ID_LIKE is not set in /etc/os-release. So we set it based on ID if
that is available. Related to #1881.

Merge branch 'smattr/409a27e8-3d56-43a8-96aa-8763cae871c5' into 'main'

clean up and warning squashing in plugin/core

See merge request graphviz/graphviz!1902

remove 3 -Wunused-parameter warnings

squash a -Wsign-converion warning

remove vasprintf alternative in pov rendering

When we need to maintain a fallback option anyway, having a vasprintf
implementation does not gain us much. Moreover, nothing in the build system was
checking for the existence of vasprintf, so this code was never being used
anyway.

fix reuse of va_list in pov rendering

While this often works out on certain architectures, it is technically not
allowed to use a va_list twice. This change fixes the el function for platforms
where reusing va_lists is not possible.

remove short buffer path in pov rendering el

C99 vsnprintf semantics let you call the function with a NULL pointer to
discover the required number of bytes to print the given string. With this
ability, there is no longer any advantage to having two paths through this
function.

remove unused variable assignment

mark pov rendering function el as static

This function is not used outside this file. Squashes a -Wmissing-prototype
warning.

squash 2 -Wunused-parameter warnings

squash a -Wunused-parameter warning

squash a -Wconversion warning

squash 2 -Wsign-conversion warnings

mark html_string in vml rendering as static

This is not used outside this file. Squashes a -Wmissing-prototype warning.

squash a -Wunused-parameter warning

squash 3 -Wunused-parameter warnings

squash 2 -Wunused-parameter warnings

squash 2 -Wunused-parameter warnings

squash some -Wundef warnings

squash 2 -Wunused-parameter warnings

squash 3 -Wunused-parameter warnings

squash a -Wunused-parameter warning

squash 2 -Wunused-parameter warnings

squash 3 -Wconversion warnings

squash a -Wsign-conversion warning

squash some -Wsign-conversion warnings

squash 2 -Wunused-parameter warnings

squash 3 -Wconversion warnings

squash a -Wsign-conversion warning

squash a -Wsign-conversion warning

use a stronger type to toplevel parameter of set_attrwf for consistency

use a stronger type to squash 3 -Wconversion warnings in set_attrwf

remove the need to cast xdot_str parameters

Squashes a -Wcast-qual warning.

squash some -Wunused-parameter warnings

squash a -Wsign-conversion warning

squash a -Wconversion warning in xdot_begin_graph

Unclear why this variable was an int when it was even named as if it was an
unsigned short.

rewrite versionStr2Version to use strtoul

The atoi function has no ability to report failure. There is no advantage to
using it over the safer strto* functions. This change also removes a
-Wconversion warning.

remove shadowing of xbuf global

remove commented out code

remove unused PIC_COORDS_PER_LINE

Merge branch 'smattr/f7e0325f-8ecb-48b3-8e51-bb05a383d72d' into 'main'

more clean up of manual memory management in visio plugin

See merge request graphviz/graphviz!1900

remove CreateHyperlink

This function was adding no value over just calling the Hyperlink constructor
directly.

manage Hyperlinks by value instead of pointer in the visio plugin

There is no inheritance involved here, so there was no advantage to manually
managing memory for Hyperlinks or passing them around by pointer.

fix: use managed memory for Hyperlink’s string pointers

Similar to the prior fix to Run, this removes a latent safety issue with the
management of these pointers.

manage _run member of Text as a value instead of a pointer

This member is never set to NULL and the Run class is not involved in any
inheritance hierarchy. So there is no need to heap-allocate it or pass it around
by pointer.

fix: use managed memory for Run’s _text member

This removes the need to manually manage memory for this member but, more
importantly makes this class copy-safe. The implicit copy constructor of this
class would copy the pointer in the _text member. When either the original
object or the new copy was deleted, the pointer would be freed leaving its
duplicate in the other copy dangling. Any attempt to access this would result in
use-after-free and deleting the object would cause a double-free.

Following this change, it is safe to copy a Run object. This issue was latent,
because no existing code causes a Run object to be copied.

Merge branch 'smattr/ad4aa51f-c3fd-4ba1-97a5-61d0fba501eb' into 'main'

some warning clean up in cmd/smyrna

See merge request graphviz/graphviz!1898

squash -Wmissing-prototypes warning

remove unused parameter from glmotion_zoom

remove unused inc_value variable in glmotion_zoom_inc

remove unused Status variable

This was only used by LoadGLTextures that was recently removed.

squash a -Wfloat-conversion warning

remove unused LoadGLTextures

squash -Wunused-parameter in EmbedText

squash -Wunused-parameter warning in InsertImage

squash -Wunused-parameter warning in SetFont

squash -Wunused-parameter warnings in SetStyle

squash -Wunused-parameter in SetPenColor

squash -Wunused-parameter warning in SetFillColor

fix -Wpointer-compare warning

remove unused filled variable from DrawPolygon

remove unused parameter from set_options

squash a -Wstrict-prototypes warning

remove unused parameter from arcmouseDrag

remove unused parameter from arcmouseClick

remove unused code

Merge branch 'smattr/5EDB9D68-064A-4027-AFC3-A46101BAF4E5' into 'main'

in Windows CI tasks, install Python 3 from Chocolatey

See merge request graphviz/graphviz!1896

remove Python 3.8 fallback in Windows CI tasks

As of the previous commit, Python >3.8 is installed prior to running this
script, so no fallback should be required. Python 3.8 can be removed from the
Windows build utilities repository.¹

  ¹ https://gitlab.com/graphviz/graphviz-build-utilities

install Python 3 as a pre-requisite in Windows CI tasks

Prior to this change, Python 3.8 was used on Windows in CI via the Windows
build dependencies repository.¹ This works but has a number of shortcomings:

  1. Upgrading the version of Python used in CI requires access to a Windows
     machine; and
  2. Upgrading the version of Python used in CI requires building CPython from
     source; and
  3. Python-dependent failures are hard to root cause because they may be an
     issue with how we built CPython.²

As of this commit, Python 3 is installed via the Chocolatey³ package manager.
At time of writing, this installs Python 3.9.4. This is not pinned to a specific
version – i.e. a new Chocolatey package of Python 3 will result in Graphviz
immediately using that in CI. I believe this is actually the behavior we want
right now; pro-actively catching any issues caused by Python upgrades.

This does not change the baseline requirement for Graphviz CI/test scripts from
Python 3.6. Scripts should still be written to be backwards compatible to 3.6.

As this change introduces a new non-trivial step in the Windows CI tasks,
execution time is relevant. At time of writing, the macOS jobs dominate Graphviz
CI execution time, both before and after this change. Thus I think this is
acceptable.

  ¹ https://gitlab.com/graphviz/graphviz-build-utilities
  ² E.g. #2000, “CI Python OSErrors from Windows tasks”
  ³ https://chocolatey.org/

Merge branch 'smattr/d3552219-a755-4a56-9a55-2eebb5819810' into 'main'

more vpsc clean up

See merge request graphviz/graphviz!1895

push a variable closer its use in generateYConstraints

push a variable closer to its use in generateXConstraints

use range-based for loops to clean up some iteration

cut variable dependency and push closer to its use in generateYConstraints

cut variable dependency and push closer to its use in generateXConstraints