pango: initialize a variable

Squash
warning: ‘availfaces’ may be used uninitialized in this function

Actually uninitialized value is not used because it is used
in pair with family_name, but compiler still complains.
So just add initializtion.

gvpr: initialize a variable

Squash
warning: ‘alpha’ may be used uninitialized in this function

One branch of conditional statement leaves variable alpha
uninitialized. Actually uninitialized value is not used,
but compiler still complains. So just add one assignment to
assumed 0.

BTW, the code can be refactored and shortened.
But it is preferable to just minimize the changes.

dotgen: initialize a variable

warning: ‘maxthispass’ may be used uninitialized in this function

tools: initialize a variable

Squashing
warning: ‘sym’ may be used uninitialized in this function

Here was no real usage of uninitialized variable, but
compiler still complains.

Pointer sym is allocated when flag with_val is set:

    if (with_val) {
        sym = agattr(g, AGEDGE, "len", "1");
    }

So just squash the warning by setting the
pointer to NULL and test it in guard condition
instead of testing the flag:

            if (sym && val) {
                sprintf(buf, "%f", val[j]);
                agxset(e, sym, buf);
            }

htmltable: initialize a variable

Squash
warning: ‘mxysize’ may be used uninitialized in this function [-Wmaybe-uninitialized]

Merge branch 'smattr/3404C6B4-EB48-44F0-8CD6-983567F718EC' into 'main'

gvgen: remove unused unistd.h #include

See merge request graphviz/graphviz!2543

gvgen makeTreeGen: squash a 'time_t'→'unsigned' conversion warning

gvgen makeRandom: squash a 'time_t'→'unsigned' conversion warning

gvgen makeTree: remove use of 'unsigned int' type mixing

Everything else in this function including its callers and callees uses `int`.
Using `unsigned int` within it was leading to MSVC C4018 warnings.

gvgen: remove unused unistd.h #include

Gitlab: #2204

Merge branch 'smattr/E49AF984-3CD6-407B-A6F9-EB8A1DF600FB' into 'main'

CMake: enable DevIL plugin

See merge request graphviz/graphviz!2549

Merge branch 'smattr/0C611C83-0C0E-441A-8341-2736681C3F5F' into 'main'

remove build system support for Lefty

See merge request graphviz/graphviz!2542

CMake: enable DevIL plugin

Gitlab: #1836

remove build system support for 'lefty'

04a78cc9ae01b2a7e5d5435146cd503c45af90b9 disabled Lefty in the Autotools build
system. The only fall out from this seems to have been Gentoo discovering
mistakes in the Graphviz Autotools files.¹ With mild evidence that there is no
one downstream relying on Lefty, we take the next step in its removal. If
another three months pass with no issues, we can remove Lefty from the Graphviz
tree altogether.

CC: David Seifert <soap@gentoo.org>
Gitlab: related to #1836, #1854

¹ https://gitlab.com/graphviz/graphviz/-/merge_requests/2520

remove build system support for 'dotty'

`dotty` is built on top of `lefty` which is deprecated and will be removed in a
future commit.

remove build system support for 'lneato'

`lneato` is built on top of `lefty` which is deprecated and will be removed in a
future commit.

Merge branch 'smattr/B664DAE8-8F55-42C9-A090-8D658C3F6777' into 'main'

WebP plugin: squash warnings and enable in the CMake build system

See merge request graphviz/graphviz!2546

CMake: enable WebP plugin

Gitlab: #1836

WebP plugin: remove unnecessary parens

webp_loadimage_cairo: squash -Wunused-parameter warning

This function is used as a callback, so needs to have this type signature.

webp_really_loadimage: correct type for 'data_size'

Squashes a -Wconversion warning.

webp_format: squash -Wsign-conversion warnings

WebP plugin: remove unnecessary casts

Merge branch 'smattr/495CECF8-863F-4028-A599-2376050D52EA' into 'main'

upgrade Pylint

See merge request graphviz/graphviz!2545

upgrade Pylint

Merge branch 'smattr/A69045D6-DB2A-400C-AAC4-6E626F818E3E' into 'main'

CI: remove Ubuntu 21.04

See merge request graphviz/graphviz!2544

CI: remove Ubuntu 21.04

Ubuntu 21.04 was EOLed on 2022-01-20.¹

¹ https://wiki.ubuntu.com/Releases

Merge branch 'smattr/0F04F849-05F9-46B6-A6B8-B4DB5F4DBF82' into 'main'

