]> granicus.if.org Git - graphviz/commit
neatogen mkOverlapSet: return actual point count in 'cntp'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Tue, 6 Dec 2022 05:16:46 +0000 (21:16 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Tue, 6 Dec 2022 05:16:46 +0000 (21:16 -0800)
commit41e939eed8a117b0ec89083f08989c8966ce8304
tree150d0328f3bdf5f42a3d6ae5e11455225ff3d123
parent0801d43e0d210681938cace212d575dab0326ae3
neatogen mkOverlapSet: return actual point count in 'cntp'

For reasons unclear to me, the `mkOverlapSet` function was constructing a set of
points with an implicit initial 0 point and then notifying the caller of a count
1 _less_ than the total point count. There was nothing wrong with this, but it
led to several instances of non-idiomatic follow-on code that had to account for
an array that was actually 1 longer than what its count variable said.
lib/neatogen/constraint.c