GDK plugin gdk_format: squash -Wswitch-default warning

The device ID can only be one of the values configured at the bottom of this
file, so this switch is exhaustive.

GDK plugin writer: squash -Wunused-parameter warning

This function is used as a callback so the `error` parameter cannot be removed.

GDK plugin writer: remove an unnecessary cast

GDK plugin writer: abbreviate

GDK plugin argb2rgba: squash -Wconversion warnings

remove unused 'HAVE_GDK_PIXBUF'

remove unused 'HAVE_GDK'

CMake: enable GDK plugin

Gitlab: #1836

Merge branch 'pinc' into 'main'

Remove unused constant PINC

See merge request graphviz/graphviz!2568

Merge branch 'remove-dots' into 'main'

Remove now-unreferenced html{n} graphs.

See merge request graphviz/graphviz!2567

Merge branch 'smattr/D2614CA4-AE24-4715-86E4-D53125CA77C4' into 'main'

more #2204 progress

See merge request graphviz/graphviz!2557

common: remove unused unistd.h #include

gvpr: remove unnecessary casts

gvc: remove a use of the deprecated 'bcopy'

As the man page for `bcopy` notes:

  bcopy() was deprecated in IEEE Std 1003.1-2001 (“POSIX.1”) and removed in
  IEEE Std 1003.1-2008 (“POSIX.1”).

gvc: remove unused unistd.h #include

Gitlab: #2204

tred: remove unused unistd.h #include

Gitlab: #2204

Merge branch 'smattr/5292DDA9-FA03-48DB-A0BB-51A3A04D698E' into 'main'

CMake: add Glitz plugin

See merge request graphviz/graphviz!2558

remove unused 'HAVE_GLITZ'

CMake: add Glitz plugin

Gitlab: #1836

Merge branch 'unused-result' into 'main'

squash warn_unused_result

See merge request graphviz/graphviz!2556

squash warning in mm_read_unsymmetric_sparse

warning: ignoring return value of ‘fscanf’, declared with attribute warn_unused_result [-Wunused-result]

On  success,  these functions return the number of input items
successfully matched and assigned; this can be fewer  than
provided  for,  or even zero, in the event of an early
matching failure.

squash warning in SparseMatrix_import_matrix_market/MATRIX_TYPE_COMPLEX

warning: ignoring return value of ‘fscanf’, declared with attribute warn_unused_result [-Wunused-result]

On  success,  these functions return the number of input items
successfully matched and assigned; this can be fewer  than
provided  for,  or even zero, in the event of an early
matching failure.

squash warning in SparseMatrix_import_matrix_market/MATRIX_TYPE_PATTERN

warning: ignoring return value of ‘fscanf’, declared with attribute warn_unused_result [-Wunused-result]

On  success,  these functions return the number of input items
successfully matched and assigned; this can be fewer  than
provided  for,  or even zero, in the event of an early
matching failure.

squash warning in SparseMatrix_import_matrix_market/MATRIX_TYPE_INTEGER

warning: ignoring return value of ‘fscanf’, declared with attribute warn_unused_result [-Wunused-result]

On  success,  these functions return the number of input items
successfully matched and assigned; this can be fewer  than
provided  for,  or even zero, in the event of an early
matching failure.

squash warning in SparseMatrix_import_matrix_market/MATRIX_TYPE_REAL

warning: ignoring return value of ‘fscanf’, declared with attribute warn_unused_result [-Wunused-result]

On  success,  these functions return the number of input items
successfully matched and assigned; this can be fewer  than
provided  for,  or even zero, in the event of an early
matching failure.

Merge branch 'smattr/3500F667-71BC-42B8-990A-AEE64BF09390' into 'main'

CMake: enable Ghostscript plugin

See merge request graphviz/graphviz!2559

Ghostscript plugin gvloadimage_gs_cairo: squash -Wunused-parameter warning

This parameter cannot be removed because this function is used as a callback.

Ghostscript plugin gvloadimage_gs_load: squash -Wsign-compare warning

