Merge branch 'smattr/DFDACC2F-624B-4360-BC9C-9B2A116EB11C' into 'main'

more boolean → modernization

See merge request graphviz/graphviz!2391

is_internal_to_cluster; return a C99 bool instead of an ambiguous int

is_nonconstraint: return a C99 bool instead of an ambiguous int

is_empty: return a C99 bool instead of an ambigous int

is_cluster: return a C99 bool instead of an ambiguous int

is_a_cluster: return a C99 bool instead of ambiguous int

This function was already morally returning a boolean value.

mapbool: take a 'const char*' parameter instead of 'char*'

This parameter is not modified internally.

mapbool: return a C99 bool instead of a boolean

mapBool: take a 'const char*' parameter instead of 'char*'

This parameter is not modified within the function.

mapBool: use C99 bools for boolean parameters

Note that this required a somewhat awkward work around in `getAdjustMode` that
was abusing being able to fit something other than a boolean into the type used
for booleans.

late_bool: use C99 bools for boolean parameters

ortho segment: remove unused 'flipped' member

node_distinct_coloring: take 'weightedQ' parameter as a C99 bool

gvmap plot_dot_map: use a C99 bool for local 'plot_polyQ'

gvprintpointflist: replace a boolean literal with C99 bool 'true'

Merge branch 'unify-storage-class-declarations-in-xdot' into 'main'

Unify storage class declarations in xdot

See merge request graphviz/graphviz!2390

xdot: [nfc] align storage-class declarations to Graphviz standard in xdot.h

xdot: [nfc] remove useless default extern declaration of functions in xdot.h

xdot: use GVDLL instead of _WIN32 to control storage-class attributes in xdot.h

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

xdot: [nfc] remove whitespace in preprocessor directives in xdot.h

xdot: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes in
xdot.h to be controlled by the GVDLL symbol.

Without this change, errors like this would occur in lib/gvc:

LINK : fatal error LNK1104: cannot open file 'C:\Users\magja\graphviz\Debug\Graphviz\bin\xdot.lib'
Done building project "gvc.vcxproj" -- FAILED.

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

Merge branch 'misc-nfc-unify-storage-class-declarations' into 'main'

Miscellaneous non-functional changes to unify storage class declarations

See merge request graphviz/graphviz!2389

sfio: [nfc] rename EXTERN macro to SFIO_API in sfio.h

ortho: [nfc] align storage-class declarations to Graphviz standard in ortho.h

lib/gvpr: [nfc] remove extra _WIN32 condition to apply storage-class attributes in gvpr.h

This is a non-functional change since this code is guarded with GVDLL
which is only ever defined when _WIN32 is defined.

lib/gvpr: [nfc] remove whitespace in preprocessor directives in gvpr.h

gvc: [nfc] remove useless default extern declaration of functions in gvc.h

edgepaint: [nfc] align storage-class declarations to Graphviz standard in lab_gamut.h

edgepaint: [nfc] rename EXTERN macro to LAB_GAMUT_API in lab_gamut.h

pathutil: [nfc] remove useless default extern declaration of functions in vis.h

pathplan: [nfc] remove useless comments in vis.h

pathplan: [nfc] remove useless default extern declaration of functions in pathutil.h

pathplan: [nfc] remove useless comments in pathutil.h

pathplan: CMake: [nfc] remove useless definition of _BLD_pathplan

This symbol is not used anywhere.

pathplan: [nfc] remove useless definition of _BLD_pathplan in pathutil.h

This symbol is not used anywhere.

Merge branch 'fix-mingw-cmake-builds-and-add-tests' into 'main'

Fix MinGW CMake builds and add tests

Closes #2176 and #1780

See merge request graphviz/graphviz!2387

ci: add windows-mingw32-cmake-test job

ci: add windows-mingw64-cmake-test job

add ci/mingw-test.sh

ci/mingw-install.sh: install python-pip

test_1408: don't expect failure on MinGW

It's unclear why this test does not fail on MinGW when if fails on
other OSes. The NDEBUG symbols is not defined in MinGW builds and
assertions have been verified to be enabled.

gvtest.py: correct compiler used on MinGW

CMake: ensure PangoCairo is found on MinGW

This makes the PNG format available in CMake builds with MinGW.

CMake: ensure Cairo is found on MinGW

plugin/pango: print font map when verbose for all platforms

This effectively reverts commit
2029d4bf76360aba04a0ca779058caba7682e1e9.

Fixes this error with CMake on MinGW when compiled with -Werror:

C:/Users/magja/graphviz/plugin/pango/gvgetfontlist_pango.c:477:1: error: 'printFontMap' defined but not used [-Werror=unused-function]
  477 | printFontMap (gv_font_map*gv_fmap, int sz)
      | ^~~~~~~~~~~~

