common: don't use dllexport when GVDLL and _BLD_gvc are set

This is a temporary measure until an upcoming commit series cleans up
the control of the Windows storage-class attributes.

Without this change, an upcoming commit that defines GVDLL would cause
many warnings like this:

globals.obj : warning LNK4197: export 'Version' specified multiple times; using first specification [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]

The reason for this is that the symbols are already defined in the
gvc.def file. For more, see
https://support.microsoft.com/en-us/topic/you-receive-an-lnk4197-error-in-the-64-bit-version-of-the-visual-c-compiler-f002b07e-4ba6-46d4-eed4-0c82412b4ac6.

lib/gvpr: CMake: define EXPORT_GVPR

Without this change, an upcoming commit that defines GVDLL would cause
a warning like this:

C:\Users\magja\graphviz\lib\gvpr\gvpr.c(931,1): warning C4273: 'gvpr': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\gvpr\gvpr.vcxproj]

gvc: avoid macro redefinition warning when GVDLL is defined

Without this change, an upcoming commit that defines GVDLL would cause
warnings like this:

C:\Users\magja\graphviz\lib\gvc/gvplugin_loadimage.h(29,1): warning C4005: 'GVPLUGIN_LOADIMAGE_API': macro redefinition [C:\Users\magja\graphviz\build\plugin\gdiplus\gvplugin_gdiplus.vcxproj]

pack: CMake: set also _BLD_gvc

This is a temporary measure until an upcoming commit series cleans up
the control of the Windows storage-class attributes.

Without this change, an upcoming commit that defines GVDLL also for
CMake builds, would cause many errors like this with minGW:

pack/CMakeFiles/pack_obj.dir/ccomps.c.obj:ccomps.c:(.text+0x1c95): undefined reference to `__imp_Verbose'

gvc: remove multiple declarations of two functions from gvcproc.h

These are already declared in gvc.h with Windows storage-class
attributes.

Without this change, an upcoming commit that defines GVDLL also for
CMake builds, would cause the following errors with native Windows:

C:\Users\magja\graphviz\lib\gvc/gvc.h(67,13): error C2375: 'gvLayoutJobs': redefinition; different linkage [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]
C:\Users\magja\graphviz\lib\gvc/gvcproc.h(117): message : see declaration of 'gvLayoutJobs' [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]
C:\Users\magja\graphviz\lib\gvc/gvc.h(91,13): error C2375: 'gvFreeLayout': redefinition; different linkage [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]
C:\Users\magja\graphviz\lib\gvc/gvcproc.h(116): message : see declaration of 'gvFreeLayout' [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]

common: remove multiple declaration of attach_attrs from utils.h

This function is already declared in the public gvc.h.

Without this change, an upcoming commit that defines GVDLL also for
CMake builds, would cause the following error with native Windows:

C:\Users\magja\graphviz\lib\gvc/gvc.h(75,14): error C2375: 'attach_attrs': redefinition; different linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\utils.h(39): message : see declaration of 'attach_attrs' [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]

plugin/core: include gvc/gvc.h in gvrender_core_dot.c

E.g. attach_attrs used in gvrender_core_dot.c is declared in
lib/gvc/gvc.h.

Removes this warning (from using CMake with MinGW):

C:/Users/magja/graphviz/plugin/core/gvrender_core_dot.c:469:6: warning: implicit declaration of function 'attach_attrs' [-Wimplicit-function-declaration]

pack: include gvc/gvc.h in ptest.c

E.g. attach_attrs used in ptest.c is declared in lib/gvc/gvc.h.

This change is not strictly necessary since gvc/gvc.h is indirectly
included through common/render.h anyway, but a direct inclusion is
clearer and will avoid confusion when an upcoming commit removes a
duplicated declaration of attach_attrs from utils.h.

common: include gvc/gvc.h in output.c

E.g. attach_attrs used in output.c is declared in lib/gvc/gvc.h.

This change is not strictly necessary since gvc/gvc.h is indirectly
included through common/render.h anyway, but a direct inclusion is
clearer and will avoid confusion when an upcoming commit removes a
duplicated declaration of attach_attrs from utils.h.

Merge branch 'smattr/F874E210-6DEB-4514-B871-F4072917198C' into 'main'

remove legacy expr NMake makefile and Makefile.orig

See merge request graphviz/graphviz!2180

remove legacy expr NMake makefile and Makefile.orig

Maintaining three build systems is already more work than is manageable.
Maintaining four does not seem desirable.

Merge branch 'smattr/5598559D-FB4F-41C0-A52A-E3FBC862B5CD' into 'main'

lefty: a few things missed in recent clean up

See merge request graphviz/graphviz!2179

lefty SFcompareEntries: remove MSVC 'strcascmp' fall back

Commit 1c2c986ebb2d16e491c68656217f5352b3d809c8 replaced `SFCompareEntries`
implementation with `strcasecmp` or `_stricmp` depending on whether we were
compiling with a non-MSVC compiler or with MSVC, respectively. However, this
code is part of the X11 GUI support and hence never built with MSVC. This commit
removes this unnecessary fall back which was dead code.

lefty Dir.c: remove unnecessary include of ctype.h

Commit 1c2c986ebb2d16e491c68656217f5352b3d809c8 removed the only use of
`islower` and `toupper` in this file, which was why this header was included.

lefty SFfindFile: remove MSVC 'strncasecmp' fall back

Commit 69f8a328e1d41b1cd799c715a8398ab0f096a8f4 replaced `SFstrncmp` with
`strncasecmp` or `_strnicmp` depending on whether we were compiling with a
non-MSVC compiler or with MSVC, respectively. However, this code is part of the
X11 GUI support and hence never built with MSVC. This commit removes this
unnecessary fall back which was dead code.

Merge branch 'smattr/4BB774CF-C72E-4136-B8F5-908D835C1A87' into 'main'

abbreviate .clang-format to just pure LLVM style

See merge request graphviz/graphviz!2178

abbreviate .clang-format to just pure LLVM style

The clang-format style used in Graphviz is already pure LLVM style, but was
generated with clang-format-12. This means older versions of clang-format fail
because they do not understand some of the style options that were emitted by
clang-format-12. Reverting to using only `BasedOnStyle` makes it more obvious
that pure LLVM style is in use and permits older versions of clang-format to be
used. None of the newer options introduced by clang-format-12 are critical for
Graphviz formatting.

Merge branch 'smattr/DCA87935-2488-470F-AE3F-983AC3FD6428' into 'main'

more -Wmissing-field-initializer squashing

See merge request graphviz/graphviz!2177

p_assembly: squash a -Wmissing-field-initializer warning

p_noverhang: squash a -Wmissing-field-initializer warning

p_threepoverhang: squash a -Wmissing-field-initializer warning

p_fivepoverhang: squash a -Wmissing-field-initializer warning

p_restrictionsite: squash a -Wmissing-field-initializer warning

p_primersite: squash a -Wmissing-field-initializer warning

p_proteinstab: squash a -Wmissing-field-initializer warning

p_proteasesite: squash a -Wmissing-field-initializer warning

p_rnastab: squash a -Wmissing-field-initializer warning

p_ribosite: squash a -Wmissing-field-initializer warning

Merge branch 'smattr/96b0799b-6f75-44f5-906f-3df53c73ddee' into 'main'

more warning squashing and clean up

See merge request graphviz/graphviz!2181

tclpathplan.c: rephrase open coded fmax/fmin

tclpathplan.c: remove open coded NULL, excess bracketing and casts

tclpathplan.c: remove 'strncmp' micro-optimizations

There is no need for this on modern machines.

tclpathplan.c: remove commented out code

triangle_callback: mark as static

This function is not called outside of its containing file.

tclpathplan between: rephrase to avoid -Wfloat-equal warnings

Similar to commit a420a1a449b23f4be4fb891e1025f20b21c81369. This is intended to
have the same semantics.

tclpathplan gt: rephrase into something more obvious

Similar to commit 1cf8e0225a6a215c93437a9972497f1cae0ffc8c. 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.

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

Similar to commit fe231a8f6beae5dd8ac8cd8b9338cc815881a714.

find_ints.c: remove some commented out code

make_map_internal: flip order of 'bbm1' if-then-else ladder

Squashes a -Wfloat-equal warning.

make_map_internal: flip order of 'bbm0' if-then-else ladder

Squashes a -Wfloat-equal warning.

make_map_internal: flip the order of 'bounding_box_margin' if-then-else ladder

Squashes a -Wfloat-equal warning.

solve2: flip the order of an if-then-else ladder to squash -Wfloat-equal warning

Merge branch 'smattr/D035A7A7-F419-4B11-9BA8-3C003CC0B4AC' into 'main'

smyrna: more dead code removal

See merge request graphviz/graphviz!2176

smyrna: remove commented out prototypes

smyrna: remove prototype of unimplemented 'drawRotatingTools'

smyrna: remove unused 'drawEllipse'

smyrna: remove unused 'rasters' global

smyrna: remove unused 'point_within_ellips_with_coords'

smyrna: remove unused 'point_within_sphere_with_coords'

smyrna: remove macOS Leopard hack

Leopard was macOS 10.5, released in 2007. Apparently this hack is no longer
required as Smyrna is built on macOS in CI without this.

smyrna: remove unused 'texture' global

smyrna: remove some prototypes of unimplemented functions

smyrna: remove SmyrnaVerbose

Nothing uses this variable.

smyrna: remove unused SmrynaVerbose declaration

This variable name is typoed, but nothing uses it anyway.

Merge branch 'smattr/AC34DC16-D8EF-4259-8FE4-5A5080DE6602' into 'main'

Move back to 2.49 development series

See merge request graphviz/graphviz!2183

Move back to 2.49 development series

Merge branch 'smattr/B37B1A9A-4D07-4914-AB34-4EF0B33EE330' into 'main'

Stable Release 2.49.1

See merge request graphviz/graphviz!2167

Stable Release 2.49.1

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.