Ghostscript plugin gs_writer: squash -Wsign-conversion warnings

Ghostscript plugin: remove unnecessary casts

CMake: enable Ghostscript plugin

Gitlab: #1836

Remove unused constant PINC

All usages seem to be removed.

Remove now-unreferenced html{n} graphs.

They were moved to graphviz.gitlab.io a while ago, and we don't need two
copies of them.

Now live here:
https://gitlab.com/graphviz/graphviz.gitlab.io/-/tree/main/static/doc/info

Merge branch 'smattr/6C4DD5DD-DAE3-460A-BF2F-D4A9DC6EC0E6' into 'main'

Windows: remove some unused config items

See merge request graphviz/graphviz!2552

Windows: remove unused 'gid_t' typedef

Windows: remove unused 'X_DISPLAY_MISSING' #define

Windows: remove unused 'PATHSEPARATOR' #define

Merge branch 'smattr/FFDE9B74-2C34-4E11-AE12-C0888E544B11' into 'main'

some CMake tidying up

See merge request graphviz/graphviz!2555

CMake: fix missing SO version and MinGW work around in WebP plugin

This should have been included in 645817efcb9926bdd42f3616eb52e8ea5c199283.

CMake: standardize on always specifying library paths in 'install' steps

Given the differences in how Windows vs everything else treats library vs binary
paths, it seems clearer and less confusing to always specify `RUNTIME`,
`LIBRARY`, and `ARCHIVE` paths during `install` steps. This also avoids
accidentally installing libraries to incorrect paths if they are considered for
installation as dependents of a binary, though I do not know whether this
behavior ever occurs.

CMake: remove 'TOP_BINARY_DIR'

This can be located through standard CMake mechanisms.

CMake: fix a missing dependency in common

CMake: remove 'TOP_SOURCE_DIR'

This can be located through standard CMake mechanisms.

CMake: remove 'GRAPHVIZ_LIB_DIR'

`target_include_directories` supports relative paths that are interpreted
relative to the current source directory. So we can write these paths shorter
and more obviously.

Merge branch 'smattr/91C61E37-F5BB-43AD-80B1-48993359F44D' into 'main'

GTK plugin: remove unused code and #includes

See merge request graphviz/graphviz!2554

GTK plugin: remove unnecessary casts

GTK plugin: remove unused #includes

Gitlab: #2204

GTK plugin: remove unused 'add_pixmap_directory'

GTK plugin: remove unused 'create_pixmap'

GTK plugin: remove unused 'create_pixbuf'

GTK plugin: remove unused 'glade_set_atk_action_description'

Merge branch 'smattr/987BF4EC-2F10-420E-A41B-9C7319942F63' into 'main'

acyclic: remove unused unistd.h #include

See merge request graphviz/graphviz!2551

acyclic: squash a -Wswitch-default warning

acyclic: remove unnecessary parens

acyclic: remove unused unistd.h #include

Gitlab: #2204

Merge branch 'smattr/7D15E021-6096-4341-872C-AB744E8E5C19' into 'main'

gml2gv: remove unused unistd.h #include

See merge request graphviz/graphviz!2553

gml2gv: squash a -Wswitch-default warning

gml2gv: remove unused unistd.h #include

Gitlab: #2204

Merge branch 'smattr/FC3254B7-1B42-48E9-AB7D-9297029D5E2A' into 'main'

misc cleanup

See merge request graphviz/graphviz!2541

remove 'SMYRNA' #define

This seems unused.

CMake: consolidate Python checks

This essentially removes the conditional check, when an unconditional check was
already being performed later.

standardize some CMake spacing

GDI+ plugin: fix: swap duplicated 'gvwrite' prototypes for a header include

Note that these prototypes were wrong. `gvwrite` takes a `char` pointer not
`unsigned char` pointer, and a `size_t` not `unsigned int` length.

abbreviate some trivial file reads and writes in Python