plugin/pango: include common/globals.h in gvgetfontlist_pango.c

An upcoming commit which enables code that uses the Verbose global
variable would otherwise cause this error:

gvgetfontlist_pango.obj : error LNK2019: unresolved external symbol _Verbose referenced in function _get_font_mapping [C:\GitLab-Runner\builds\magjac\graphviz\build\plugin\pango\gvplugin_pango.vcxproj]
C:\GitLab-Runner\builds\magjac\graphviz\build\plugin\pango\Release\gvplugin_pango.dll : fatal error LNK1120: 1 unresolved externals [C:\GitLab-Runner\builds\magjac\graphviz\build\plugin\pango\gvplugin_pango.vcxproj]

plugin/pango: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit which enables code that uses the Verbose global
variable would otherwise cause this error:

gvgetfontlist_pango.obj : error LNK2001: unresolved external symbol _Verbose

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

gvpr: correct fix for absolute paths to work also on MinGW

Fixes https://gitlab.com/graphviz/graphviz/-/issues/1780 on MinGW.

Without this fix, test_1780 fails:

__________________________________ test_1780 __________________________________
Traceback (most recent call last):
  File "C:/Users/magja/graphviz2/rtest/test_regression.py", line 594, in test_1780
    subprocess.check_call(["gvpr", "-f", clustg], stdin=subprocess.DEVNULL)
  File "C:/tools/msys64/mingw64/lib/python3.9/subprocess.py", line 373, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['gvpr', '-f', WindowsPath('C:/Users/magja/graphviz2/cmd/gvpr/lib/clustg')]' returned non-zero exit status 4294967295.
---------------------------- Captured stderr call -----------------------------
gvpr: gvpr
Usage: gvpr gvpr

plugin/pango: work around CMake not generating DLLs with version numbers on MinGW

Fixes https://gitlab.com/graphviz/graphviz/-/issues/2176.

plugin/neato_layout: work around CMake not generating DLLs with version numbers on MinGW

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

plugin/gdiplus: work around CMake not generating DLLs with version numbers on MinGW

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

plugin/gd: work around CMake not generating DLLs with version numbers on MinGW

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

plugin/dot_layout: work around CMake not generating DLLs with version numbers on MinGW

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

plugin/core: work around CMake not generating DLLs with version numbers on MinGW

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

CMake: detect "dot -c" failure when running cpack

This reveals this error when "dot -c" is executed when running cpack
on MinGW:

CPack: Create package using ZIP
CPack: Install projects
CPack: - Install project: Graphviz []
CMake Error at C:/Users/magja/graphviz2/cmake/configure_plugins.cmake:38 (execute_process):
  execute_process abnormal exit: Access violation
Call Stack (most recent call first):
  C:/Users/magja/graphviz2/build/cmd/dot/cmake_install.cmake:91 (include)
  C:/Users/magja/graphviz2/build/cmd/cmake_install.cmake:42 (include)
  C:/Users/magja/graphviz2/build/cmake_install.cmake:60 (include)

CPack Error: Error when generating package: Graphviz

See https://gitlab.com/graphviz/graphviz/-/issues/2176.

Merge branch 'smattr/B7A7E3A2-8D16-48E6-A2EE-1E56CD300DF2' into 'main'

delete unused header constants

See merge request graphviz/graphviz!2382

API BREAK: remove unused '_DUMMY_ELEM'

ISO C99 and C++11 both permit trailing commas in enum members. Graphviz has been
compiled using C99 since fe3f9411d2c59b463ab1b64eecfd19f2db55d2fc and C++11
since a74e0e104e114bcf890eb8419a0bf02a22e01d84. This wart was not removed
previously as removing it was considered an API break not worth doing. The
upcoming release is projected to bump the major version number, making breaks
like this more acceptable.

API BREAK: remove unused 'MAXSHORT' definition

The last use of this was removed in 580fa88477fa59aca01e405c986c686ceee21b5f.
But its definition was retained in this header as removing it was considered an
API break not worth causing. The upcoming release is projected to bump the major
version number, so we can now remove it.

API BREAK: remove local 'INT_MAX' and 'INT_MIN' definitions

These constants are defined in ISO C to be available in limits.h, but were
commonly available pre-ISO C too. These definitions remained here for legacy
reasons, because removing them was considered an API break and not worth doing
for this minor clean up. However the upcoming release is projected to involve a
major version number bump.

Merge branch 'unify-storage-class-declarations-in-common' into 'main'

Unify storage-class declarations in common headers with Graphviz standard

See merge request graphviz/graphviz!2386

common: [nfc] remove useless default extern declaration of functions in memory.h

plugin/visio: use only the GVDLL symbol enable storage-class attributes in gvplugin_visio.c

Anything that depends on storage-class attributes already define both
GVC_EXPORTS and WIN32_DLL.