remove 'WIN32_LEAN_AND_MEAN' manipulation

See merge request graphviz/graphviz!2538

remove 'WIN32_LEAN_AND_MEAN' manipulation

This macro is a legacy toggle to suppress parts of the Windows API surface in
order to accelerate compilation. In a parallel compilation of a code base the
size of Graphviz, it has a negligible effect. As always, Raymond Chen gives the
back story.¹ This commit removes these tricks to ease future maintenance.

¹ https://devblogs.microsoft.com/oldnewthing/20091130-00/?p=15863

Merge branch 'smattr/469B07AD-AF55-4239-AFAD-9601CEFA4059' into 'main'

CMake: enable Smyrna on Linux

See merge request graphviz/graphviz!2540

CI: enable Smyrna on Ubuntu CMake jobs

CMake: enable 'smyrna'

Gitlab: #1836, #1854

tests: strip 'XType' warnings in '-?' usage test

On macOS, Smyrna prints some warnings prior to the usage text:

  2022-03-26 19:32:21.216 smyrna[70079:124370] XType: com.apple.fonts is not
    accessible.
  2022-03-26 19:32:21.216 smyrna[70079:124370] XType: XTFontStaticRegistry is
    enabled.
  Usage: smyrna [-v?] <file>

Gitlab: related to #1836, #1854

tests: when squashing macOS 'XType' warnings, also swallow trailing '\n'

Enabling Smyrna on macOS fails the test `test_tools` because running `smyrna -?`
outputs some XType warnings prior to the usage text. Suppressing these with
`remove_xtype_warnings` still does not fix the issue because the seen text now
begins “\n\nusage:…”. This change makes `remove_xtype_warnings` act more
precisely and strip the end of line character that is part of the warning as
well.

Gitlab: related to #1836, #1854

tests: move 'XType' stripping into a library function

We will need to use this in another test case in an upcoming commit.

lib/glcomp: silence deprecation warnings

Warnings about OpenGL function usage fail the CMake build when Smyrna is enabled
on macOS.

Gitlab: related to #1836, #1854

add an OpenGL abstraction header

On macOS, the OpenGL implementation is provided by the XCode SDK under the
“OpenGL/” prefix instead of “GL/” as on other operating systems. Note that
glut.h is still provided by the Freeglut package under “GL/”.

Gitlab: related to #1836, #1854

CI: replicate Qt tweak explanation from one macOS file into another

Makes it slightly less cryptic to the reader why these steps are being done.

CMake: make all glcomp checks required if 'with_smyrna=ON'

Given there is a build system flag `with_smyrna` that defaults to `OFF`, it
seems to make more sense to unconditionally attempt to build glcomp when this
flag is on. The current silent disabling behavior when dependencies are missing
results in a confusing link failure when glcomp cannot be found. Following this
change, this is instead a configuration failure pointing to the missing
dependency(ies).

Merge branch 'smattr/A63E118C-81BA-4D68-8659-80C2FE6883AE' into 'main'

move installation test into CI-only tests

See merge request graphviz/graphviz!2528

move installation test into CI-only tests

This test was introduced in !1347¹ in response to some version confusion due to
the Graphviz version being generated in several different places. Locally, this
test could fail misleadingly if the developer had new commits in their Graphviz
checkout and had not recompiled. But these failures are not relevant; running
the test suite against a slightly different version of Graphviz should not
unconditionally fail one of the tests. Moreover a developer working locally is
generally only running a single work flow, so their local Graphviz version is
generated in a single place.

The place where we do worry about version divergence is in CI, where the steps
for generating the Graphviz version for different platforms are in different
places and can accidentally skew. So this change makes the test only run in CI,
not locally.

Note that we can also drop the conditional logic to recover the version from the
GRAPHVIZ_VERSION file because the `$GV_VERSION` environment variable is defined
in all the CI testing jobs.

¹ https://gitlab.com/graphviz/graphviz/-/merge_requests/1347

Merge branch 'smattr/6E28F98D-AFCD-4FD0-8D05-340CC2F883A7' into 'main'

CMake: some minor clean up

See merge request graphviz/graphviz!2534

CMake: replace a hard-coded libm reference with '$MATH_LIB'

CMake: use 'SYSTEM' for third-party include directories

This tells CMake to use `-isystem` for these directories instead of `-I`,
suppressing any compiler warnings in third-party code.

