use a stronger return type in irrelevant_subgraph

use a stronger type for root variable

use a stronger type for hasName variable

simplify agcanonhtmlstr implementation

use a stronger type for backslash_pending variable

use a stronger type for needs_quotes variable

condense some boolean code

use a stronger type for a boolean variable

remove some commented out code

remove unused HAVE__BOOL define

Now that we compile with C99, stdbool.h is guaranteed to be available.

remove commented out iconv code

remove unused exstrfree

remove unused AgraphVersion

Merge branch 'smattr/62d61954-d50d-44bc-9fde-6df2d9924c7c' into 'main'

profile target generator

See merge request graphviz/graphviz!1847

add a script for generating graphs to profile

Merge branch 'smattr/338d3e7f-bace-4703-a761-ea022a3aac14' into 'main'

more gvmap cleanup

See merge request graphviz/graphviz!1844

remove unused whatout gvmap variable

remove unused plot_labels

remove unused plot_edges

remove unused plot_points

remove unused plot_polys

remove unused make_map_from_point_groups

remove excess space, bracketing, and commented out code

remove useless use_line parameter in make_map.c

remove useless is_river parameter in make_map.c

rewrite sbuff construction in gvmap to use an agxbuf

This massively simplifies string juggling in gvmap by delegating all of this to
agxbuf. We reduce three function parameters to one, remove manual offset
tracking, remove pre-output length calculation, and all sorts of other
previously-iniline complications.

remove unused edge_bridge_tol parameter to make_map_internal

mark make_map_internal static

This function is not used outside its containing file.

remove unused xcombined parameter to get_polygons

remove unused local in plot_dot_polygons

remove unused GRP_BBOX parameter from get_polygon_solids

remove unused GRP_RANDOM parameter from get_polygon_solids

remove unused poly_groups parameter from get_polygon_solids

remove unused groups parameter from get_polygon_solids

remove unused exclude_random parameter from get_polygon_solids

remove unused bg_color parameter to plot_dot_map

remove unused processing_polygon

remove unused parameter from plot_dot_map

remove unused processing_one_poly

Merge branch 'smattr/0788fe56-dd32-4e9a-b8bf-3371c58ee5d4' into 'main'

Fix for TBbalance attribute code

Closes #1980 and #1339

See merge request graphviz/graphviz!1846

Fix for TBbalance attribute code

As described in #1339 TBbalance is an undocumented graph attribute that is
supposed to cause nodes to "rise" or "fall" to the minimum or maximum rank
possible.  The attached code snippet seems to repair the bug in this code.

Comment from Matthew Fernandez: I committed this, but I don’t understand enough
about the context to judge its correctness. The above is Steve’s description of
the situation.

Fixes #1339 and #1980.

Merge branch 'smattr/59FA12C9-C832-4AC4-ACB6-D82DDDDD6148' into 'main'

fix incorrect reference counting of interned HTML strings

See merge request graphviz/graphviz!1857

move HTML_BIT and CNT_BIT initialization to their definition

The compiler can constant-fold these at compile time, allowing it to then
propagate these values through their uses and completely eliminate them.

fix incorrect reference counting of interned HTML strings

This typo meant that, instead of masking the reference counting bits, this check
was also including the HTML bit (bit 31). As a result, interned HTML strings
would never be freed. This was found by an introduced -Wlogical-op warning when
building an upcoming change with GCC 8.3.0.

Merge branch 'smattr/DAA0EC73-A782-4820-A059-EF0529EB2934' into 'main'

fix: don't run pylint CI job on tags

Closes #1978

See merge request graphviz/graphviz!1860

fix: don't run pylint CI job on tags

383ec8753b8007d26fc4f6eb51c61b6d8b71d12e added a pylint task in CI. This task is
unrestricted, so Gitlab thinks it is meant to run on tags as well as on master.
Every other task is restricted to only run on master (including the dependencies
of the pylint job) so CI now fails on tags. This change limits the lint job to
not run on tags. Fixes #1978.

Merge branch 'smattr/db8f6c58-e183-4b46-a760-45021fce63e6' into 'main'

more clean up and compiler warning removal

See merge request graphviz/graphviz!1856

remove a strcmp micro-optimization

rephrase unnecessarily cryptic bitwise operations

This is just doing boolean logic. Sometimes there is a motivation to use bitwise
operators in place of boolean operators to optimize hot paths that are
negatively affected by the stalling semantics of boolean shortcut logic. But
this is not the case here. This is not a hot path.

remove commented out code

remove some unnecessary casts

clean up some awkward manual scoping

C99 lets us just declare the index scoped to the for loop itself.

remove some unnecessary bracketing

remove some useless boolean logic

remove some commented out code

remove an open coded NULL

abbreviate some accumulated operations

stronger types for set_attrwf parameters

This squashes some -Wconversion warnings.

remove some excess bracketing

unify an allocation path

realloc(NULL, ...) is equivalent to malloc().

stronger types for startgraph parameters

This suppresses two -Wconversion warnings.

remove some cryptic comments

use C99 initializers for some objects

This squashes a number of -Wmissing-field-initializer warnings.

Merge branch 'smattr/90b1c88b-9fb0-42ad-bab4-5e3e7ad145d9' into 'main'

