spline: refactor

Generally nothing of importance, just readability.

- There was a particularly scary typo where:

      const unsigned end = std::min(unsigned(value_count), unsigned(p2_x * mult_));

  clamping to value_count doesn't make sense, given arbitrary p2_x.

- Try harder to avoid assigning s->points. Also the overlap
  threshold constant was arbitrarily too high.

- Sort predicate is meant to use strict ordering,
  not partial total order.

0 files changed, 0 insertions, 0 deletions