remove shadowing of view in glexpose_main

remove shadowing of view in glexpose_drawgraph

remove shadowing of view in glexpose_grid

remove shadowing of view in glupdatecamera

remove shadowing of view in drawBorders

Merge branch 'smattr/41d547b2-c0af-430e-a0f6-df23eaa36b37' into 'main'

remove more manual memory management in lib/vpsc

See merge request graphviz/graphviz!1909

replace vpsc uses of NULL with more modern nullptr

use range-based for loops to abbreviate some manual iteration

manage NodeSets by value instead of pointer

There is no need to manually allocate memory for these sets. We can simply let
the compiler take care of this. Note that this is also unlikely to affect the
performance characteristics of getLeftNeighbours and getRightNeighbours because
Named Return Value Optimization (NRVO) is applicable to these functions.

mark getRightNeighbours static

This is not used outside of its containing file.

make getLeftNeighbours static

This is not used outside of its containing file.

Merge branch 'smattr/4984C046-2548-442C-9D61-CC86DF57501E' into 'main'

clean up and warning squashing in lib/neatogen

See merge request graphviz/graphviz!1908

remove unused parameter to mds_model

remove unused parameter from majorization

remove shadowing of i in majorization

abbreviate an increment

remove some commented out code

remove function pointer noise

When calling through a function pointer in C, you do not need to explicitly
dereference it. The compiler understands how to deal with these addressing
complications.

reflow some text

use a stronger return type for PQempty

abbreviate an increment

abbreviate an increment

abbreviate a decrement

abbreviate an increment

remove unnecessary bracketing

remove some no-op casts

These pointers implicitly coerce.

squash two -Wfloat-conversion warnings

remove unnecessary dynamic allocation

Using the heap for a statically-known array of two pointers is inefficient. This
can just be stack-allocated, improving the compiler’s visibility and ability to
optimize. Removes a -Wsign-conversion compiler warning.

treat SEPFACT as a float

This constant is only ever used as a float, but triggers warnings because 0.8
cannot be precisely expressed as a 32-bit IEEE float. If we want to use the
closest 32-bit float to 0.8 let's just say so. This change squashes 3
-Wfloat-conversion warnings.

remove -Wshadow warning related to margin

remove unused parameter from updateGraph

remove now-unused local variable in sAdjust

remove unused dim parameter to makeMatrix

remove some unnecessary bracketing

remove useless cast

fabs already returns a double, and type promotion results in a double result
from the division.

remove noise in nextOne

remove unused parameter from rePos

push declarations in chkBoundBox closer to their use

remove commented out code

remove shadowing of ymax

remove shadowing of ymin

remove shadowing of xmax

remove shadowing of xmin

Merge branch 'smattr/6B336651-C5E7-4BD2-A2D0-BAEDFBB4C765' into 'main'

fix multiple definitions of CDT variables under MinGW on Windows

Closes #1940

See merge request graphviz/graphviz!1760

fix multiple definitions of globals

Similar to preceding commits, this fixes an issue where MinGW was seeing
 __declspec alternatives but implementing GCC semantics and hence treating
declarations as definitions.

remove unused globals

fix multiple definitions of gvrender variables

Similar to preceding commits, this fixes an issue where MinGW was seeing
 __declspec alternatives but implementing GCC semantics and hence treating
declarations as definitions.

fix multiple definition of SFIO variables

Similar to preceding commits, this fixes an issue where MinGW was seeing
 __declspec alternatives but implementing GCC semantics and hence treating
declarations as definitions.

fix multiple definitions of MemTest and GvExitOnUsage

Similar to preceding commits, this fixes an issue where MinGW was seeing
 __declspec alternatives but implementing GCC semantics and hence treating
declarations as definitions.

fix multiple definitions of lab_gamut variables

Similar to preceding commits, this fixes an issue where MinGW was seeing
 __declspec alternatives but implementing GCC semantics and hence treating
declarations as definitions.

fix multiple definitions of gvplugin_neato_layout_LTX_library

Similar to preceding commits, this fixes an issue where MinGW was seeing
 __declspec alternatives but implementing GCC semantics and hence treating
declarations as definitions.

fix multiple definitions of dot_builtins variables

Similar to preceding commits, this fixes an issue where MinGW was seeing
 __declspec alternatives but implementing GCC semantics and hence treating
declarations as definitions.

fix multiple definitions of tcldot_builtins variables

Similar to preceding commits, this fixes an issue where MinGW was seeing
 __declspec alternatives but implementing GCC semantics and hence treating
declarations as definitions.

fix multiple definitions of lt_preloaded_symbols under MinGW

Similar to preceding commits, this fixes an issue where MinGW was seeing
__declspec alternatives but implementing GCC semantics and hence treating
declarations as definitions.

remove commented out code

fix multiple definitions of cgraph variables

Similarly to the previous commit, this was causing problems on MinGW where the
compiler would see the __declspec alternatives for CGRAPH_API, but then
implement GCC semantics where a non-extern marked declaration is a definition.
Related to #1940. Thanks to @swaldhoer for guiding me to a patch here.

fix multiple definitions of CDT variables under MinGW on Windows

Building with MinGW on Windows resulted in lots of build errors of the form:

  …/x86_64-w64-mingw32/bin/ld.exe: CMakeFiles\cdt.dir/objects.a(dtdisc.c.obj):
    dtdisc.c:(.bss+0x0): multiple definition of `Dtset';
    CMakeFiles\cdt.dir/objects.a(dtclose.c.obj):dtclose.c:(.bss+0x0): first
    defined here

The problem is that this environment defines _WIN32, so it sees the __declspec
alternatives for CDT_API in cdt.h. This is fine as GCC understands __declspec,
but it results in variable declarations in this header then missing the extern
qualifier. GCC’s semantics, in contrast to MSVC, interpret this as a definition
rather than a declaration.

The solution here is to *always* apply extern to these declarations, as MSVC is
happy with this as well. This is only addressing the immediate cause, and not
the longer term issue that __declspec(dllexport) and __declspec(dllimport) in a
Windows build should really be mapped to __attribute__((visibility("default")))
and __attribute__((visibility("hidden"))) in a non-Windows build, not to extern.

Thanks to @Synoecium and @swaldhoer for guidance on this fix.

Merge branch 'smattr/892B65E2-F263-411D-819B-504029B701A1' into 'main'

move towards standardizing Python style

See merge request graphviz/graphviz!1907

port generate-configuration-table.py to more modern Python style

remove legacy encoding tag on a Python file

These stanzas are no longer required in modern Python that is assumed to be
UTF-8.

standardize on two spaces for indentation in Python

The existing code was a mixture of two spaces and four spaces. Aligning on two
spaces allows writing more concise code and more closely matches the LLVM C
style we aspire to for new C code in the code base.

Removes 379 Pylint warnings.

standardize on double quotes for string delimiters in Python

Python supports using either single quotes or double quotes for strings. Given
Graphviz is predominantly a C application, it is expected developers working
within the code base will be most familiar with C. So using double quotes is
more intuitive for such programmers.

Merge branch 'smattr/d684a3df-0af7-4025-a1af-59f66b995e54' into 'main'

gdiplus plugin: some clean up and modernization

See merge request graphviz/graphviz!1906

remove some excess white space

gdiplus plugin: replace use of NULL with more modern nullptr

gdiplus plugin: swap a C-style cast for less powerful const_cast

This allows the compiler to warn if, e.g., us->name ever changes type to
something other than a char pointer.

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.