emit_xdot: remove unnecessary bracketing

init_job_pagination: use fmax instead of open coding

init_job_pagination: remove unnecessary bracketing

firstlayer: remove unnecessary bracketing

init_layering: remove unnecessary bracketing

map_output_bspline: remove unnecessary bracketing

bisect: remove unnecessary bracketing

isFilled: return a C99 bool instead of int

rename 'ifFilled' to 'isFilled'

From the leading comment, this is what it was intended to be called.

stripedBox: remove unnecessary bracketing

wedgedEllipse: remove unnecessary bracketing

checkClusterStyle: remove unnecessary bracketing

getObjId: remove unnecessary bracketing

layerPagePrefix: remove unnecessary bracketing

colorseg_t: use a C99 bool for 'hasFraction' field

isRect: remove unnecessary bracketing

isRect: use a C99 bool return type instead of boolean

interpretCRNL: use a C99 bool for 'backslash_seen' local

remove some open coded NULLs

parse_style: use a C99 bool for 'in_parens' local

parse_style: use a C99 bool for 'is_first' local

is_style_delim: use a C99 bool return type instead of boolean

nodeIntersect: use a C99 bool for 'explicit' local

emit_edge_graphics: use a C99 bool for 'tapered' local

node_in_box: use a C99 bool return type instead of boolean

clust_in_layer: use a C99 bool return type instead of boolean

edge_in_layer: remove unnecessary bracketing

edge_in_layer: use a C99 bool return type instead of boolean

node_in_layer: remove unnecessary bracketing

node_in_layer: use a C99 bool return type instead of boolean

write_node_test: use a C99 bool return type instead of boolean

write_edge_test: use a C99 bool return type instead of boolean

validpage: remove unnecessary bracketing

validpage: use a C99 bool return type instead of boolean

validlayer: remove unnecessary bracketing

validlayer: use a C99 bool return type instead of boolean

selectedLayer: remove unnecessary bracketing

selectedLayer: use a C99 bool return type instead of boolean

is_natural_number: use more precise type for input parameter

We can be more strict and rigorous with how we deal with strings here.

is_natural_number: use a C99 bool return type instead of boolean

more standard usage of isdigit and friends

remove some commented out code

colorcmpf: remove strcmp micro-optimization

This is unnecessary when using a modern compiler.

colorcmpf: remove unnecessary casts

These pointers implicitly coerce.

colorxlate: remove some unnecessary bracketing

colorxlate: remove strcmp micro-optimization

This is unnecessary when using a modern compiler.

replace header guards in lib/common with more modern #pragma once

remove cryptic globals.h comment

The link in this comment is dead and it is unclear what it is referring to.

colorxlate: squash some -Wswitch-default warnings

colorxlate: squash some -Wconversion warnings

By design, these calculations cannot overflow.

colorCvt: remove commented out code

rgb2cmyk: call fmin instead of open coding floating point comparison

replace use of MIN, MAX macros in colxlate.c with standard fmin, fmax

canontoken: use char pointers instead of unsigned char pointers

It is not clear to me why this code was using unsigned char pointers.

canontoken: remove commented out code

hsv2rgb: remove unnecessary bracketing

remove unnecessary casts around usage of 'bsearch'

colorxlate: remove unnecessary bracketing

colorxlate: remove a strcmp micro-optimization

This is unnecessary when using modern compilers.

colorxlate: use char pointers instead of unsigned char pointers

It is not clear to me why this code was using unsigned char pointers.

Merge branch 'smattr/7e2248f2-8a26-4cfa-9908-b5d191b0070e' into 'main'

fix various -Wmissing-field-initializer warnings

See merge request graphviz/graphviz!2130

p_plain: squash -Wmissing-field-initializer warning

p_plaintext: squash -Wmissing-field-initializer warning

p_square: squash -Wmissing-field-initializer warning

p_box: squash -Wmissing-field-initializer warning

