Merge branch 'b316e0e6-214b-4ae2-ae43-9d3a55c05715' into 'master'

fix glGenTextures failure detection

See merge request graphviz/graphviz!1559

fix handling of glGenTextures failure

According to the OpenGL docs, glGenTextures indicates failure not through
returning a value in its out parameter but rather via the glGetError API.
Moreover it could not actually indicate failure in the way this call site was
expecting because the out parameter is unsigned, not signed. This issue was
identified while investigating the cause of the following compiler warning:

  glcomptexture.c: In function ‘glCompSetAddNewTexture’:
  glcomptexture.c:45:12: warning: comparison of unsigned expression < 0 is
    always false [-Wtype-limits]

    if (t->id < 0)  /*for some opengl based error , texture couldnt be created */
              ^

squash -Wmissing-prototypes warnings for glCompLoadPng

remove call to cairo_image_surface_get_format whose result is unused

squash a -Wconversion warning in glCompCreateFontFile

squash a -Wunused-parameter warning in GetFixedOGLPos

squash unused variable warnings in glcompmouse.c

squash -Wfloat-conversion warning in glprintfglut

remove unused variables in glCompRenderText

Merge branch 're-add-lefty-to-windows-msbuild' into 'master'

Re-add lefty to Windows MSBuild builds

See merge request graphviz/graphviz!1565

Remove surplus characters in previous CHANGELOG.md entry

Update CHANGELOG.md with added lefty to MSBuild builds

Remove 2 -Wundef warnings in lefty.c

It happened to be 2 warnings using autotools on Ubuntu 18.04, but it
could be up to 4.

Add workaround for lefty not built with CMake on Windows

https://gitlab.com/graphviz/graphviz/-/issues/1753

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.