Merge branch 'smattr/886DC5C9-BF2B-47F2-9E00-7083AAB2423F' into 'main'

lib/pack cleanup

See merge request graphviz/graphviz!2430

gml2gv: [nfc] remove unnecessary include

lib/pack: [nfc] remove duplicate include

lib/pack: [nfc] remove open coded NULLs

lib/pack: [nfc] remove unnecessary parens

insertFn: [nfc] remove unnecessary cast

Merge branch 'upgrade-to-macos-12-xcode-13' into 'main'

Upgrade to MacOS 12, Xcode 13

See merge request graphviz/graphviz!2370

CI: upgrade to macos-12-xcode-13

Merge branch 'smattr/F06FAC81-75F7-4E90-9AA8-368F7D31E8D8' into 'main'

cmd/tools: [nfc] some minor cleanup

See merge request graphviz/graphviz!2429

gxl2gv: [nfc] remove the need to cast 'mapLookup' parameter

The equivalent of the prior commit on gxl2gv.

graphml2gv: [nfc] remove the need to cast 'mapLookup' parameter

graphml2gv: [nfc] remove the need to cast 'isAnonGraph' parameter

Equivalent of 3e09cc381e53ece08529dd35d27a7a436dd9ab0f on graphml2gv.

Merge branch 'smattr/729B6BDD-7A42-455A-8B12-AA2F3A929663' into 'main'

add a test case for #2092

See merge request graphviz/graphviz!2427

dot2gxl: de-dupe 'gcalloc' implementation

dot2gxl: [nfc] remove the need to cast 'isAnonGraph' parameter

add a test case for #2092

Merge branch 'smattr/E428EEE4-08B8-472D-B3B6-87C93C5D321E' into 'main'

minor test clean up

See merge request graphviz/graphviz!2425

test 1594: [nfc] tell 'subprocess' to discard stdout

This test case was capturing stdout and then discarding it. It is more efficient
to just attach stdout directly to /dev/null.

test 1594: remove work around for #1780

Issue #1780 was fixed in a286e358006dd49d2b3348abd8a2aebcffd24dec.

Merge branch 'smattr/650B3501-D9F6-4EDA-BE4C-1C8EA4C2283F' into 'main'

add a test case for #2179

See merge request graphviz/graphviz!2426

textspan.c: [nfc] remove unnecessary casts

add a test case for #2179

test 1449: [nfc] do not capture stdout

Graphviz is being passed `-o /dev/null` in this test case, so there is no need
to capture stdout.

Merge branch 'smattr/629BA038-E6B6-4140-8BA3-CFB49323FE14-2' into 'main'

CMake: fix CMAKE_LIBRARY_PATH is not honored

Closes #1973

See merge request graphviz/graphviz!2422

CMake: use more standard, portable mechanism for setting install paths

Fixes #1973.

Suggested-by: Satadru Pramanik

increase CMake log-level in CI

Allows us to more easily debug failures.

CMake: add a variable substitution step for plugin configuration step