p_triangle: squash -Wmissing-field-initializer warning

p_egg: squash -Wmissing-field-initializer warning

p_circle: squash -Wmissing-field-initializer warning

p_ellipse: squash -Wmissing-field-initializer warning

p_polygon: squash -Wmissing-field-initializer warning

Center: squash -Wmissing-field-initializer warning

Merge branch 'smattr/9A8B94F0-FA1E-4200-A71D-03595C913ACF' into 'main'

remove Fedora 32 CI testing

See merge request graphviz/graphviz!2128

remove Fedora ≤ 32 work around

remove Fedora 32 CI testing

Fedora 32 was EOLed in May 2021.¹

¹ https://fedoraproject.org/wiki/End_of_life?rd=LifeCycle/EOL

Merge branch 'smattr/C574183C-758B-4503-AFE2-3BEE5F2BBA87' into 'main'

some Pylint warning squashing

See merge request graphviz/graphviz!2118

fix some missing Python docstrings

squash Pylint warnings about shadowing 'root'

squash Pylint warnings about shadowing 'p'

squash Pylint wrong-import-position warnings

squash a Pylint line-too-long warning

Merge branch 'smattr/8D176743-758B-4021-8E02-E18155F7503A' into 'main'

smyrna: remove expat installation when using MS Build

See merge request graphviz/graphviz!2115

smyrna: remove expat installation when using MS Build

AFAICT Smyrna does not directly depend on expat. It depends on it indirectly,
but should then pick up expat installation steps from dependent MS Build files.

Merge branch 'add-mingw-cmake-builds-to-ci' into 'main'

Add MinGW CMake build jobs to CI

See merge request graphviz/graphviz!2107

CI: add windows-mingw64-cmake-build job

CI: add windows-mingw32-cmake-build job

add ci/mingw-build.sh script

ci/build.sh: CMake: move the MinGW zip and exe build artifacts to the release directory

remove import/export declaration of lt_preloaded_sybols in gvcext.h

lt_preloaded_symbols is always defined in the executable that links to
the gvc lib, so it should only have a standard extern declaration.

Fixes this error with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: cmd/dot/CMakeFiles/dot.dir/dot.c.obj:dot.c:(.text+0x1e9): undefined reference to `__imp_lt_preloaded_symbols'

Also removes this warning with MinGw:

C:/Users/magja/graphviz/cmd/dot/no_builtins.c:13:14: warning: 'lt_preloaded_symbols' redeclared without dllimport attribute: previous dllimport ignored [-Wattributes]
   13 | lt_symlist_t lt_preloaded_symbols[] = { { 0, 0 } };
      |              ^~~~~~~~~~~~~~~~~~~~

Also removes these three warnings in standard Windows MSVC builds:

C:\Users\magja\graphviz\cmd\dot\no_builtins.c(13,37): warning C4273: 'lt_preloaded_symbols': inconsistent dll linkage [C:\Users\magja\graphviz\build\cmd\dot\dot.vcxproj]
C:\Users\magja\graphviz\cmd\tools\gvpack.c(34,37): warning C4273: 'lt_preloaded_symbols': inconsistent dll linkage [C:\Users\magja\graphviz\build\cmd\tools\gvpack.vcxproj]
LINK : warning LNK4217: symbol 'lt_preloaded_symbols' defined in 'no_builtins.obj' is imported by 'dot.obj' in function 'main' [C:\Users\magja\graphviz\build\cmd\dot\dot.vcxproj]

label: CMake: ensure label symbols are exported from the gvc lib

The object modules from the static label lib are statically linked
into the gvc shared lib through the common object library which means
that their symbols should be exported from gvc instead of
imported. Defining GVC_EXPORTS achieves this.

This is what the autotools build already does.

Fixes this error with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../label/liblabel.a(xlabels.c.obj):xlabels.c:(.text+0x98): undefined reference to `__imp_zmalloc'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../label/liblabel.a(xlabels.c.obj):xlabels.c:(.text+0x1f40): undefined reference to `__imp_zmalloc'