CMake: fix indentation in lib/mingle’s CMakeLists.txt

lneato on Windows: remove unused 'panic' function

CMake: remove generator expressions in comments

Generator expressions¹ within strings are not expanded. So these lines were
leading to confusing build output indicating the literal text of the generator
expression was being symlinked:

  [ 79%] Linking C executable dot
  Linking $<TARGET_FILE_NAME:dot> as circo
  Linking $<TARGET_FILE_NAME:dot> as fdp
  Linking $<TARGET_FILE_NAME:dot> as neato
  Linking $<TARGET_FILE_NAME:dot> as osage
  Linking $<TARGET_FILE_NAME:dot> as patchwork
  Linking $<TARGET_FILE_NAME:dot> as sfdp
  Linking $<TARGET_FILE_NAME:dot> as twopi
  [ 79%] Built target dot

¹ https://cmake.org/cmake/help/latest/manual/cmake-generator-expressions.7.html

Merge branch 'smattr/AEEEBED2-E88B-4F17-BA22-76A206A83DAD' into 'main'

gdiplus plugin: remove MS Visual Studio 8 patches

See merge request graphviz/graphviz!2539

gdiplus plugin: remove MS Visual Studio 8 patches

These patches appear to have been working around some non-portable code in the
headers shipped with Microsoft Visual Studio 8 (a.k.a. Microsoft Visual Studio
2005). This version of Visual Studio was EOLed on 2016-04-12. Neither the CMake
nor MS Build build systems (the only CI environments for which compilation of
this plugin is enabled) were using these patches. So apparently they have not
been useful for a long time.

Merge branch 'smattr/17FC7676-22B2-48E6-9D6F-0BC12096581A' into 'main'

remove unused Smyrna icons

See merge request graphviz/graphviz!2537

share/gui: remove duplicate 'zoom.png'

share/gui: remove duplicate 'move.png'

remove 'rect_select*.png' Smyrna icons

These appear to have never been used.

remove 'preview.png' Smyrna icon

This appears to have never been used.

share/gui: remove duplicate pan.png

remove 'node.png' Smyrna icon

This seems to have never been used.

remove 'fit.png' Smyrna icon

The last use of this was removed in 68ba63a7c72e61bc0b2f12db9041f4dde4ab47cf.

remove 'fish.PNG' Smyrna icon

This seems to have never been used.

remove 'edge.png' Smyrna icon

This seems to have never been used.

remove 'ltwopi.png' Smyrna icon

The last use of this was removed in 89ba8c1507b84cef5bc892322ee767ed048101f4.

remove 'lneato.png' Smyrna icon

The last use of this was removed in 89ba8c1507b84cef5bc892322ee767ed048101f4.

remove 'lfdp.png' Smyrna icon

The last use of this was removed in 89ba8c1507b84cef5bc892322ee767ed048101f4.

remove 'ldot.png' Smyrna icon

The last use of this was removed in 89ba8c1507b84cef5bc892322ee767ed048101f4.

remove 'lcirco.png' Smyrna icon

The last use of this was removed in 89ba8c1507b84cef5bc892322ee767ed048101f4.

remove 'glade-3.png' Smyrna icon

This seems to have never been used.

remove 'dot.PNG' Smyrna icon

This seems to have never been used.

remove 'rotate.*' Smyrna icons

The last use of these was removed in f565e185203d287fd659af1f11d83bcf44c351f3.

remove 'zoomplus.raw' Smyrna icon

The last use of this was removed in 9a9cfd4e90775aa14a11a0db6745783f3a59ef56.

remove 'zoomminus.raw' Smyrna icon

The last use of this was removed in 9a9cfd4e90775aa14a11a0db6745783f3a59ef56.

remove 'ok*.raw' Smyrna icons

These seem to have never been used.

remove 'select.png' Smyrna icon

The last use of this was removed in 1c6ea45d410c76f6b98dc9d33c7f91a6e02f7e7e.

remove 'newzoom.png' Smyrna icon

The last use of this was removed in 1c6ea45d410c76f6b98dc9d33c7f91a6e02f7e7e.

remove 'new_node.png' Smyrna icon

The last use of this was removed in 1c6ea45d410c76f6b98dc9d33c7f91a6e02f7e7e.

remove 'magnifier_zoom_in.png' Smyrna icon

The last use of this was removed in 1c6ea45d410c76f6b98dc9d33c7f91a6e02f7e7e.

