Remove bunch of allocations from the builders

reown-com · Jun 4, 2018 · b209961 · b209961
1 parent b73e09b
commit b209961
Show file tree

Hide file tree

Showing 10 changed files with 280 additions and 264 deletions.
diff --git a/examples/certificate_client.rs b/examples/certificate_client.rs
@@ -70,21 +70,21 @@ fn main() {
     let client = Client::certificate(&mut certificate, &password, endpoint).unwrap();
 
     let options = NotificationOptions {
-        apns_topic: topic,
+        apns_topic: topic.as_ref().map(|s| &**s),
         ..Default::default()
     };
 
     // Notification payload
-    let mut builder = PlainNotificationBuilder::new(message);
+    let mut builder = PlainNotificationBuilder::new(message.as_ref());
     builder.set_sound("default");
     builder.set_badge(1u32);
 
     let payload = builder.build(device_token.as_ref(), options);
+    let sending = client.send(payload);
 
     // Send the notification, parse response
     tokio::run(lazy(move || {
-        client
-            .send(payload)
+        sending
             .map(|response| {
                 println!("Sent: {:?}", response);
             })

diff --git a/examples/token_client.rs b/examples/token_client.rs
@@ -78,21 +78,21 @@ fn main() {
     ).unwrap();
 
     let options = NotificationOptions {
-        apns_topic: topic,
+        apns_topic: topic.as_ref().map(|s| &**s),
         ..Default::default()
     };
 
     // Notification payload
-    let mut builder = PlainNotificationBuilder::new(message);
+    let mut builder = PlainNotificationBuilder::new(message.as_ref());
     builder.set_sound("default");
     builder.set_badge(1u32);
 
     let payload = builder.build(device_token.as_ref(), options);
+    let sending = client.send(payload);
 
     // Send the notification, parse response
     tokio::run(lazy(move || {
-        client
-            .send(payload)
+        sending
             .map(|response| {
                 println!("Sent: {:?}", response);
             })