Fix issue in reverse_groups method

[LaurenzV]

Turns out I just copied it wrongly nested from harfbuzz... Whoops. 😅

Here is the original method, for reference:

  template <typename FuncType>
  void reverse_groups (const FuncType& group,
		       bool merge_clusters = false)
  {
    if (unlikely (!len))
      return;

    unsigned start = 0;
    unsigned i;
    for (i = 1; i < len; i++)
    {
      if (!group (info[i - 1], info[i]))
      {
	if (merge_clusters)
	  this->merge_clusters (start, i);
	reverse_range (start, i);
	start = i;
      }
    }
    if (merge_clusters)
      this->merge_clusters (start, i);
    reverse_range (start, i);

    reverse ();
  }

[RazrFalcon]

Turns out I just copied it wrongly nested from harfbuzz...

Yeah, if only HB used spaces for indents... I had similar issues as well. Eventually I simply reformatted the whole project during initial porting.

Is this related to #132?

[LaurenzV]

Is this related to #132?

Yes, it fixes that exact issue.

[RazrFalcon]

Thanks!