smyrma: push the scope of 'package_*' variables inwards

These do not need to be exposed beyond main.c or even need to be globals. They
are not used outside of a single function.

smyrna: remove 'glade_set_atk_action_description' prototype

This function was removed in 514a0056e9dfaa90d2bef63bb2150b6bd86152c9.

smyrna support.h: remove commented out code

smyrna: remove 'create_pixmap' prototype

This function was removed in 514a0056e9dfaa90d2bef63bb2150b6bd86152c9.

smyrna: remove 'lookup_widget' prototype

This function was removed in 514a0056e9dfaa90d2bef63bb2150b6bd86152c9.

smyrna: remove prototype of 'freeFocus'

This function was removed in af92d6ce857bfa235d6fe771088074a9979b7172.

smyrna set_color_theme_color: remove 'smooth' parameter

This function is only ever called with `1` as this value.

smyrna: remove unused 'respond' global

smyrma: remove unused 'redraw'

smyrna: remove some commented out code

smyrna: remove unused 'width' and 'height' globals

Merge branch 'smattr/C34D084E-BE41-42C0-9742-E887E6D06631' into 'main'

smyrna: even more dead code removal

See merge request graphviz/graphviz!2185

smyrna: remove unused save_graph_clicked

smyrna: remove unused open_graph_clicked

smyrna: remove unused new_graph_clicked

smyrna: remove unused btnTVFilterApply_clicked_cb

smyrna: remove unused btnTVReverse_clicked_cb

smyrna: remove unused btnTVCancel_clicked_cb

smyrna: remove unused btnTVEdit_clicked_cb

smyrna: remove unused mAttributesSlot

smyrna: remove unused buf

smyrna: remove unused write_to_console

Merge branch 'smattr/9475BC85-3C8A-4113-A179-ACF4D166E17E' into 'main'

more -Wmissing-field-initializer warning squashing

See merge request graphviz/graphviz!2190

common Shapes: squash -Wmissing-field-initializer warnings

p_lpromoter: squash -Wmissing-field-initializer warning

p_larrow: squash -Wmissing-field-initializer warning

p_rarrow: squash -Wmissing-field-initializer warning

p_promoter: squash -Wmissing-field-initializer warning

p_signature: squash -Wmissing-field-initializer warning

Merge branch 'smattr/C64DD787-8A55-438E-9B8D-01EFBEB8718E' into 'main'

lefty clean up

See merge request graphviz/graphviz!2187

lefty: abbreviate some boolean returns in 'orderop'

lefty: remove unused 'Eshowcalls'

lefty: remove unused 'Eoktorun'

lefty: remove unnecessary casts of char* members

lefty: consistently treat 'CBUFINCR' as a 'long'

lefty: consistently treat 'MARKINCR' as a 'long'

lefty: consistently treat 'OTINCR' as a 'long'

lefty: remove C++ guards

This program is entirely C, so does not need these explicit linkage specs.

lefty: replace header guards with more modern #pragma once

This omits the `_COMMON_H` guards as these are used in multiple files and may
have some other interaction.

lefty: mark 'GFXbuttoncb' static

This function is not used outside of its containing file.

lefty: mark 'GFXtextcb' static

This function is not used outside of its containing file.

lefty: remove 'Gerrno'

Nothing ever reads this variable.

Merge branch 'smattr/6AF46EA6-D097-41D0-A29B-E5F270074178' into 'main'

CMake: fix: build gv2gml instead of symlinking gml2gv

Closes #2131

See merge request graphviz/graphviz!2192

CMake: fix: build gv2gml instead of symlinking gml2gv

Fixes #2131.

add a test case provoking gv2gml/gml2gv confusion

Related to #2131.

Merge branch 'fix-bom-in-collection-file-in-mingw-build-jobs' into 'main'

CI: avoid BOM in COLLECTION file in MinGW build jobs

Closes #2130

See merge request graphviz/graphviz!2188

CI: avoid BOM in COLLECTION file in MinGW build jobs

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

Merge branch 'define-GVDLL-in-CMake-builds' into 'main'

Define GVDLL also in CMake builds for Windows

See merge request graphviz/graphviz!2182

CMake: set GVDLL for WIN32

The GVDLL symbol has for a long time been used by the autotools build
system to control whether to apply the Microsoft storage-class
attributes, __declspec(dllexport) and __declspec(dllimport), that are
needed when building dynamic-link libraries (DLLs) on Windows, also
when using MinGW. The autotools build system does not define GVDLL
when static linking is requested.

Since the introduction of the CMake build system, this functionality
has gradually become more and more broken for both dynamic-link
libraries with MinGW and for static libraries in general under
Windows.

Upcoming commit series will try to restore this functionality. Rather
than having a plethora of different symbols controlling the
storage-class attributes in different ways for different build
systems, they will settle on something like this across all build
systems:

This means that in order to continue to support dynamic-link libraries
with CMake, the GVDLL symbol must be set.

The CMake build system does not currently support building static
libraries, so the GVDLL symbol can be unconditionally set for WIN32
until such support is introduced.

This is also a very small step towards
https://gitlab.com/graphviz/graphviz/-/issues/2058.

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