Merge branch 'smattr/fa555883-42dd-4da0-9444-24d3ba18dad0' into 'main'

fix inability to include and use gvc.h from a parent CMake project

Closes #2109 and #1477

See merge request graphviz/graphviz!2164

fix inability to include and use gvc.h from a parent CMake project

When attempting to declare a dependency on gvc when using Graphviz as a CMake
subproject, the include path necessary to compile gvc headers would not be
propagated to dependent targets. The result would be a compilation error when
the transitive includes of gvc.h could not be found.

To validate this change, create an empty directory and add the following
CMakeLists.txt:

  project(foo)
  add_subdirectory(graphviz)
  add_executable(bar main.c)
  target_link_libraries(bar gvc)

Add the following main.c:

  #include <gvc/gvc.h>
  int main(void) { return 0; }

Clone Graphviz into the subdirectory 'graphviz'. Then the standard CMake steps:

  mkdir build
  cd build
  cmake ..
  make

Note that Graphviz dependencies and then 'bar' are correctly built.

I am not a CMake expert; this was written based on my best guess of what
Graphviz should be doing according to the CMake docs. Assuming this is correct,
other Graphviz targets probably need similar fixes.

Fixes #1477, #2109.

remove libast from gvc’s include path in CMake

This library does not seem to need libast in its include path to compile. This
is consistent with lib/gvc/Makefile.am.

Merge branch 'smattr/650c059b-2ccb-4c4b-9a03-6ded54e32325' into 'main'

more warning squashing

See merge request graphviz/graphviz!2168

fixLabelOrder: use a C99 bool for 'haveBackedge' local instead of an int

use a C99 bool to pass around 'reverse' instead of an int

use a C99 bool to pass around 'hasfixed' instead of a boolean

reorder: convert local booleans to C99 bools

ReMincross: change from boolean to C99 bool

lib/sparse: remove some unnecessary casts

lib/sparse: replace header guards with more modern #pragma once

IntStack: remove unnecessary bracketing

IntStack: track size and capacity as 'size_t' values

Squashes two -Wsign-compare warnings and generally makes this code more
portable.

IntStack_print: remove incorrect comment

This seems to have been incorrectly copied from IntStack_pop.

IntStack: remove commented out code

Merge branch 'smattr/df178dd0-42ca-429b-aa4b-c7927d2956ee' into 'main'

cmd/ warning squashing

See merge request graphviz/graphviz!2169

gxl2gv gxl_to_gv: match XML_Parse, squashing a -Wconversion warning

gxl2gv characterDataHandler: squash -Wsign-conversion warnings

gxl2gv endElementHandler: remove shadowing of 'name'

gxl2gv endElementHandler: use 'size_t' to track buffer size

Squashes -Wconversion warnings.

gxl2gv setGlobalEdgeAttr: remove unused 'ud' parameter

gxl2gv setGlobalNodeAttr: remove unused 'ud' parameter

gxl2gv: squash -Wunused-parameter warnings

gxl2gv pushString: use 'size_t' to track buffer size

Squashes -Wconversion warnings.

gv2gxl initState: squash a -Wconversion warning

gv2gxl writeEdge: remove unused 'terminate' parameter

gv2gxl writeHdr: use 'size_t' to track buffer size

Squashes -Wconversion warnings.

gv2gxl createEdgeId: use 'size_t' to track buffer size

Squashes -Wconversion warnings.

gv2gxl _xml_string: use 'size_t' to track buffer size

Squashes -Wconversion warnings.

gv2gxl: squash -Wunused-parameter warnings

cvtgxl: squash a -Wswitch-default warning

colxlate canoncolor: operate on 'char' instead of 'unsigned char'

It is unclear to me why this code was using unsigned char variables. This was
unnecessary, induced noisy casting and caused build warnings. This also changes
the `orig` parameter to a const pointer as the function does not modify the
pointed to data.

gvcolor: squash a -Wswitch-default warning

gc wcp: remove shadowing of 'fname'

gc cntCluster: squash a -Wunused-parameter warning

gc: squash a -Wswitch-default warning

dot intr: squash a -Wunused-parameter warning

Merge branch 'smattr/10DCD620-10CF-4101-8025-7495C4DEE38C' into 'main'

lefty: remove cgraph include directory in MSBuild

See merge request graphviz/graphviz!2171

lefty: remove cgraph include directory in MSBuild

Lefty does not appear to depend on cgraph.

Merge branch 'some-autotools-cleanup' into 'main'

Some autotools linking cleanup

See merge request graphviz/graphviz!2175

cmd/edgepaint: autotools: link to the gvc lib instead of the static common lib

The common lib is part of gvc and should not be linked to directly.

This will allow an upcoming MR to correct the storage class attributes
in lib/common/globals.h.

Wihout this commit, many errors like this would appear when linking
the edegpaint executable:

/usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: ../../lib/edgepaint/.libs/libedgepaint_C.a(node_distinct_coloring.o): in function `node_distinct_coloring_internal2':
/home/magja/graphviz/lib/edgepaint/node_distinct_coloring.c:137: undefined reference to `__imp_Verbose'

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

cluster: autotools: link to the gvc lib instead of the static common lib

The common lib is part of gvc and should not be linked to directly.

This will allow an upcoming MR to correct the storage class attributes
in lib/common/globals.h.

Wihout this commit, many errors like this would appear when linking
cluster:

/usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: ../../lib/edgepaint/.libs/libedgepaint_C.a(node_distinct_coloring.o): in function `node_distinct_coloring_internal2':
/home/magja/graphviz/lib/edgepaint/node_distinct_coloring.c:137: undefined reference to `__imp_Verbose'

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

gvmap: autotools: link to the gvc lib instead of the static common lib

The common lib is part of gvc and should not be linked to directly.

This will allow an upcoming MR to correct the storage class attributes
in lib/common/globals.h.

Wihout this commit, many errors like this would appear when linking
gvmap:

/usr/lib/gcc/x86_64-pc-cygwin/10/../../../../x86_64-pc-cygwin/bin/ld: ../../lib/edgepaint/.libs/libedgepaint_C.a(node_distinct_coloring.o): in function `node_distinct_coloring_internal2':
/home/magja/graphviz/lib/edgepaint/node_distinct_coloring.c:137: undefined reference to `__imp_Verbose'

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

plugin/pango: autotools: add linking to cgraph

E.g. plugin/pango/gvgetfontlist_pango.c uses agxbputc from
lib/cgraph/agxbuf.c.

Fixes errors like this with Cygwin:

/usr/lib/gcc/x86_64-pc-cygwin/11/../../../../x86_64-pc-cygwin/bin/ld: .libs/gvgetfontlist_pango.o: in function `agxbputc':
/cygdrive/c/GitLab-Runner/builds/magjac/graphviz/graphviz-2.49.1~dev.20210911.2106/plugin/pango/../../lib/cgraph/agxbuf.h:93: undefined reference to `agxbmore'

plugin/visio: autotools: add linking to cgraph

E.g. VisioRender.cpp uses agroot from lib/cgraph/obj.c.

plugin/core: autotools: add linking to xdot

E.g. plugin/core/gvrender_core_dot.c uses parseXDot from
lib/xdot/xdot.c.

Fixes these errors when building with Cygwin from the portable source
produced by the Ubuntu 18.03 runner in CI:

/cygdrive/c/GitLab-Runner/builds/magjac/graphviz/graphviz-2.49.1~dev.20210915.2014/plugin/core/gvrender_core_json.c:313: undefined reference to `parseXDot'
/usr/lib/gcc/x86_64-pc-cygwin/11/../../../../x86_64-pc-cygwin/bin/ld: /cygdrive/c/GitLab-Runner/builds/magjac/graphviz/graphviz-2.49.1~dev.20210915.2014/plugin/core/gvrender_core_json.c:331: undefined reference to `freeXDot'

For some reason, this error did not show up when building using a
build system generated on Cygwin with autogen.sh.

lefty: autotools: don't try to build lefty when it's not possible

Fixes this error with Cygwin:

In file included from garray.c:14:
../../../../cmd/lefty/g.h:20:10: fatal error: X11/Intrinsic.h: No such file or directory
   20 | #include <X11/Intrinsic.h>
      |          ^~~~~~~~~~~~~~~~~

plugin/webp: autotools: add linking to gvc

E.g. plugin/webp/gvloadimage_webp.c used gvusershape_file_access from
lib/gvc/gvusershape.c.

Fixes the following errors with Cygwin:

gvloadimage_webp.o: in function `webp_loadimage':
plugin/webp/gvloadimage_webp.c:139: undefined reference to `__imp_gvusershape_file_access'
plugin/webp/gvloadimage_webp.c:153: undefined reference to `__imp_gvusershape_file_release'
plugin/webp/gvloadimage_webp.c:153: undefined reference to `__imp_gvusershape_file_release'
gvdevice_webp.o: in function `writer':
plugin/webp/gvdevice_webp.c:43: undefined reference to `gvwrite'

plugin/neato_layout: autotools: remove reference to nonexistent .def file

The gvplugin_neato_layout.def file has never existed in the git repo.

This reference was added in e74824c819a069dca2162c57deea96a7b3e5df1c,
but the .def file was never committed.

Fixes this error with MinGW:

libtool:   error: symbol file '../../plugin/neato_layout/gvplugin_neato_layout.def' does not exist

Merge branch 'smattr/508C5931-2894-41DE-9B0F-C8B9CE32549B' into 'main'

lefty clean up

See merge request graphviz/graphviz!2170

lefty SFclearList: abbreviate some float literals

lefty: remove no-op doLayout function

