API BREAK: libxdot: use size_t instead of int for op sizes/counts
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 21 Aug 2022 01:58:20 +0000 (18:58 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 24 Aug 2022 00:14:20 +0000 (17:14 -0700)
commit48743a1e619a2043099a7b60eec0dd1c80fc73f7
treede901f45684d31ca7fe80f6d35ee5950a3c31497
parent28e2a7e11654b1d10dd5efd522f4f26ea211fd82
API BREAK: libxdot: use size_t instead of int for op sizes/counts

This allows exceeding operation counts of 2³¹ - 1, which is not particularly
likely. But its more important effect is to make arithmetic operations less
error prone and make it impossible by-construction to arrive at a negative
operation size or count.

Doing this as a single sweeping change seemed less error prone and clearer,
despite how large the resulting diff is.

For lines that would have been changed anyway, this commit also fixes white
space and tightens variable scoping where possible.
CHANGELOG.md
cmd/smyrna/topviewfuncs.c
lib/common/emit.c
lib/xdot/xdot.c
lib/xdot/xdot.h
plugin/core/gvrender_core_json.c