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

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

Without this change, errors like this would occur:

EXEC : error : failed to resolve gvplugin_gd_LTX_library in C:\Users\magja\graphviz\Debug\Graphviz\bin\gvplugin_gd.dll

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

plugin/gd: [nfc] remove whitespace in preprocessor directives in gvplugin_gd.c

plugin/dot_layout: use GVDLL instead of _WIN32 to enable dllexport storage-class attribute in gvplugin_dot_layout.c

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

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

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

Without this change, errors like this would occur:

EXEC : error : failed to resolve gvplugin_dot_layout_LTX_library in C:\Users\magja\graphviz\Debug\Graphviz\bin\gvplugin_dot_layout.dll

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

plugin/dot_layout: [nfc] remove whitespace in preprocessor directives in gvplugin_dot_layout.c

plugin/core: use GVDLL instead of _WIN32 to enable dllexport storage-class attribute in gvplugin_core.c

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

plugin/core: [nfc] remove whitespace in preprocessor directives in gvplugin_core.c

Merge branch 'install-python-with-nuget' into 'main'

CI: install Python with nuget instead of with choco

See merge request graphviz/graphviz!2392

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

more boolean → modernization

See merge request graphviz/graphviz!2391

ci: install python with nuget in windows-cygwin-build-using-autogen job

ci: remove now obsolete choco cache from windows-msbuild-Win32-debug-build job

ci: remove now obsolete choco cache from windows-msbuild-Win32-release-build job

ci: remove now obsolete choco cache from windows-cmake-x64-debug-build job

ci: remove now obsolete choco cache from windows-cmake-x64-release-build job

ci: remove now obsolete choco cache from windows-cmake-Win32-debug-build job

ci: remove now obsolete choco cache from windows-cmake-Win32-release-build job

ci: install python with nuget in windows_build_definition job template

ci: remove now obsolete choco cache from windows-mingw64-cmake-build job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

ci: remove now obsolete choco cache from windows-mingw32-cmake-build job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

ci: remove now obsolete choco cache from windows-mingw64-build-step2-make job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

ci: remove now obsolete choco cache from windows-mingw64-build-step1-configure job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

ci: remove now obsolete choco cache from windows-mingw32-build-step2-make job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

ci: remove now obsolete choco cache from windows-mingw32-build-step1-configure job

Commit 3f4d8dfc07899d113ddc49e2610b669113ea2810 removed the use of the
choco cache, but missed to remove the cache itself from this job.

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.