ortho: CMake: ensure ortho symbols are exported from gvc

The object modules from the static ortho lib are statically linked
into the gvc shared lib which means that its symbols should be
exported from gvc instead of imported. Defining GVC_EXPORTS achieves
this.

Fixes errors like this with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../ortho/CMakeFiles/ortho_obj.dir/fPQ.c.obj:fPQ.c:(.text+0x6d): undefined reference to `__imp_gcalloc'

common: ensure correct import/export declarations of expat with MinGW

The expat library does not seem to recognize MinGW and therefore does
not apply the Microsoft storage class attributes to its function
declarations when using MinGW. Defining XML_USE_MSC_EXTENSIONS forces
it to use those.

Fixes errors like this with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: ../common/CMakeFiles/common_obj.dir/htmllex.c.obj:htmllex.c:(.text+0x1cec): undefined reference to `XML_ParserCreate'

CMake: fix expat runtime library not found by cpack on msys2

Fixes these errors with MinGW:

CMake Error at C:/Users/magja/graphviz/build/lib/gvc/cmake_install.cmake:75 (file):
  file INSTALL cannot find
  "C:/Users/magja/graphviz/lib/gvc/EXPAT_RUNTIME_LIBRARIES-NOTFOUND": No
  error.
Call Stack (most recent call first):
  C:/Users/magja/graphviz/build/lib/cmake_install.cmake:70 (include)
  C:/Users/magja/graphviz/build/cmake_install.cmake:58 (include)

CMake Error at C:/Users/magja/graphviz/build/cmd/tools/cmake_install.cmake:269 (file):
  file INSTALL cannot find
  "C:/Users/magja/graphviz/cmd/tools/EXPAT_RUNTIME_LIBRARIES-NOTFOUND": No
  error.
Call Stack (most recent call first):
  C:/Users/magja/graphviz/build/cmd/cmake_install.cmake:44 (include)
  C:/Users/magja/graphviz/build/cmake_install.cmake:60 (include)

CPack Error: Error when generating package: Graphviz

gvc: WIN32: install expat runtime library only if we have it

gvc: CMake: link to ltdl even on Windows with MinGW

Fixes these errors with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x4f7): undefined reference to `lt_dlinit'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x529): undefined reference to `lt_dlopen'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x578): undefined reference to `lt_dlerror'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x69e): undefined reference to `lt_dlsym'
collect2.exe: error: ld returned 1 exit status

CMake: FindLTDL: ensure ltdl is found with MinGW

CMake defines both WIN32 and MINGW when the compiler is some version
of MinGW. With MinGW we do want to use the ltdl library athough on
native Windows we only use the ltdl compatibility header introduced in
d70cdba7937724c19393d97bd280320cdf8c3c8f, not the library itself.

Fixes the following errors with MinGW:

C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x4f7): undefined reference to `lt_dlinit'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x529): undefined reference to `lt_dlopen'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x578): undefined reference to `lt_dlerror'
C:/tools/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.3.0/../../../../x86_64-w64-mingw32/bin/ld.exe: CMakeFiles/gvc.dir/objects.a(gvplugin.c.obj):gvplugin.c:(.text+0x69e): undefined reference to `lt_dlsym'

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

CMake: add use_win_pre_inst_libs option

This option will be used in an upcoming commit that adds MinGW jobs to
CI.

Merge branch 'more-linking-clean-up' into 'main'

More linking cleanup

See merge request graphviz/graphviz!2127

common: link to expat privately instead of publicly

Downstream libraries that explicitly use expat now all link to expat
themselves and no longer rely on a transitive dependency to expat
through gvc via common.

plugin/neato_layout: remove linking to cdt

The neato_layout plugin source files do not refer to cdt.

plugin/gd: remove linking to cdt

The gd plugin source files do not refer to cdt.