Don't clone the context

serenity-rs · Jan 28, 2017 · 5ee5fef · 5ee5fef
1 parent b7cbf75
commit 5ee5fef
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/client/dispatch.rs b/src/client/dispatch.rs
@@ -66,13 +66,13 @@ pub fn dispatch(event: Event,
             let mut framework = framework.lock().unwrap();
 
             if framework.initialized {
-                dispatch_message(&context,
+                dispatch_message(context.clone(),
                                  event.message.clone(),
                                  event_store);
 
                 framework.dispatch(context, event.message);
             } else {
-                dispatch_message(&context, event.message, event_store);
+                dispatch_message(context, event.message, event_store);
             }
         },
         other => handle_event(other, conn, data, login_type, event_store),
@@ -91,20 +91,18 @@ pub fn dispatch(event: Event,
                                   conn,
                                   data,
                                   login_type);
-            dispatch_message(&context,
+            dispatch_message(context,
                              event.message,
                              event_store);
         },
         other => handle_event(other, conn, data, login_type, event_store),
     }
 }
 
-fn dispatch_message(context: &Context,
+fn dispatch_message(context: Context,
                     message: Message,
                     event_store: &Arc<RwLock<EventStore>>) {
     if let Some(handler) = handler!(on_message, event_store) {
-        let context = context.clone();
-
         thread::spawn(move || (handler)(context, message));
     }
 }