This CMake script hard codes some library paths in a way that thwarts
user/system preferences (#1973). Attempts to solve this by falling back on
standard CMake mechanisms have been unsuccessful because an install script like
this runs in an isolated environment where it (correctly) does not have access
to the configuration from the build environment. To resolve this we need a way
to pass the install script parameters/arguments from the build context.
Introducing a level of indirection here gives us such a mechanism.

This change by itself does nothing. However an upcoming change will take
advantage of the `@…@` substitution mechanism enabled by this change.

Note that the word “configure” in these files is used to refer to two distinct
mechanisms:

  1. The CMake `configure_file` command does something analogous to Autotools’
     variable substitution during `./configure`.

  2. The Graphviz `dot -c …` configure step registers installed Graphviz
     plugins.

Gitlab: #1973

Merge branch 'smattr/A8790232-0AB5-4072-B45F-50CCECF99A65' into 'main'

remove support for '-m' command-line option

See merge request graphviz/graphviz!2424

remove support for '-m' command-line option

As described in the changelog entry, LSan and friends are a more effective way
of performing this testing now.

Merge branch 'smattr/224C0C90-0ABD-44D2-9D3F-DCAD4702E26D' into 'main'

fix Gvedit `-?` and enable building in CI on macOS

See merge request graphviz/graphviz!2419

CI: add installation of Qt5 on macOS

This is a dependency for building Gvedit on macOS. Note that installation is not
added to the CMake test job because Gvedit is not currently built in the CMake
build system on macOS.

Gitlab: #1842, #1854

fix: teach Gvedit `-?` on macOS

This is the equivalent of aa695fabf4ff342488229aabd43c0970470d711f and friends,
but for Gvedit. It was not previously detected that this problem also affects
Gvedit because it is not currently built on macOS in CI. That will be improved
in an upcoming commit.

Gitlab: #1842, #1852

Merge branch 'smattr/3D701FC9-FA35-4BB3-BD91-3C6492C688BF' into 'main'

CI: add a job for enforcing CMake file formatting

See merge request graphviz/graphviz!2423

CI: add a job for enforcing CMake file formatting

This should hopefully reduce the mental load on MR reviewers going forwards.

CMake: [nfc] add a leading comment to a macro

Squashes a “[C0111] Missing docstring on function or macro declaration”
`cmake-lint` warning.

CMake: [nfc] remove excess blank line

Squashes a “[C0305] too many newlines between statements” `cmake-lint` warning.

CMake: [nfc] uppercase 'dotlink' variable

Upcoming changes that introduce `cmake-format` checks complain that this
identifier does not comply with `[A-Z][0-9A-Z_]+|_[0-9a-z_]+`

CMake: [nfc] uppercase 'dotcopy' variable

Upcoming changes that introduce `cmake-format` checks complain that this
identifier does not comply with `[A-Z][0-9A-Z_]+|_[0-9a-z_]+`

CMake: [nfc] uppercase 'sources' variable

Upcoming changes that introduce `cmake-format` checks complain that this
identifier does not comply with `[A-Z][0-9A-Z_]+|_[0-9a-z_]+`

CMake: add comments to 'add_custom_command', 'add_custom_target' steps

Squashes “[C0113] Missing COMMENT in statement which allows it” `cmake-lint`
warnings.

CMake: [nfc] uppercase 'create_test' macro

Upcoming changes that introduce `cmake-format` checks complain that this
identifier does not comply with `[0-9A-Z_]+`.

CMake: [nfc] wrap to 80 character column width

Upcoming changes to introduce `cmake-format` checks default to this, which suits
Graphviz well as it matches the (claimed) `clang-format` C/C++ style.

CMake: [nfc] fix a comment typo

CMake: [nfc] standardize on 2-space indentation

Upcoming changes to introduce `cmake-format` checks default to this, which suits
Graphviz well as it matches the (claimed) `clang-format` C/C++ style.

CMake: [nfc] uppercase 'PangoCairo_*' variables

Upcoming changes that introduce `cmake-format` checks complain that these
identifiers do not comply with `[A-Z][0-9A-Z_]+|_[0-9a-z_]+`.

CMake: [nfc] uppercase 'Svgpp_*' variables

Upcoming changes that introduce `cmake-format` checks complain that these
identifiers do not comply with `[A-Z][0-9A-Z_]+|_[0-9a-z_]+`.

CMake: [nfc] uppercase 'RapidXml_*' variables

Upcoming changes that introduce `cmake-format` checks complain that these
identifiers do not comply with `[A-Z][0-9A-Z_]+|_[0-9a-z_]+`.

CMake: [nfc] uppercase 'Getopt_*' variables

Upcoming changes that introduce `cmake-format` checks complain that these
identifiers do not comply with `[A-Z][0-9A-Z_]+|_[0-9a-z_]+`.

CMake: [nfc] uppercase 'Cairo_*' variables

Upcoming changes that introduce `cmake-format` checks complain that these
identifiers do not comply with `[A-Z][0-9A-Z_]+|_[0-9a-z_]+`.

Merge branch 'fix-stdout-and-stderr-flushing-at-exit-on-mingw' into 'main'

Fix stdout and stderr flushing at exit on MinGW

Closes #2178

See merge request graphviz/graphviz!2416

Merge branch 'smattr/B165A187-BED3-4E13-8876-799331BFE43F' into 'main'

remove unused ltmain.sh.patch

See merge request graphviz/graphviz!2420

add a CHANGELOG entry for the stdout/stderr not flushed fix

replace all instances of return from main with graphviz_exit()

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

replace all instances of exit() with graphviz_exit()

Partial fix for https://gitlab.com/graphviz/graphviz/-/issues/2178.

lneato: autotools: add 'lib' to include directories

An upcoming commit need this in order to be able to add an include of
cgraph/exit.h.

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

lefty: autotools: add 'lib' to include directories

An upcoming commit need this in order to be able to add an include of
cgraph/exit.h.

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

dotty: autotools: add 'lib' to include directories

An upcoming commit need this in order to be able to add an include of
cgraph/exit.h.

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

diffimg: autotools: add 'lib' to include directories

An upcoming commit need this in order to be able to add an include of
cgraph/exit.h.

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

lneato: MSBuild: add 'lib' to additional include directories

An upcoming commit need this in order to be able to add an include of
cgraph/exit.h.

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

lefty: MSBuild: add 'lib' to additional include directories

An upcoming commit need this in order to be able to add an include of
cgraph/exit.h.

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

dotty: MSBuild: add 'lib' to additional include directories

An upcoming commit need this in order to be able to add an include of
cgraph/exit.h.

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

diffimg: MSBuild: add 'lib' to additional include directories

An upcoming commit need this in order to be able to add an include of
cgraph/exit.h.

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

sparse: add a graphviz_exit function to general.h if STANDALONE

An previous commit added the same function to cgraph/exit.h, but that
library is not supposed to be used when STANDALONE is defined.

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

sparse: remove useless include of cgraph/sprint.h

The last use of anything from sprint.h was removed in
https://gitlab.com/graphviz/graphviz/-/commit/c2d3526427a7259aa656ab72c83b43232ebfd806.

cgraph: add a graphviz_exit function in a new header file exit.h

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

remove unused ltmain.sh.patch

Use of this patch was dropped in b9a51d568fc4c842b8d4a930c50a655526a08fa7 in
2007.

Merge branch 'smattr/11D3486A-E1F5-44A2-B19C-139B87699C57' into 'main'

suppress XType warnings during testing on macOS

See merge request graphviz/graphviz!2418

suppress XType warnings during testing on macOS

When upgrading from macOS 11 to macOS 12 in CI, running Graphviz outputs:

  dot[32116:62347] XType: com.apple.fonts is not accessible.
  dot[32116:62347] XType: XTFontStaticRegistry is enabled.

This causes some spurious failures in tests that are expecting stderr to be
silent. These warnings do not seem a serious issue, so this change suppresses
them during testing.

Gitlab: #2164

Merge branch 'smattr/A97D3899-3C26-441B-809F-236E5F6E02FF' into 'main'

CI: add Automake installation for macOS

See merge request graphviz/graphviz!2417

CI: add Automake installation for macOS

Automake (and its dependent, Autoconf) are needed for Autotools-based Graphviz
compilation. They are currently available in the CI Gitlab macOS 11 shared
runner environment incidentally due to a transitive dependency via php.¹ The
same is not true in the macOS 12 environment.²

This commit moves to explicitly installing these Graphviz dependencies rather
relying on getting them implicitly. This makes the CI task for the macOS
Autotools build pass on macOS 12.

Gitlab: #2164

¹ https://gitlab.com/gitlab-org/ci-cd/shared-runners/images/macstadium/orka/-/blob/df5b9cad3a482ec8b79b401c8d03bbc60f2f97c4/toolchain/big-sur.yml#L105-111
² https://gitlab.com/gitlab-org/ci-cd/shared-runners/images/macstadium/orka/-/blob/df5b9cad3a482ec8b79b401c8d03bbc60f2f97c4/toolchain/monterey.yml#L48-49

Merge branch 'smattr/3b702435-a919-4479-b146-c17909819e6e' into 'main'

work towards removing FALSE, TRUE

See merge request graphviz/graphviz!2410

lefty: use C99 bools for 'Erun' variables

lefty: use a C99 bool for 'gmapon'

scan_num: use a C99 bool for 'saw_rp'

scan_num: use a C99 bool for 'saw_digit'

lefty boolop: fix incorrect type in return statement

This was accidentally omitted in e5b641c6891e52d503bc63cb9f354b54abcdce8e.

lefty: use C99 bools for 'errdo' globals

lefty: return a C99 bool from 'orderop'

lefty: use C99 bool constants to set 'Gnocallbacks'

SparseMatrix_symmetrize: take a C99 bool instead of an int

SparseMatrix_is_symmetric: take a C99 bool instead of an int

lefty Mhaspointers: use C99 bool values

use C99 bools to set 'onstack'

remove comparisons against 'TRUE' literal

The variables involved here are all booleans or used exclusively as booleans.

remove comparisons against 'FALSE' literal

Merge branch 'smattr/1C578045-99DC-414C-BF4A-C517F2EC40B8' into 'main'

mingle: replace some `unique_ptr` with `vector`

See merge request graphviz/graphviz!2412

mingle bundle: replace a 'unique_ptr' with a 'vector'

Similar to the prior commit, when the `std::unique_ptr` usage was added here in
176c8a163a951a2b801956ed4a00da05603e3a06, I was incorrectly too focused on
preserving the previous property of this being a heap-allocated array. This is
not necessary, and using a `std::vector` instead allows more flexibility. E.g.
depending on the `std::vector` implementation, it could choose to allocate this
array on the stack instead of the heap.

mingle genBundleColors: replace a 'unique_ptr' with a 'vector'

When the `std::unique_ptr` usage was added here in
869891f9c87b326ddef4de461b92de98ec647b82, I was incorrectly too focused on
preserving the previous property of this being a heap-allocated array. This is
not necessary, and using a `std::vector` instead allows more flexibility. E.g.
depending on the `std::vector` implementation, it could choose to allocate this
array on the stack instead of the heap.

Merge branch 'smattr/AC433706-8F92-4102-816C-75433E82B00D' into 'main'

expr: fix misinterpretation of shifts

Closes #2103

See merge request graphviz/graphviz!2409

expr: fix misinterpretation of shifts

It is fairly clear from the surrounding code that these instances were writing
to the wrong destination and, in some cases, reading from the wrong sources. The
effect of this bug was a little slippery as the underlying storage is a union
with `integer` and `floating` occupying the same storage, so this change does
not alter which bytes are stored to. Additionally users rarely left or right
shift within GVPR scripts, so it is unlikely this affected any real world code.

Gitlab: Fixes #2103

expr: remove legacy IBM compiler work around

Despite dredging the commit log, comp.lang, and Stack Overflow, I have been
unable to determine what these comments refer to. They read to me like working
around a register allocator bug. AFAIK every production compiler from the last
decade has been capable of these kind of one-line shifts without work arounds.

Gitlab: #2103

Merge branch 'smattr/D5729AAE-102F-4B2D-8A3A-8E02AD0D9A7A' into 'main'

reformat example code

See merge request graphviz/graphviz!2408

Merge branch 'add-missing-changlog-entry-for-fixed-lneato' into 'main'

add a CHANGELOG entry for lneato fix

See merge request graphviz/graphviz!2415

add a CHANGELOG entry for lneato fix

dot.demo/dot.c: [nfc] align with C99 style and clang-format

dot.demo/demo.c: [nfc] align with C99 style and clang-format

dot.demo/simple.c: [nfc] align with C99 style and clang-format

dot.demo/example.c: [nfc] align with C99 style and clang-format

dot.demo/neatopack.c:[nfc]  align with C99 style and clang-format

Merge branch 'smattr/97832B69-55F8-4C57-9704-188684569B12' into 'main'

remove Lefty from CI and portable source tarball

See merge request graphviz/graphviz!2413