simplify addEdgePoint with agxbprint

simplify addNodeGraphics with agxbprint

simplify addEdgeLabelGraphics with agxbprint

use agxbprint in genBundleColors

use agxbprint in genBundleInfo

use agxbprint in deparseAttr

use agxbprint in genBundleSpline, removing the need for a temporary buffer

implement a printf-like interface to agxbuf

Merge branch '66384902-699A-410B-96FF-8DA8ACB35B15' into 'master'

remove sfmutex() stub

See merge request graphviz/graphviz!1567

remove sfmutex() stub

This function is unused, but more problematically it gives the caller the
impression of providing a mutual exclusion primitive while being a no-op. We
remove it to avoid future confusion or accidental usage.

Merge branch '1785-2' into 'master'

disambiguate #includes (2/5)

See merge request graphviz/graphviz!1566

disambiguate cmd/tools #includes

Related to #1785.

remove unnecessary cmd/tools include path

disambiguate cmd/smyrna #includes

Related to #1242 and #1785.

remove unnecessary cmd/smyrna include path

disambiguate cmd/mingle #includes

Related to #1785.

remove unnecessary cmd/mingle include path

disambiguate cmd/gvpr #includes

Related to #1785.

remove unnecessary cmd/gvpr include path

disambiguate cmd/gvmap #includes

Related to #1785.

remove unnecessary cmd/gvmap include path

disambiguate cmd/gvedit #includes

Related to #1785.

disambiguate cmd/edgepaint #includes

Related to #1785.

remove unnecessary cmd/edgepaint include path

remove unnecessary cmd/dot include path

disambiguate cmd/dot #includes

Related to #1785.

Merge branch '678658f8-13b2-48cc-a8fc-ecd7765ecf05' into 'master'

remove ad hoc name mangling in HTML parser

See merge request graphviz/graphviz!1564

remove unnecessary intermediate files when generating the HTML parser

Related to #1806.

remove now-unnecessary ad hoc replacement of HTML parser prefix

Related to #1806.

use api.prefix to set HTML parser's prefix

Related to #1806.

explicitly set the output of HTML parser rule

This has no effect for now, but will once we set api.prefix. Related to #1806.

drop compatibility with Yacc, Bison < 3 in HTML parser

This does not change build requirements, as other parsers already require Bison
3. Related to #1806.

suppress Bison warnings from using non-Yacc compatible extensions in HTML parser

We do not actually use any of these yet, but we are about to. Related to #1806.

Merge branch '2F41B940-1933-48AE-84D5-A31781F6D8C9' into 'master'

suppress Bison warnings about Yacc incompatibility in Windows builds

See merge request graphviz/graphviz!1563

suppress Bison warnings about Yacc incompatibility in Windows builds

Merge branch '3515b572-982c-461f-a4d4-21e7231678f1' into 'master'

remove some compiler warnings

See merge request graphviz/graphviz!1562

remove a couple of sign comparison warnings

squash a -Wundef warning

remove unused height variable from selfRight

remove unused height variable from selfRight

remove unused width variable from selfTop

remove unused width height variable from selfBottom

fix: call printTok when DEBUG is defined in HTML lexer

Based on the prior `#ifdef DEBUG` in this file, it seems debugging is meant to
be enabled whenever DEBUG is defined, regardless of its value.

squash some -Wsign-compare warnings

squash -Wempty-body warnings from expanding trace() inside a conditional body

squash some tolower/toupper conversion warnings

remove unused tritop variable

Merge branch '5f9243f8-2a80-456a-8cf6-4659eb7f13e9' into 'master'

fix failure to find Cairo, PangoCairo under CMake < 3.12

Closes #1824

See merge request graphviz/graphviz!1575

fix failure to find Cairo, PangoCairo under CMake < 3.12

The changes introduced in 3ecf2f6206a486fecdea870b7f29ea539345642a and
b4f998c133566ef9cb416c085d1ce49e539bec30 were accidentally assuming the behavior
of PkgConfig that is only true after CMake 3.12. Namely that it sets the
*_LINK_LIBRARIES variable for packages it discovers. Closes #1824.

Merge branch 'add-metrics-report-with-warnings-to-ci-also-for-windows' into 'master'

Add a metrics report with warnings to CI also for Windows

See merge request graphviz/graphviz!1570

Merge branch 'add-ubuntu20.04-debug-build' into 'master'

Add ubuntu20.04-debug-build to CI

See merge request graphviz/graphviz!1568

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

Add a metrics report with warnings to CI also for Windows

Add ubuntu20.04-debug-build to CI

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.