lefty: remove commented out code

lefty: remove some unnecessary casts

lefty SFstatAndCheck: use 'size_t' variables when dealing with buffer sizes

Squashes two -Wconversion warnings and improves portability.

lefty SFgetDir: standardize 'qsort' usage

The `qsort` function has been ISO standardized as far back as C89. There is no
need for workaround on any toolchain that correctly implements C99.

lefty SFcompareEntries: replace case-insensitive alternative with 'strcasecmp'

This functionality is built in to libc these days.

lefty SFcompareEntries: remove some unnecessary casts

Merge branch 'smattr/0232394D-4EE9-4B97-959C-70C6A25DB850' into 'main'

more -Wmissing-field-initializer warning squashing

See merge request graphviz/graphviz!2173

p_insulator: squash a -Wmissing-field-initializer warning

p_utr: squash a -Wmissing-field-initializer warning

p_terminator: squash a -Wmissing-field-initializer warning

p_cds: squash a -Wmissing-field-initializer warning

p_promoter: squash a -Wmissing-field-initializer warning

p_star: squash a -Wmissing-field-initializer warning

p_Mcircle: squash a -Wmissing-field-initializer warning

p_Msquare: squash a -Wmissing-field-initializer warning

p_diamond: squash a -Wmissing-field-initializer warning

p_tripleoctagon: squash a -Wmissing-field-initializer warning

Merge branch 'smattr/51F91849-AB76-4338-A2A6-A23F67BF0727' into 'main'

more lefty cleanup

See merge request graphviz/graphviz!2172

lefty: remove unused 'PTR2INT' and 'INT2PTR' macros

lefty SFcompareLogins: remove some unnecessary casts

lefty SFstrncmp: replace with strncasecmp

This functionality is built in to libc these days.

lefty SFgetHomeDirs: standardize 'qsort' usage

The `qsort` function has been ISO standardized as far back as C89. There is no
need for workaround on any toolchain that correctly implements C99.

lefty: remove unnecessary getwd prototype

The only use of this function was removed in
a4468a2a875dadaee313fb2a7333881cee85fb91.

Merge branch 'smattr/0824fd6b-4a2f-49f4-99cc-ed8100eecba5' into 'main'

yet more warning squashing

See merge request graphviz/graphviz!2166

sparse_stress_subspace_majorization_kD: replace a boolean with a C99 bool

And reflow some text.

stress_majorization_kD_mkernel: replace a boolean with a C99 bool

And reflow some text.

dist_data: replace a boolean with a C99 bool

get_mq: abbreviate a malloc;memzero

The calloc function can do this more concisely and efficiently.

exisAssign: remove unnecessary bracketing

expr lexname: return a 'const char*'

The callers of this do not modify the returned value. This squashes a
-Wcast-qual warning.

expr global: squash -Wunused-parameter warning

expr global: remove unnecessary casts

expr print: use a 'size_t' for iteration

Squashes a -Wsign-compare warning.

expr scan: use a 'size_t' for iteration

Squashes a -Wsign-compare warning.

Merge branch 'smattr/f7c14ca2-789a-46b0-b5f1-2cacd08dfbc5' into 'main'

more warning squashing

See merge request graphviz/graphviz!2158

lu_decompose: remove unnecessary bracketing

lu_decompose: rephrase comparisons against 'biggest' to avoid equals

This squashes some -Wfloat-equal warnings, but has the same effect due to the
surrounding code ensuring `biggest` can only ever increase from 0.

lu_decompose: rephrase an open coded fmax

lu_decompose: remove micro-optimization of avoiding multiplication by 0

This is not on a hot path and 0 multiplications are not costly on modern
processors. This squashes a -Wfloat-equal warning.

find_ints: squash a -Wswitch-default warning

This switch on the result of a call to `gt` only ever results in -1, 1, or 0.
The 0 case was pretty clearly intended to be a no-op here.

neatogen gt: rephrase into something more obvious

This is intended to preserve the same semantics. It not only avoids two
-Wfloat-equal warnings, but makes this code so close to the leading comment that
the comment can simply be removed now.

neatogen gt: rephrase to avoid having to cast pointers to this function

neatogen between: rephrase to avoid -Wfloat-equal warnings

This is intended to have the same semantics.

find_ints: remove unused 'polygon_list' parameter

use a header with prototypes to squash bindings -Wmissing-prototypes warnings

This removes 30 build warnings and is generally a cleaner way of doing this.

Merge branch 'smattr/25F2CD31-80C6-48D6-A6EE-9D7EEEB2FA14' into 'main'

smyrna: remove unused materials.h

See merge request graphviz/graphviz!2160

smyrna: remove unused materials.h

Merge branch 'smattr/FDB29D45-8A65-4119-B80E-24C4F73985C5' into 'main'

more smyrna clean up

See merge request graphviz/graphviz!2161