RTreeInsert: move definitions closer to their use and use better types

reflow RTreeInsert2 prototype

RTreeSearch: push definition of i closer to its use and use a better type

PrintData: push definition of i closer to its use and use a better type

PrintIndex: push definition of i closer to its use and use a better type

RTreeClose2: push definition of i closer to its use

RTreeNewIndex: push definition of x closer to its use

RTreeLeafListFree: push definition of tlp closer to its use

RTreeLeafListAdd: push nlp definition closer to its use

simplify printf format in PrintNode

Under C99, we have %p for pointers.

use more modern #pragma once guards in lib/label

remove unnecessary CoverAll declaration

This symbol is unused in this file.

remove commented out code

remove cryptic comment

I guess this is a hint to some editor I am not aware of.

remove unused parameter to RTreeReInsert

remove unused parameter to objplpmks

remove hand rolled floor(log2())

We do not need a custom optimized version of this. Modern compilers know these
functions as built-ins and can do their own better optimizations.

squash some -Wunused-parameter warnings

remove some open coded fmax

Merge branch 'smattr/450AD1C4-3E9D-4E93-95C6-279FC38EFCA3' into 'main'

some clean up of neatogen

See merge request graphviz/graphviz!1865

replace unnecessary max() implementation with libc fmaxf

No need to hand roll this ourselves now that we compile with C99.

replace neatogen header guards with more modern #pragma once

remove commented out code

remove C++ alternative in embed_graph.h

We only compile this header under C.

remove inline NULL definition

This is guaranteed to exist in C99.

remove C++ alternative in dijkstra.h

Only the C versions of these functions are implemented.

remove incorrect C++ alternative

The '&' is still required, regardless of whether this code is compiled with C or
C++. Note that this issue was latent as we never build this code as C++.

remove unused sparsegraph.h C++ alternative

Similar to previous commits, this code is unused because we do not include it in
any C++ compilation.

remove unused kkutils.h C++ alternative

This code is unused and in fact many of these prototyped functions are
unimplemented.

remove commented out inline keywords

remove some unnecessary __cplusplus guards

This file is C, and we build it as such. It is not expected that anyone will
compile this source as C++.

remove alternative C++ Queue implementation in neatogen

This code was written to provide two versions of the same API, one for C and one
for C++. This is unnecessary because the C API is already usable from C++.
Moreover, one can just use std::queue in C++. This code was not being used as we
only ever #include it from C files, so we just remove it to ease ongoing
maintenance.

remove commented out #include

Merge branch 'smattr/619aeafd-3505-4392-aa61-49cdae0077aa' into 'main'

fix: correctly handle HTML content ending in ]

Closes #1893

See merge request graphviz/graphviz!1859

fix: correctly handle HTML content ending in ]

As described in the source comment in this commit, expat attempts to recognize
CDATA section enders which results in some internal buffering that Graphviz did
not anticipate. Specifically, any HTML content ending in "]" or "]]" would be
parsed with these characters truncated. The next call to expat would flush this
buffered data, resulting in Graphviz believing the next content was malformed.

For example, the content "my label]]]" would be parsed as the text "my label]".
The next call to expat would return "]]" prepended to whatever was then being
parsed.

Fixes #1893.

add a test case for #1893

Merge branch 'smattr/F391EA36-8454-4A32-B815-1FAAEFD4006E' into 'main'

some cgraph clean up

See merge request graphviz/graphviz!1858

reflow some code

condense a malloc;memset

The calloc function does the zeroing for you and can even be more efficient for
large allocations.

use a stronger return type in not_default_attrs

use a stronger return type in has_no_predecessor_below

use a stronger return type in has_no_edges

use a stronger return type in write_node_test

use a stronger return type for node_in_subg

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