Merge pull request #9 from kodemartin/8-intoiterator-addressparserres…

…ponse

[#8] Implement IntoIterator for AddressParserResponse

src/address.rs

@@ -23,6 +23,7 @@
 use std::collections::HashMap;
 use std::ffi::{CStr, CString, NulError};
 use std::slice::Iter;
+use std::vec::IntoIter;
 
 use crate::ffi;
 
@@ -40,6 +41,16 @@ impl AddressParserResponse {
     }
 }
 
+impl IntoIterator for AddressParserResponse {
+    type Item = (String, String);
+    type IntoIter = std::iter::Zip<IntoIter<String>, IntoIter<String>>;
+
+    /// Iterates over `(label, token)` pairs by consuming the value.
+    fn into_iter(self) -> Self::IntoIter {
+        self.labels.into_iter().zip(self.tokens.into_iter())
+    }
+}
+
 impl<'a> IntoIterator for &'a AddressParserResponse {
     type Item = (&'a String, &'a String);
     type IntoIter = std::iter::Zip<Iter<'a, String>, Iter<'a, String>>;
@@ -290,10 +301,8 @@ impl From<AddressParserResponse> for ParsedAddress {
     /// Create a new `ParsedAddress` from an `AddressParserResponse`.
     fn from(response: AddressParserResponse) -> Self {
         let mut parsed_address = ParsedAddress::default();
-        for (label, token) in &response {
-            parsed_address
-                .label_to_token
-                .insert(label.clone(), token.clone());
+        for (label, token) in response {
+            parsed_address.label_to_token.insert(label, token);
         }
         parsed_address
     }

tests/address.rs

@@ -5,9 +5,9 @@ use rustpostal::LibModules;
 
 fn assert_actual_eq_expected(address: &str, expected: Vec<(&str, &str)>) {
     let response = rustpostal::address::parse_address(address, None, None).unwrap();
-    let actual: Vec<(&str, &str)> = response
+    let actual: Vec<(&str, &str)> = (&response)
         .into_iter()
-        .map(|(l, t)| (l.as_str(), t.as_str()))
+        .map(|(l, t)| (l.as_ref(), t.as_ref()))
         .collect();
     assert_eq!(actual, expected);
 }