Add test of lefty usage

Fix lefty crash at exit in MSBuild debug build

Final commit that fixes
https://gitlab.com/graphviz/graphviz/-/issues/1818.

Allow lefty -? to show usage even if there's no DISPLAY (FEATURE_X11)

Parse lefty args before starting app and trying to open DISPLAY. This
allows testing that lefty can start in CI.

This change only affects the X11 variant (FEATURE_X11 or FEATURE_NONE
defined), not the Windows variant.

Correct lefty output directory for MSBuild

Fix lefty compile error on Windows

The FILE type, a.k.a. struct _iobuf does not have any _cnt field on
Windows. Setting HAVE_FILE_CNT to 0 avoids the use of that. None of
the other implementation alternatives work either so the canread macro
in lefty.c always returns 1.

Corrected name of HAVE_FILE_R macro in lefty

Changing it to HAVE_R_IN_FILE was clearly a mistake in commit
070479236d25977cde7d8306635695af2fb7c203.

From what I can tell this currectly does not affect anything.

Remove BuildLog.htm item from MSBuild project files

This file is not found after an upgrade to Visual Studio 16.7.2. It
was automatically added in a prevous upgrade of Visual Studio more
than 4 years ago in 3c8cf3624f6a8b82434115177471b6e7092f836.

Re-add lefty to graphviz solution

Retarget lefty to Visual Studio 2019 and toolset v142

Merge branch 'fix-gvedit-usage' into 'master'

Fix gvedit usage + misc cleanup of gvedit make

Closes #1813

See merge request graphviz/graphviz!1561

Unset the DISPLAY environment variable in gvedit usage test_1813

This ensures that the test fails instead of hanging if print usage and
exit fails and the program continues and tries to open the display.

Avoid unnecessary recompile of gvedit mocables

Reverts bb9a819a64de7f8446f17ecdb734aaf0f811aa04.

Remove 2 gvedit warnings about multiple recepies

Remove unnecessary gvedit executable dependency on $(MOCABLES)
alltogether.

Add temporary skip of test_1813 on Windows

The test is skipped because of
https://gitlab.com/graphviz/graphviz/-/issues/1816.

Add gvedit -? regression test

Add rpm install of graphviz-qt to ci/install-packages.sh

Contains gvedit.

Allow gvedit -? to show usage even if there's no DISPLAY

Parse gvedit args before starting app and trying to open DISPLAY. This
allows testing that gvedit can start in CI.

Update CHANGELOG.md with gvedit -? fix

Fix gvedit -? to actually print usage instead of error

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

Remove TAB from otherwise empty line in gvedit Makefile.am

Emacs said "Suspicious line 71. Save anyway? (y or n)" every time the
file was about to be saved.

Merge branch 'remove-uncompiled-code' into 'master'

Remove uncompiled code to increase readability

See merge request graphviz/graphviz!1557

Merge branch '1785-1' into 'master'

disambiguate #includes

See merge request graphviz/graphviz!1555

remove commented out #include

