Fix sorting of saddle connections in byLength() iteration

[saraedum]

It's curious that this test didn't catch this:

    SECTION("We Find the Same Connections if we Iterate By Length") {
      auto count = 0;
      for (auto connection : square->connections().byLength()) {
        if (connection > Bound(bound, 0))
          break;
        count++;
      }
      REQUIRE(count == expected * 8);
    }

The reason is that we tested with bound in powers of two…

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (edfbc0f) 85.37% compared to head (c8ecade) 85.37%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #336   +/-   ##
=======================================
  Coverage   85.37%   85.37%           
=======================================
  Files         107      107           
  Lines        5764     5765    +1     
=======================================
+ Hits         4921     4922    +1     
  Misses        843      843           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.