Offset produces incorrect result

[olologin]

Hi, I was trying to update Clipper2 version from revision f9a40f0 to the current head and I see a regression in closed curve offset.
Somehow it misses this small circle in the new result, while it was producing correct result at the revision f9a40f0

Testcase (Please add this to your testcases if you fix this problem):
Two curves in the input, it is one body with hole. With negative offset outer body should become smaller, its hole should become bigger. The issue is that its hole completely disappears now.

TEST(Clipper2Tests, TestOffsets7)
{
    Paths64 subjects = {
        {{508685336, -435806096},
         {509492982, -434729201},
         {509615525, -434003092},
         {509615525, 493372891},
         {509206033, 494655198},
         {508129138, 495462844},
         {507403029, 495585387},
         {-545800889, 495585387},
         {-547083196, 495175895},
         {-547890842, 494099000},
         {-548013385, 493372891},
         {-548013385, -434003092},
         {-547603893, -435285399},
         {-546526998, -436093045},
         {-545800889, -436215588},
         {507403029, -436215588}},
        {{106954765, -62914568},
         {106795129, -63717113},
         {106340524, -64397478},
         {105660159, -64852084},
         {104857613, -65011720},
         {104055068, -64852084},
         {103374703, -64397478},
         {102920097, -63717113},
         {102760461, -62914568},
         {102920097, -62112022},
         {103374703, -61431657},
         {104055068, -60977052},
         {104857613, -60817416},
         {105660159, -60977052},
         {106340524, -61431657},
         {106795129, -62112022}} };

    Clipper2Lib::ClipperOffset offseter(2, 104857.61318750000);
    Clipper2Lib::Paths64 tmpSubject;

    offseter.AddPaths(subjects, Clipper2Lib::JoinType::Round, Clipper2Lib::EndType::Polygon);
    Clipper2Lib::Paths64 paths;
    offseter.Execute(-2212495.6382562499, paths);

    EXPECT_EQ(paths.size(), 2); // Only one curve is in the output at the current trunk, it is wrong.
}

[AngusJohnson]

Replace -

Clipper2/CPP/Clipper2Lib/src/clipper.offset.cpp

Lines 579 to 582 in 466efe7

	// when shrinking, then make sure the path can shrink that far (#593)
	if (group_delta_ < 0 &&
	std::min(path_rect.Width(), path_rect.Height()) < -group_delta_ * 2)
	continue;

with -

// when shrinking, make sure the path can shrink that far (#593)
// but also make sure this isn't a hole which will be expanding (#715)
if ((group_delta_ < 0) &&
  (std::min(path_rect.Width(), path_rect.Height()) < -group_delta_ * 2) &&
    ((Area(path_in) < 0) == group.is_reversed))
      continue;

[olologin]

@AngusJohnson yes, that helps. Thank you!

[AngusJohnson]

It's still not quite right (ie it still needs tweaking), but this shouldn't be problematic as it is now.