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

squash -Wunused-parameter warnings in button_release_event

squash -Wunused-parameter warnings in button_press_event

remove unused variable in button_press_event

squash -Wunused-parameter warnings in expose_event

squash -Wunused-parameter warnings in configure_event

squash -Wunused-parameter warning in realize

remove non-existent directory from Smyrna Makefile

Squashes some -Wmissing-include-dir warnings.

squash some -Wfloat-conversion warnings

squash a -Wfloat-conversion warning

squash a -Wfloat-conversion warning

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