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

rgb2cmyk: call fmin instead of open coding floating point comparison

replace use of MIN, MAX macros in colxlate.c with standard fmin, fmax

canontoken: use char pointers instead of unsigned char pointers

It is not clear to me why this code was using unsigned char pointers.

canontoken: remove commented out code

hsv2rgb: remove unnecessary bracketing

remove unnecessary casts around usage of 'bsearch'

colorxlate: remove unnecessary bracketing

colorxlate: remove a strcmp micro-optimization

This is unnecessary when using modern compilers.

colorxlate: use char pointers instead of unsigned char pointers

It is not clear to me why this code was using unsigned char pointers.

Merge branch 'smattr/7e2248f2-8a26-4cfa-9908-b5d191b0070e' into 'main'

fix various -Wmissing-field-initializer warnings

See merge request graphviz/graphviz!2130

p_plain: squash -Wmissing-field-initializer warning

p_plaintext: squash -Wmissing-field-initializer warning

p_square: squash -Wmissing-field-initializer warning

p_box: squash -Wmissing-field-initializer warning

p_triangle: squash -Wmissing-field-initializer warning

p_egg: squash -Wmissing-field-initializer warning

p_circle: squash -Wmissing-field-initializer warning

p_ellipse: squash -Wmissing-field-initializer warning