Quartz plugin quartz_textlayout: squash -Wunused-parameter warning

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

Quartz plugin quartzgen_bezier: squash -Wunused-parameter warnings

These parameters cannot be removed because this function is used as a callback.

Quartz plugin quartzgen_begin_page: remove open coded 'MAP_FAILED'

Later on this code goes on to do an unchecked `mmap`, so it is unclear exactly
what the author believed the failure semantics of `mmap` are. Nevertheless, this
seems like a slight improvement.

Quartz plugin: mark 'quartz_textlayout' static

This function is not used outside of its containing file.

Quartz plugin: mark 'quartzgen_textspan' static

This function is not used outside of its containing file.

Quartz plugin quartzgen_begin_anchor: squash -Wunused-parameter warnings

These parameters cannot be removed because this function is used as a callback.

Quartz plugin: squash a -Wsign-compare warning

Quartz plugin: remove unnecessary casts

Quartz plugin: squash -Wundef warnings

Quartz plugin quartz_loadimage_quartz: squash -Wunused-parameter warning

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

Quartz plugin quartz_format: squash -Wsign-conversion warning

We know `device.id` is always a valid `format_type` because its only possible
values are defined within this plugin.

Quartz plugin: mark 'memory_data_consumer_get_byte_pointer' static

This function is not used outside of its containing file.

CMake: enable Quartz plugin

It is unclear how valuable this is, given this is only applicable to macOS and
its dependency discovery (based on the Autotools behavior) is fairly ad hoc. But
we still drag it into the CMake world to make progress on unifying the Graphviz
build system.

Gitlab: #1836

Merge branch 'smattr/A76659AC-7B7E-4C6C-803F-23D7EDCA5C11' into 'main'

CMake: enable GTK plugin

See merge request graphviz/graphviz!2561

CMake: enable GTK plugin

Gitlab: #1836

GTK plugin create_window1: remove unused 'tmp_toolbar_icon_size'

Merge branch 'smattr/AD90CABB-D995-48F5-84F6-FAD9B13D0FB8' into 'main'

remove some more unused unistd.h #includes

See merge request graphviz/graphviz!2563

nop: remove unused unistd.h #include

Gitlab: #2204

gv2gml: remove unused unistd.h #include

Gitlab: #2204

ccomps: remove unused unistd.h #include

Gitlab: #2204

Merge branch 'smattr/5A077557-8681-4EB9-BAD2-0D329EBA379B' into 'main'

CMake: enable Poppler plugin and squash compiler warnings

See merge request graphviz/graphviz!2564

Poppler plugin gvloadimage_poppler_load: fix: match Glib allocation and free

The variable `absolute` is allocated using Glib’s `g_strdup` and friends.
Quoting the Glib docs:¹

  It's important to match `g_malloc()` (and wrappers such as `g_new()`) with
  `g_free()`, `g_slice_alloc()` (and wrappers such as `g_slice_new()`) with
  `g_slice_free()`, plain `malloc()` with `free()`, and (if you're using C++)
  `new` with `delete` and `new[]` with `delete[]`. Otherwise bad things can
  happen, since these allocators may use different memory pools (and new/delete
  call constructors and destructors).

So a custom allocation scheme or arena can be in play. Basically if you
`g_strdup` and then pair this with `free` (as was done in the code prior to this
commit), you risk leaking memory from the Glib pool and corrupting your system
allocator.

Having said that, this is no longer a concern in newer Glib:

  Since GLib 2.46 `g_malloc()` is hardcoded to always use the system malloc
  implementation.

Still, why tempt fate?

¹ https://developer-old.gnome.org/glib/stable/glib-Memory-Allocation.html

Poppler plugin gvloadimage_poppler_load: remove open coded 'NULL'

Poppler plugin gvloadimage_poppler_cairo: remove unnecessary cast

Poppler plugin gvloadimage_poppler_cairo: squash -Wunused-parameter warning

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

CMake: enable Poppler plugin

Gitlab: #1836

Merge branch 'smattr/3FCAE05F-BC51-4094-AEE1-F3CD1E65AE93' into 'main'

WebP plugin: fix use of raw '%zu'

See merge request graphviz/graphviz!2573

WebP plugin: fix use of raw '%zu'

03a5c3621e3185e4ca116805317a98cdc8595443 introduced another usage of the format
specifier `%zu` which has some compatibility issues on Windows. This change
switches to what it should have done to begin with, used the compatibility shim.

Merge branch 'smattr/5CA08F5D-25F7-4765-8FF7-76F300F4DFD9' into 'main'

CMake: enable Rsvg plugin

See merge request graphviz/graphviz!2565

CMake: enable Rsvg plugin

Gitlab: #1836

Merge branch 'smattr/73694127-C889-4E10-B8C2-D161D5177777' into 'main'

CMake: enable GDK plugin

See merge request graphviz/graphviz!2560

GDK plugin gdk_loadimage_cairo: squash -Wunused-parameter warning

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

GDK plugin gdk_format: squash -Wsign-conversion warnings

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