Merge branch 'smattr/FE2C3BFC-BD30-47F6-A514-1367DC81F480' into 'main'

tests: suppress Boost deprecation compiler warnings

See merge request graphviz/graphviz!2550

tests: suppress Boost deprecation compiler warnings

The third-party library Svgpp calls Boost in a way that is deprecated, resulting
in compiler warnings during the CMake build. This change teaches CMake that none
of this is our code and we do not want to be warned about it.

Merge branch 'smattr/70525209-2ACA-491A-8B14-E34C24BB7949' into 'main'

convert fc-fix.cpp from UTF-16 to UTF-8

See merge request graphviz/graphviz!2548

clang-format fc-fix.cpp

Given the prior commit rewrote every byte in this file, we may as well also fix
the odd white space decisions here while we are at it. Though it is unclear who
if anyone uses this tool any more.

convert fc-fix.cpp from UTF-16 to UTF-8

Of all the source files in the Graphviz tree, this file alone was encoded in
UTF-16 with a BOM.¹ BOMs have mostly fallen out of favor these days, with people
preferring to let the host operating system or locale determine encoding. Git
will happily translate text files to your local encoding on checkout. With that
in mind, we can convert this file to UTF-8 and stop forcing developers on other
operating systems to pay the price for Windows’ poor past decisions.

¹ https://en.wikipedia.org/wiki/Byte_order_mark

Merge branch 'smattr/7C359EA7-F1EE-4BAD-A23C-32490FAF859B' into 'main'

Xlib plugin, vimdot: Squash all warnings and enable in the CMake build system

See merge request graphviz/graphviz!2547

CMake: enable xlib plugin and 'vimdot'

Gitlab: #1836

xlib plugin xlib_initialize: squash a -Wsign-conversion warning

xlib plugin init_window: squash a -Wsign-conversion warning

xlib plugin handle_file_events: squash a -Wsign-conversion warning

xlib plugin handle_file_events: rephrase a switch into an if

Avoids a -Wswitch-default warnings and simplifies the code that was clearly
indicating an intent to ignore everything except `IN_MODIFY`.

xlib plugin handle_file_events: fix some misuse of types

xlib plugin handle_file_events: more tightly scope a long-lived dynamic buffer

This buffer was being retained in a long-lived `static` pointer, increasing peak
memory usage and making it harder to use tools like Valgrind and ASan with
Graphviz. This change makes it more obvious to both users and tools what this
buffer is used for.

Note this also tweaks an error message to be more informative.

xlib plugin handle_file_events: remove unnecessary cast and '&'

The `name` member of the `inotify_event` struct is a character array.

xlib plugin handle_stdin_events: remove unused 'stdin_fd' parameter

xlib plugin init_window: squash -Wsign-conversion warning for window depth

xlib plugin init_window: squash -Wsign-conversion warnings for job

xlib plugin update_display: squash -Wsign-conversion warning for window depth

xlib plugin update_display: squash -Wsign-conversion warnings for job

xlib plugin handle_xlib_events: squash -Wswitch-default warning

This switch is deliberately only handling the events of relevance to it.

xlib plugin handle_xlib_events: squash -Wsign-conversion warnings

The Xlib API¹ claims to only return buttons 1-5.

¹ See the table in
  https://www.x.org/releases/X11R7.7/doc/libX11/libX11/libX11.html#Keyboard_and_Pointer_Events_b

xlib plugin handle_expose: squash -Wsign-conversion warnings

CMake: fix fcntl.h detection

This seems to have gone unnoticed because none of the Graphviz components
currently integrated into the CMake build system rely critically on fcntl.h.
This issue was discovered when integrating the xlib plugin.

CMake: drop some conditional pkgconf logic in glcomp

pkgconf has already been unconditionally required prior to these lines.

CMake: standardize on lowercase for functions

add Gentoo and David Seifert to downstream packaging list

Merge branch 'fix-uninitialized-variables' into 'main'

fix some uninitialized variables

See merge request graphviz/graphviz!2536

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.