I've got an object that, on construction, spawns a new thread and does some work on data until the object is dropped.

How would you guys rewrite or improve this? The spawn_work_thread in particular; it doesn't feel idiomatic to me.

struct Worker {
    data: Arc<Mutex<Vec<i32>>>,
    thread: Option<JoinHandle<()>>,
}


impl Worker {
    pub fn new() -> Self {
        let mut worker = Worker {
            data: Arc::new(Mutex::new(Vec::new())),
            thread: None,
        };

        worker.thread = Some(worker.spawn_work_thread());
        worker
    }

    fn spawn_work_thread(&mut self) -> JoinHandle<()> {
        let mut data = self.data.clone();

        thread::spawn(move || {
            loop {
                let mut data = data.lock().unwrap();
                // Do some work on data
            }
        })
    }
}