impl Clone for Timer; make Timer::cancel take &self

src/app.rs

@@ -4,14 +4,15 @@ use std::{fmt, result};
 
 use crate::{backend, IntoInnerError, Result};
 
+#[derive(Clone)]
 pub struct Timer {
     inner: backend::TimerInner,
     // ensure !Send and !Sync on all platforms
     _marker: PhantomData<*mut ()>,
 }
 
 impl Timer {
-    pub fn cancel(self) {
+    pub fn cancel(&self) {
         self.inner.cancel();
     }
 }

src/backend/cocoa/timer.rs

@@ -126,13 +126,14 @@ impl Drop for Timers {
     }
 }
 
+#[derive(Clone)]
 pub struct TimerInner {
     app_state: Weak<AppState>,
     timer: CFRunLoopTimerRef,
 }
 
 impl TimerInner {
-    pub fn cancel(self) {
+    pub fn cancel(&self) {
         if let Some(app_state) = self.app_state.upgrade() {
             if app_state.timers.timers.borrow_mut().remove(&self.timer) {
                 unsafe {

src/backend/win32/timer.rs

@@ -85,13 +85,14 @@ impl Timers {
     }
 }
 
+#[derive(Clone)]
 pub struct TimerInner {
     app_state: Weak<AppState>,
     timer_id: usize,
 }
 
 impl TimerInner {
-    pub fn cancel(self) {
+    pub fn cancel(&self) {
         if let Some(app_state) = self.app_state.upgrade() {
             if let Some(_) = app_state.timers.timers.borrow_mut().remove(&self.timer_id) {
                 unsafe {

src/backend/x11/timer.rs

@@ -125,13 +125,14 @@ impl Timers {
     }
 }
 
+#[derive(Clone)]
 pub struct TimerInner {
     app_state: Weak<AppState>,
     timer_id: TimerId,
 }
 
 impl TimerInner {
-    pub fn cancel(self) {
+    pub fn cancel(&self) {
         if let Some(app_state) = self.app_state.upgrade() {
             app_state.timers.timers.borrow_mut().remove(&self.timer_id);
         }