remove 'magnifier.png' Smyrna icon

The last use of this was removed in f565e185203d287fd659af1f11d83bcf44c351f3.

remove 'magifier_zoom_out.png' Smyrna icon

The last use of this was removed in 1a65f4b82be8bb588edf6895e4712012f1cbe17a.

remove 'hourglass.png' Smyrna icon

The last use of this was removed in 1a65f4b82be8bb588edf6895e4712012f1cbe17a.

remove 'find_node.png' Smyrna icon

The last use of this was removed in 1c6ea45d410c76f6b98dc9d33c7f91a6e02f7e7e.

remove 'find_edge.png' Smyrna icon

The last use of this was removed in 1c6ea45d410c76f6b98dc9d33c7f91a6e02f7e7e.

remove 'delete_node.png' Smyrna icon

The last use of this was removed in 1c6ea45d410c76f6b98dc9d33c7f91a6e02f7e7e.

remove 'deletE_edge.png' Smyrna icon

The last use of this was removed in 1c6ea45d410c76f6b98dc9d33c7f91a6e02f7e7e.

remove 'AntiselectRect' Smyrna icons

The last use of these was removed in 1c6ea45d410c76f6b98dc9d33c7f91a6e02f7e7e.

remove 'add_edge.png' Smyrna icon

The last use of this was removed in 1c6ea45d410c76f6b98dc9d33c7f91a6e02f7e7e.

Merge branch 'smattr/5B54693E-4A14-4B13-8A39-B13568AEE42D' into 'main'

CMake: enable 'diffimg'

Closes #1788

See merge request graphviz/graphviz!2535

CMake: enable 'diffimg'

Note that the rtest/rtest.py work around is still not fully removed because
libgd is not currently available in the 64-bit Windows CI environment.

Gitlab: closes #1788

Merge branch 'smattr/0E5378C5-27A2-4E7A-A4A8-8BC527899AF6' into 'main'

smyrna: clean up and warning squashing

See merge request graphviz/graphviz!2533

cmd/smyrna: remove unused return value from 'close_graph'

cmd/smyrna/viewport.c: fix -Wnon-literal-null-conversion with Clang

This was causing the CMake build (to be enabled in a future commit) on macOS to
fail.

Gitlab: #1836, #1854

cmd/smyrna: fix a comment typo

cmd/smyrna: fix -Wnon-literal-null-conversion with Clang

This was causing the CMake build (to be enabled in a future commit) on macOS to
fail.

Gitlab: #1836, #1854

cmd/smyrna: remove no-op if statement

When enabling Smyrna in the CMake build system, compilation fails with a
-Wempty-body warning about this line. I am guessing that it was intended to not
be a conditional here.

Gitlab: #1836, #1854

fix: teach Smyrna '-?' on macOS and Windows

This is the equivalent of cd80b6262c7e1b95307756f13c89a9f6b2900520 and friends
but for Smyrna. It was not previously detected that this problem also affects
Smyrna because it is not currently built on macOS or Windows in CI. This will be
improved in future.

Gitlab: #1852

lib/glcomp: fix -Wnon-literal-null-conversion with Clang

This was causing the CMake build (to be enabled in a future commit) on macOS to
fail.

Gitlab: #1836, #1854

lib/glcomp: remove unused 'glCompButton.customptr'

With Clang on macOS, this causes compiler warnings due to initializing a pointer
with `'\0'`. This fails the CMake build which will be enabled in a future
commit. We could rephrase this, but it is simpler to just remove this unused
member.

Gitlab: #1836, #1854

smyrna: remove unnecessary 'gpointer' cast

`gpointer` is just a typedef for `void*` that implicitly coerces to all other
pointer types.

smyrma: remove unnecessary 'gint' cast

`G_PRIORITY_DEFAULT` is already a `gint` (typedef for `int`).

smyrna: remove use of 'guint' cast

`guint` is just a Glib typedef for `unsigned int`. So we can write this simpler.

Merge branch 'smattr/1D9B0E32-37CA-4547-A966-AE877344C814' into 'main'

tests: remove unused cdiff.c

See merge request graphviz/graphviz!2532

tests: remove unused cdiff.c

Merge branch 'smattr/63D62C3D-D094-4ED8-A675-538D7283B9E6' into 'main'

use suffixed literals instead of casts to construct 'GLfloat' values

See merge request graphviz/graphviz!2530