always use exterior_coords_iter for derived covers

Author: cookiedan42

geo/src/algorithm/covers/mod.rs

@@ -90,7 +90,7 @@ macro_rules! impl_covers_from_intersects {
                     if HasDimensions::is_empty(target) {
                         return false;
                     }
-                    target.coords_iter().all(|pt| self.intersects(&pt))
+                    target.exterior_coords_iter().all(|pt| self.intersects(&pt))
                 }
             }
         )*
@@ -111,7 +111,7 @@ macro_rules! impl_covers_from_intersects {
                     if HasDimensions::is_empty(self) || HasDimensions::is_empty(target){
                         return false;
                     }
-                    target.coords_iter().all(|pt| self.intersects(&pt))
+                    target.exterior_coords_iter().all(|pt| self.intersects(&pt))
                 }
             }
         )*

geo/src/algorithm/covers/rect.rs

@@ -2,28 +2,11 @@ use super::{Covers, impl_covers_from_intersects};
 use crate::GeoNum;
 use crate::{CoordsIter, Intersects, geometry::*};
 
-impl<T> Covers<Polygon<T>> for Rect<T>
-where
-    T: GeoNum,
-{
-    fn covers(&self, rhs: &Polygon<T>) -> bool {
-        rhs.exterior_coords_iter().all(|c| self.intersects(&c))
-    }
-}
-
-impl<T> Covers<MultiPolygon<T>> for Rect<T>
-where
-    T: GeoNum,
-{
-    fn covers(&self, rhs: &MultiPolygon<T>) -> bool {
-        rhs.exterior_coords_iter().all(|c| self.intersects(&c))
-    }
-}
-
 impl_covers_from_intersects!(Rect<T>, [
 Point<T>,MultiPoint<T>,
 Line<T>,
 LineString<T>, MultiLineString<T>,
 Rect<T>, Triangle<T>,
+Polygon<T>,  MultiPolygon<T>,
 Geometry<T>, GeometryCollection<T>
 ]);

geo/src/algorithm/covers/triangle.rs

@@ -2,28 +2,11 @@ use super::{Covers, impl_covers_from_intersects};
 use crate::GeoNum;
 use crate::{CoordsIter, Intersects, geometry::*};
 
-impl<T> Covers<Polygon<T>> for Triangle<T>
-where
-    T: GeoNum,
-{
-    fn covers(&self, rhs: &Polygon<T>) -> bool {
-        rhs.exterior_coords_iter().all(|c| self.intersects(&c))
-    }
-}
-
-impl<T> Covers<MultiPolygon<T>> for Triangle<T>
-where
-    T: GeoNum,
-{
-    fn covers(&self, rhs: &MultiPolygon<T>) -> bool {
-        rhs.exterior_coords_iter().all(|c| self.intersects(&c))
-    }
-}
-
 impl_covers_from_intersects!(Triangle<T>, [
 Point<T>,MultiPoint<T>,
 Line<T>,
 LineString<T>, MultiLineString<T>,
 Rect<T>, Triangle<T>,
+Polygon<T>,  MultiPolygon<T>,
 Geometry<T>, GeometryCollection<T>
 ]);