Merge branch 'smattr/F500167B-4E3B-48CB-B3CC-D74707171300' into 'main'

yet more Pylint warning squashing

See merge request graphviz/graphviz!2136

test_shapes.py: address missing docstrings

rtest.py: remove shadowing of 'TEST'

rtest.py: remove 'OUTDIR' parameter to 'doDiff'

This parameter is available as a global. This change removes a Pylint
redefined-outer-name warning.

rtest.py: remove 'REFDIR' parameter to 'doDiff'

This parameter is available as a global. This change removes a Pylint
redefined-outer-name warning and a Pylint too-many-arguments warning.

rtest.py: remove unused TESTNAME

Every usage of this within this file is actually referring to a closer variable
that shadows this global.

rtest.py: address Pylint missing-function-docstring warnings

shapes.py: remove variable shadowing

Squashes Pylint redefined-outer-name warnings.

Merge branch 'smattr/07ffb8b9-aba1-45d9-a6b4-abf34e9278e4' into 'main'

ellipse.c clean up

See merge request graphviz/graphviz!2125

change ellipse.c #if STANDALONE to #ifdef

From later usage of this macro in the same file, it looks like this is what was
intended. Squashes a -Wundef build warning.

genEllipticPath: remove shadowing of 'etaB' variable

genEllipticPath: remove shadowing of 'dEta' variable

endPath: remove shadowing of global 'path' variable

curveTo: remove shadowing of global 'path' variable

lineTo: remove shadowing of global 'path' variable

moveTo: remove shadowing of global 'path' variable

genEllipticPath: remove shadowing of global 'path' variable

remove some unused struct names

genEllipticPath: remove 'degree' parameter

This function is only ever called with the argument 3.

genEllipticPath: remove 'threshold' parameter

This is always called with the same threshold.

remove now-unused hand rolled boolean type

endPath: remove 'close' parameter

This function is only ever called with the argument true.

genEllipticPath: remove 'isSlice' parameter

This function is only ever called with the argument true.

genEllipticPath: remove unnecessary bracketing

genEllipticPath: use a C99 bool instead of boolean type

Merge branch 'smattr/CC730FC7-8FD1-477A-B0A9-3F40CD40836A' into 'main'

clean up in lib/common

See merge request graphviz/graphviz!2126

findStopColor: remove unnecessary bracketing

gvRenderJobs: remove unnecessary bracketing

style_token: remove unnecessary bracketing

emit_clusters: remove unnecessary bracketing

init_job_viewport: remove unnecessary bracketing

init_job_dpi: remove unnecessary bracketing

init_gvc: remove unnecessary bracketing

xdotBB: remove unnecessary bracketing

expandBB: rephrase open coding to fmax, fmin

emit_edge_label: remove unnecessary bracketing

remove unnecessary void* casts in calls to strdup_and_subst_obj

These pointers implicitly coerce.

emit_begin_edge: remove unnecessary bracketing

emit_edge_graphics: remove unnecessary bracketing

taperfun: remove unnecessary bracketing

bothfunc: remove unnecessary bracketing

nonefunc: remove unnecessary bracketing

revfunc: remove unnecessary bracketing

forfunc: remove unnecessary bracketing

multicolor: remove unnecessary bracketing

emit_attachment: use a more appropriate type for 's' local

emit_node: remove unnecessary bracketing

emit_begin_node: remove unnecessary bracketing

setup_page: remove unnecessary bracketing

emit_xdot: remove unnecessary bracketing

init_job_pagination: use fmax instead of open coding

init_job_pagination: remove unnecessary bracketing

firstlayer: remove unnecessary bracketing

init_layering: remove unnecessary bracketing

map_output_bspline: remove unnecessary bracketing

bisect: remove unnecessary bracketing

isFilled: return a C99 bool instead of int

rename 'ifFilled' to 'isFilled'

From the leading comment, this is what it was intended to be called.

stripedBox: remove unnecessary bracketing

wedgedEllipse: remove unnecessary bracketing

checkClusterStyle: remove unnecessary bracketing

getObjId: remove unnecessary bracketing

layerPagePrefix: remove unnecessary bracketing

colorseg_t: use a C99 bool for 'hasFraction' field

isRect: remove unnecessary bracketing

isRect: use a C99 bool return type instead of boolean

interpretCRNL: use a C99 bool for 'backslash_seen' local

remove some open coded NULLs

parse_style: use a C99 bool for 'in_parens' local

parse_style: use a C99 bool for 'is_first' local

is_style_delim: use a C99 bool return type instead of boolean

nodeIntersect: use a C99 bool for 'explicit' local

emit_edge_graphics: use a C99 bool for 'tapered' local

node_in_box: use a C99 bool return type instead of boolean

clust_in_layer: use a C99 bool return type instead of boolean

edge_in_layer: remove unnecessary bracketing

edge_in_layer: use a C99 bool return type instead of boolean

node_in_layer: remove unnecessary bracketing

node_in_layer: use a C99 bool return type instead of boolean

write_node_test: use a C99 bool return type instead of boolean

write_edge_test: use a C99 bool return type instead of boolean

validpage: remove unnecessary bracketing

validpage: use a C99 bool return type instead of boolean

validlayer: remove unnecessary bracketing

validlayer: use a C99 bool return type instead of boolean

selectedLayer: remove unnecessary bracketing

selectedLayer: use a C99 bool return type instead of boolean

is_natural_number: use more precise type for input parameter

We can be more strict and rigorous with how we deal with strings here.

is_natural_number: use a C99 bool return type instead of boolean

more standard usage of isdigit and friends

remove some commented out code

colorcmpf: remove strcmp micro-optimization

This is unnecessary when using a modern compiler.

colorcmpf: remove unnecessary casts

These pointers implicitly coerce.

colorxlate: remove some unnecessary bracketing

colorxlate: remove strcmp micro-optimization

This is unnecessary when using a modern compiler.

replace header guards in lib/common with more modern #pragma once

remove cryptic globals.h comment

The link in this comment is dead and it is unclear what it is referring to.

colorxlate: squash some -Wswitch-default warnings

colorxlate: squash some -Wconversion warnings

By design, these calculations cannot overflow.

colorCvt: remove commented out code