Bump i_overlay dependency to 3.4.1

geo/Cargo.toml

@@ -31,7 +31,7 @@ proj = { version = "0.30.0", optional = true }
 robust = "1.1.0"
 rstar = "0.12.0"
 serde = { version = "1.0", optional = true, features = ["derive"] }
-i_overlay = { version = "2.0.0, < 2.1.0", default-features = false }
+i_overlay = { version = "3.4.1", default-features = false }
 
 [dev-dependencies]
 approx = ">= 0.4.0, < 0.6.0"

geo/src/algorithm/bool_ops/i_overlay_integration.rs

@@ -47,11 +47,8 @@ pub(super) mod convert {
 
     pub fn polygon_from_shape<T: BoolOpsNum>(shape: Vec<Vec<BoolOpsCoord<T>>>) -> Polygon<T> {
         let mut rings = shape.into_iter().map(|path| {
-            // From i_overlay: > Note: Outer boundary paths have a clockwise order, and holes have a counterclockwise order.
-            // Which is the opposite convention we use.
             let mut line_string = line_string_from_path(path);
             line_string.close();
-            line_string.0.reverse();
             line_string
         });
         let exterior = rings.next().unwrap_or(LineString::new(vec![]));

geo/src/algorithm/bool_ops/mod.rs

@@ -184,9 +184,9 @@ pub fn unary_union<'a, B: BooleanOps + 'a>(
         .collect::<Vec<_>>();
 
     let fill_rule = if winding_order == Some(WindingOrder::Clockwise) {
-        FillRule::Positive
-    } else {
         FillRule::Negative
+    } else {
+        FillRule::Positive
     };
 
     let shapes = FloatOverlay::with_subj(&subject).overlay(OverlayRule::Subject, fill_rule);