My bad on the previous changes on monday -- I forgot to re-run the unit tests. Tested both manually, and auto, and also with the ifdefs for interactive debugging on.
I tested this in vertex/edge modes and it produces results that are (to me at least) intuitive, at least in the cases I tried. Do you have an example of it misbehaving to point me to what I was…
6fd0d448 I did the C style brace initializer, since all zeros is correct, and it was less overall code than switching to a class and writing a proper constructor. Let me know if you prefer it as…
e39ced2d Let me know if this wording is better. Tried to make it simpler and let the inline comments in the code fill in the details.
e1caee28 Silly mistake, sorry. I forgot to re-run the unit tests after I did some rebasing and fixed a merge conflict the wrong way.
f868bad3
I spent a while looking at this and can't come up with a good reason not to allow a true zero improvement. The minimum is removed and the test just says, "If it's less than zero, make…