MSBuild: remove now obsolete definition of WIN32_DLL from all project files

pack: autotools: remove now obsolete definition of _BLD_gvc

pack: CMake: remove now obsolete definition of _BLD_gvc

ortho: autotools: remove now obsolete definition of _BLD_gvc

ortho: CMake: remove now obsolete definition of _BLD_gvc

gvc: autotools: remove now obsolete definition of _BLD_gvc

gvc: CMake: remove now obsolete definition of _BLD_gvc

common: autotools: remove now obsolete definition of _BLD_gvc

common: CMake: remove now obsolete definition of _BLD_gvc

gvc: remove useless definition of _BLD_gvc in gvusershape.c

This symbols is already defined by all build systems that depend on
it.

common: use only GVC_EXPORTS to control storage-class attributes in global.h

Anything that depends on storage-class attributes already define both
GVC_EXPORTS and _BLD_gvc.

common: use only the GVDLL symbol enable storage-class attributes in global.h

Anything that depends on storage-class attributes already define both
GVC_EXPORTS and WIN32_DLL.

osage: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes in
globals.h to be controlled only by the GVDLL symbol, not also by the
WIN32_DLL symbol.

Without this change, errors like this would occur:

osage.lib(osageinit.obj) : error LNK2001: unresolved external symbol _Ndim

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

patchwork: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes in
globals.h to be controlled only by the GVDLL symbol, not also by the
WIN32_DLL symbol.

Without this change, errors like this would occur:

patchwork.lib(patchworkinit.obj) : error LNK2001: unresolved external symbol _Ndim

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

twopigen: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes in
globals.h to be controlled only by the GVDLL symbol, not also by the
WIN32_DLL symbol.

Without this change, errors like this would occur:

twopigen.lib(twopiinit.obj) : error LNK2001: unresolved external symbol _Ndim

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

sfdp: MSBuild: define GVDLL to ensure correct storage-class attributes

An upcoming commit will change the storage-class attributes in
globals.h to be controlled only by the GVDLL symbol, not also by the
WIN32_DLL symbol.

Without this change, errors like this would occur:

sfdp.lib(spring_electrical.obj) : error LNK2001: unresolved external symbol _Verbose

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

common: [nfc] rename DECLSPEC macro to GLOBALS_API in globals.h

common: [nfc] remove useless default extern declaration of functions in geomprocs.h

Merge branch 'smattr/046089E8-9134-4984-9478-E5E84A0C637F' into 'main'

CI: stop installing a custom Python on CentOS 7

See merge request graphviz/graphviz!2385

CI: stop installing a custom Python on CentOS 7

It is not clear to me why this was pulling in a custom Python 3.6 when Python
3.6 is in the default repositories on CentOS 7. It is possible it was not
packaged there at the time this Dockerfile was written.

Merge branch 'install-msys2-from-msys2.org' into 'main'

CI: install msys2 from msys2.org instead of with choco

See merge request graphviz/graphviz!2375

ci: install msys2 from msys2.org instead of with choco

Merge branch 'unify-storage-class-declarations-in-cgraph' into 'main'

Unify storage-class declarations in cgraph headers with Graphviz standard

See merge request graphviz/graphviz!2380

add a CHANGELOG entry for the breaking GVDLL change in cgraph.h

cgraph: [nfc] align storage-class declarations to Graphviz standard in sprint.h

cgraph: [nfc] rename DECLSPEC macro to SPRINT_API

cgraph: use GVDLL instead of _WIN32 to control storage-class attributes in csprint.h

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

cgraph: [nfc] align storage-class declarations to Graphviz standard in cgraph.h

cgraph: use GVDLL instead of _WIN32 to control storage-class attributes in cgraph.h

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

cgraph: [nfc] remove whitespace in preprocessor directives in cgraph.h

cgraph: [nfc] align storage-class declarations to Graphviz standard in cghdr.h

cgraph: [nfc] remove useless default extern declaration of functions in cghdr.h

cgraph: use GVDLL instead of _WIN32 to control storage-class attributes in cghdr.h

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

cgraph: [nfc] remove whitespace in preprocessor directives in cghdr.h

cgraph: remove useless definition of _BLD_cgraph in cghdr.h

This symbol is not used anywhere.

cgraph: move include of config.h to before defining storage-class attributes in cghdr.h

The GVDLL symbol is defined in config.h for autotools builds and an
upcoming commit which changes cghdr.h to use GVDLL to control the
storage-class attribute declarations would otherwise cause errors like
this:

C:\Users\magja\graphviz\cmd\tools/gc.c:333: undefined reference to `__imp_agapply'

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

cgraph: [nfc] align storage-class declarations to Graphviz standard in agxbuf.h

cgraph: [nfc] remove useless default extern declaration of functions in agxbuf.h