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

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