disambiguate lib/xdot/*.c #includes

Related to #1785.

disambiguate lib/vpsc/*.cpp #includes

Related to #1785.

disambiguate lib/vmalloc/*.c #includes

Related to #1785.

disambiguate lib/twopigen/*.c #includes

Related to #1785.

remove unnecessary lib/twopigen include directory

diambiguate lib/topfish/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/topfish include directory

disambiguate lib/spine/*.c #includes

Related to #1785.

remove unnecessary lib/spine include directory

disambiguate lib/sparse/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/sparse include directory

dismbiguate lib/sfdpgen/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/sfdpgen include directory

disambiguate lib/rbtree/*.c #includes

Related to #1785.

remove unnecessary lib/rbtree include directory

disambiguate lib/pathplan/*.c #includes

Related to #1785.

disambiguate lib/patchwork/*.c #includes

Related to #1785.

remove unnecessary lib/patchwork include directory

disambiguate lib/pack/*.c #includes

Related to #1785.

remove unnecessary lib/pack include directory

disambiguate lib/osage/*.c #includes

Related to #1785.

remove unnecessary lib/osage include directory

disambiguate lib/ortho/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/ortho include directory

disambiguate lib/neatogen/*.c #includes

Related to #1785.

remove unnecessary lib/neatogen include directory

disambiguate lib/mingle/*.c #includes

Related to #1785.

remove unnecessary lib/mingle include directory

disambiguate lib/label/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/label include directory

disambiguate lib/inkpot/*.c #includes

Related to #1785.

disambiguate lib/ingraphs/*.c #includes

Related to #1785.

remove unnecessary lib/ingraphs include directory

disambiguate lib/gvpr/*.c #includes

Related to #1785.

further disambiguate inclusion of colortbl.h

This generated header needs some extra acrobatics. Related to #1785.

remove unnecessary lib/gvpr include directory

disambiguate lib/gvc/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/gvc include directory

disambiguate lib/glcomp/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/glcomp include directory

disambiguate lib/fdpgen/*.c #includes

Related to #1785.

remove unnecessary lib/fdpgen include directory

disambiguate lib/expr/*.c #includes

Related to #1785.

remove unnecessary lib/expr include directory

disambiguate lib/edgepaint/*.c #includes

Related to #1785.

remove unnecessary lib/edgepaint include directory

disambiguate lib/dotgen/*.c #includes

Related to #1785.

remove unnecessary lib/dotgen include directory

disambiguate lib/common/*.c #includes

Related to #1242 and #1785.

remove unnecessary lib/common include directory

disambiguate lib/circogen/*.c #includes

Related to #1785.

remove unnecessary lib/circogen include directory

include internal headers with <...> in lib/ast/*.c

It turns out to deal with #1785, we will need to include internal headers this
way as well as external headers. 2ecb837a05be3d69971001708e7acccfd470ad97 should
have gone in this direction to begin with.

make lib/cgraph/*.c #includes unambiguous

Related to #1785.

remove unnecessary cgraph #include path

remove #define EXTERN trick in cgraph

This was designed to allow one declaration of Ag_G_global serve as both
declaration and definition. However, this obscures the actual definition of this
variable from any code browser unable to evaluate macro expansion (e.g. Cscope).
There's no real need for this trick, as we can just define the variable as
normal.

disambiguate lib/cdt/*.c #includes

Related to #1785.

remove unused cdt 'Version'

Merge branch '3a0128e0-8665-418d-b92d-642f114890ce' into 'master'

remove unused SFIO 'disciplines'

See merge request graphviz/graphviz!1556

remove unused SFIO 'disciplines'

Merge branch 'fcec7644-4a9d-47cb-ba10-81305417378d' into 'master'

remove ad hoc name mangling for lib/expr/grammar.y

See merge request graphviz/graphviz!1553

remove unnecessary intermediate build artifact

This is the equivalent of c498c4bb3e12e48e3bab9aba8d3162535e163c92 for
exparse.y. Related to #1806.

use Bison's api.prefix instead of name mangling in expr

This is the equivalent of cc2cb52756f545d6863d6fd801fa23762a81704c. My original
intent was to split the changes that remove the build-time symbol mangling (the
equivalent of 9240da12ab7857975aa34279520f1f6aad113d48) into a separate commit.
However, the expression parser does a lot of messy tricks to collide with and
suppress Bison-generated macros. It was simpler to undo all of this in one go
then try to thread this convoluted needle. Related to #1806.

explicitly name output of lib/expr/exparse.y rule

This has no effect right now, but an upcoming change modifies api.prefix which,
without this, would alter the name of the generated parser. Related to #1806.

explicitly drop Yacc, Bison < 3 compatibility in exparse.y

Yacc was already not supported as this parser is using Bison extensions, but
this at least makes explicit what our requirements are. This is the equivalent
of 4a13c9598ad3d950a251e5d3e5be8d05c59c6214. Related to #1806.

suppress warnings from using Bison extensions in lib/expr/exparse.y

This is the equivalent of 8c099fc032074a0bb5961d67ef4df05ddca7eba4 but for a
different parser. Surprisingly exparse.y was already using Bison extensions, so
this actually suppresses two warnings:

  bison -y -dtv ../../lib/expr/exparse.y
  ../../lib/expr/exparse.y:131.1-7: warning: POSIX Yacc does not support %binary [-Wyacc]
   %binary       <op>    EQ      NE
   ^~~~~~~
  ../../lib/expr/exparse.y:132.1-7: warning: POSIX Yacc does not support %binary [-Wyacc]
   %binary       <op>    '<'     '>'     LE      GE
   ^~~~~~~

Related to #1806.

remove incorrect EXSTYPE redefinition

Bison's -pPREFIX -- which, incidentally, this parser is not using -- does *not*
miss YYSTYPE. Upcoming changes expose that this detrimentally interferes with
attempts to properly prefix the expr parser. Related to #1806.

Merge branch 'remove-warnings-related-to-issue-1800' into 'master'

Remove warnings related to issue 1800

See merge request graphviz/graphviz!1554

Remove 1 -Wswitch-default warning in attr.c

Remove 1 -Wunused-but-set-variable warning in attr.c

Remove 2 -Wsign-conversion warnings in attr.c

Remove 1 -Wconversion warning in attr.c

Remove 4 -Wmissing-prototypes warnings in attr.c by adding static

Remove 2 -Wmissing-field-initializers warnings in attr.c

Remove 1 -Wswitch-default warning in agdelrec

Remove 3 -Wunused-parameter warnings in rec.c

Remove 2 -Wconversion warnings in set_data