Off-by-one in ot_shape_normalize?

[simoncozens]

I just noticed this:

harfrust/src/hb/ot_shape_normalize.rs

Lines 418 to 427 in 0dd7b14

	// We are going to do a O(n^2). Only do this if the sequence is short.
	if end - i <= MAX_COMBINING_MARKS {
	buffer.sort(i, end, compare_combining_class);
	if let Some(reorder_marks) = ctx.plan.shaper.reorder_marks {
	reorder_marks(ctx.plan, buffer, i, end);
	}
	}
	i = end + 1;

where harfbuzz has:

      /* We are going to do a O(n^2).  Only do this if the sequence is short. */
      if (end - i > HB_OT_SHAPE_MAX_COMBINING_MARKS) {
        i = end;
        continue;
      }


      buffer->sort (i, end, compare_combining_class);


      if (plan->shaper->reorder_marks)
        plan->shaper->reorder_marks (plan, buffer, i, end);


      i = end;

[behdad]

I think HR uses a while, while HB uses a for. No?