examples/utils/winit_app.rs

@@ -116,7 +116,6 @@ where
     }
 
     fn suspended(&mut self, _event_loop: &ActiveEventLoop) {
-        // TODO: Should we run a destruction function for the user?
         let surface_state = self.surface_state.take();
         debug_assert!(surface_state.is_some());
         drop(surface_state);

examples/winit_multithread.rs

@@ -26,7 +26,7 @@ mod ex {
         loop {
             println!("waiting for render...");
             if do_render.recv().is_err() {
-                // Main thread is dead.
+                // The surface state returned from init_surface below is destroyed.
                 break;
             }
 
@@ -82,11 +82,10 @@ mod ex {
                     Arc::new(Mutex::new(surface))
                 };
 
-                // TODO: This sample will now spawn a new thread every time the surface is
-                // (re)created.  We should instead start it only once, and send an `Arc`ed clone of
-                // the surface through do_render?
-
-                // Spawn a thread to handle rendering.
+                // Spawn a thread to handle rendering for this specific surface. The channels will
+                // be closed and the thread will be stopped whenever this surface (the returned
+                // context below) is dropped, so that it can all be recreated again (on Android)
+                // when a new surface is created.
                 let (start_render, do_render) = mpsc::channel();
                 let (render_done, finish_render) = mpsc::channel();
                 println!("starting thread...");