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

Remove 1 -Wswitch-default warning in agbindrec by removing redundant switch

Remove 1 -Wconversion warnings in cccomps

Remove 2 -Wsign-conversion warnings in cccomps

Remove 3 -Wsign-conversion warnings in ccomps

Remove 3 -Wsign-conversion warnings in pccomps

Remove 2 -Wconversion warnings in setPrefix

Remove 1 -Wconversion warning in clMarkFn

It's safe to cast to char since the value is always 0 or 1.

Remove 1 -Wconversion warning in markFn

It's safe to cast to char since the value is always 0 or 1.

Remove 1 -Wbad-function-cast warning in agdelsubg

Restores the public API to what's documented in cgraph.3 by instead of
casting the void pointer that dtdelete returns to long, returns an int
which is 0 if the pointer is NULL and 1 otherwise. Internally the
return value is however not used today.

Remove 1 -Wbad-function-cast warning in agdtdelete

Changes the *internal* API by instead of casting the void pointer that
dtdelete returns to long, returns an int which is 0 if the pointer is
NULL and 1 otherwise. It doesn't really matter though, since the
return value is not used anywhere today.

Remove 1 -Wunused-parameter warning in agrefstrdump when -DDEBUG

Remove 1 -Wconversion warning in aghtmlstr

Remove 9 -Wsign-conversion warnings in _DTCMP

Casting to size_t is safe since sz is always positive when it happens.

Remove 1 -Wunused-parameter warning in idopen

Remove 1 -Wconversion warning in idalloc

It's safe to cast to int from long since the value is always FALSE (0).