squash some compiler warnings

See merge request graphviz/graphviz!1845

remove some -Wsign-conversion warnings

This one expression was responsible for 4 compiler warnings, which are now dealt
with.

remove some -Wsign-compare warnings

suppress some -Wunused-parameter warnings

remove some unnecessary bracketing

suppress some -Wunused-parameter warnings

remove some unnecessary casts

These pointers coerce implicitly.

remove an unnecessary use of MIN

remove unnecessary guard to free() call

It’s safe to free NULL.

squash -Wsign-compare warnings in Pango plugin

remove unused get_font_list

Merge branch 'smattr/C0B831E7-0AA4-4A1E-B32C-1E0F8265FF88' into 'main'

add DEVELOPERS.md note about release blockers

See merge request graphviz/graphviz!1855

add DEVELOPERS.md note about release blockers

Merge branch 'smattr/d4e26fd5-fe22-4200-b75a-f0fd591fb919' into 'main'

remove remaining references to the DIA format

Closes #689

See merge request graphviz/graphviz!1861

remove remaining references to the DIA format

Core support for this was removed in 2009. A partial plugin for this was written
but never matured. This was removed in 5243eaac9cbe134fc7935a8c029860632e19d523.
This change removes the final dangling references to DIA. Fixes #689.

Merge branch 'smattr/92E357BD-EBAC-4152-BA3C-87069D8131C5' into 'main'

change references to 'master' branch to 'main'

See merge request graphviz/graphviz!1864

change references to 'master' branch to 'main'

To move towards using more inclusive terminology, this removes references to the
“master” branch of the repository and replaces them with references to “main.”
At time of writing, master is still the default branch of the Graphviz
repository but after merging this commit the default branch can be changed to
main.

Merge branch 'smattr/DEA0397C-9C4C-458C-BD45-236AD5A1CAAA' into 'master'

remove unused HAVE_ARGZ_APPEND

See merge request graphviz/graphviz!1843

remove unused HAVE_ARGZ_APPEND

It is unclear where this originated from. It was added in the original commit of
this file but, at the time, the Autotools build system was not generating this
variable. The argz functionality seems related to the (now removed) dependency
RxSpencer, but Graphviz never used this and never checked whether it was
available.

Merge branch 'grammar-align' into 'master'

Replace deprecated `<TD ALIGN>` attr with CSS

See merge request graphviz/graphviz!1863

Replace deprecated `<TD ALIGN>` attr with CSS

Fixes an HTML validation warning downstream.

Merge branch 'shapes-md' into 'master'

Update mkcolors.awk to output less-deprecated HTML

See merge request graphviz/graphviz!1862

Update mkcolors.awk to output less-deprecated HTML

Allows removing some warnings about bgcolor and font tags.

Merge branch 'smattr/f628b6fb-27de-42a8-a29f-e0c8db438306' into 'master'

make PIC plugin generate valid code

Closes #131

See merge request graphviz/graphviz!1841

fix: emit Bézier curves as point-by-point spline shapes in PIC plugin

With this change, the PIC plugin is finally usable. Bézier curves were being
emitted as a sequence of space-separated numbers. Similar to the previous
situation with boxes, I’m not sure what the intent of this was. This is invalid
PIC syntax. As of this commit, the PIC plugin produces output that is accepted
by GNU PIC. Fixes #131.

remove an intermediate buffer in PIC plugin

The values being printed to the intermediate buffer can just be emitted directly
into the final output stream.

fix: print a point array as a point-by-point box in PIC back end

The existing code was emitting the points as a space-separated list of numbers.
It’s not clear what this was attempting to achieve. This is not a valid PIC
command. This change causes this to emit a sequence of line commands to draw the
described box. Related to #131.

fix incorrect macro reference in PIC plugin

This looks once again copied from the core PIC implementation
(lib/common/picgen.c) without accounting for the fact the core implementation’s
attrs macros are generated dynamically. The plugin generates a single macro,
attrs0, which it was incorrectly calling as attrs1. Related to #131.

fix comment used in PIC files by plugin back end

The PIC plugin seems to have copied some details from the core PIC
implementation (lib/common/picgen.c) without noticing that the core
implementation juggles its comment format to emit PIC comments on pages and only
use TROFF comments at the top level. Related to #131.

fix CHANGELOG entry incorrectly added to old version

Merge branch 'smattr/3178EBB6-E879-4846-8F8B-23D5431F7D2C' into 'master'

remove superfluous empty @param in doxygen comments

Closes #1977

See merge request graphviz/graphviz!1840

remove superfluous empty @param in doxygen comments

Fixes #1977.

Merge branch 'shapes-md' into 'master'

Remove template shapes.html.j2

See merge request graphviz/graphviz!1854

Remove template shapes.html.j2

It's all been migrated to the graphviz.gitlab.io repo, so we can retire this python code.

Merge branch 'smattr/24836a24-1f74-4195-aaab-01112907fd7a' into 'master'

document missing libxdot API functions

Closes #1957

See merge request graphviz/graphviz!1839

document libxdot JSON functionality

Closes #1957.

document parseXDotFOn in the libxdot API

Related to #1957.