feat(metadata): remove manual Send + Sync impls for metadata types (#…

…640)

tonic/src/metadata/map.rs

@@ -1244,9 +1244,6 @@ impl<'a> Iterator for Iter<'a> {
  }
 }
 
-unsafe impl<'a> Sync for Iter<'a> {}
-unsafe impl<'a> Send for Iter<'a> {}
-
 // ===== impl IterMut =====
 
 impl<'a> Iterator for IterMut<'a> {
@@ -1274,9 +1271,6 @@ impl<'a> Iterator for IterMut<'a> {
  }
 }
 
-unsafe impl<'a> Sync for IterMut<'a> {}
-unsafe impl<'a> Send for IterMut<'a> {}
-
 // ===== impl ValueDrain =====
 
 impl<'a, VE: ValueEncoding> Iterator for ValueDrain<'a, VE> {
@@ -1293,9 +1287,6 @@ impl<'a, VE: ValueEncoding> Iterator for ValueDrain<'a, VE> {
  }
 }
 
-unsafe impl<'a, VE: ValueEncoding> Sync for ValueDrain<'a, VE> {}
-unsafe impl<'a, VE: ValueEncoding> Send for ValueDrain<'a, VE> {}
-
 // ===== impl Keys =====
 
 impl<'a> Iterator for Keys<'a> {
@@ -1425,9 +1416,6 @@ where
  }
 }
 
-unsafe impl<'a, VE: ValueEncoding> Sync for ValueIterMut<'a, VE> {}
-unsafe impl<'a, VE: ValueEncoding> Send for ValueIterMut<'a, VE> {}
-
 // ===== impl Entry =====
 
 impl<'a, VE: ValueEncoding> Entry<'a, VE> {
@@ -2725,4 +2713,18 @@ mod tests {
  }
  assert!(found_x_word_bin);
  }
+
+ #[allow(dead_code)]
+ fn value_drain_is_send_sync() {
+ fn is_send_sync<T: Send + Sync>() {}
+
+ is_send_sync::<Iter<'_>>();
+ is_send_sync::<IterMut<'_>>();
+
+ is_send_sync::<ValueDrain<'_, Ascii>>();
+ is_send_sync::<ValueDrain<'_, Binary>>();
+
+ is_send_sync::<ValueIterMut<'_, Ascii>>();
+ is_send_sync::<ValueIterMut<'_, Binary>>();
+ }
 }