Move thread::spawn to separate function

[mgeisler]

This might make it clearer why the thread cannot borrow from the string.

@djmitche, do you think this is worth the extra vertical space?

[djmitche]

Yes, I think that's worthwhile. Does the playground run #[test] blocks if given in this context? That might save space, and I don't think there's anything in the test harness that "kills" threads between tests.

[mgeisler]

Yes, I think that's worthwhile. Does the playground run #[test] blocks if given in this context? That might save space, and I don't think there's anything in the test harness that "kills" threads between tests.

The full Playground supports running tests: the Run button changes to Test when it notices a unit test.

[djmitche]

error[E0601]: `main` function not found in crate `playground`
  --> src/main.rs:12:2
   |
12 | }
   |  ^ consider adding a `main` function to `src/main.rs`

so looks like #[test] won't work..

[mgeisler]

Yeah, unit tests only work in the full playground. The embedded playground is not smart enough to detect the unit tests. It would be nice to extend it to be as clever as the full playground.