From 661872f725ced46eb96c8b7940b14139fba19f4f Mon Sep 17 00:00:00 2001
From: edef <edef@edef.eu>
Date: Sun, 5 Mar 2017 20:19:42 +0100
Subject: [PATCH] Don't implement Send at all

On hosted platforms, libstd allows safely borrowing values from TLS with
'thread lifetime and without a Sync bound. As a result, we can't
guarantee that sending a generator across threads won't create dangling
references or data races. In freestanding environments, the notion of
thread-safety is likely to be defined by the consumer of libfringe,
so our Send bounds and implementation are unlikely to be meaningful
anyway.
---
 src/generator.rs   | 7 ++-----
 tests/generator.rs | 2 --
 2 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/generator.rs b/src/generator.rs
index 8f6f7b4..0a5728a 100644
--- a/src/generator.rs
+++ b/src/generator.rs
@@ -88,9 +88,6 @@ pub struct Generator<'a, Input: 'a, Output: 'a, Stack: stack::Stack> {
   phantom:   PhantomData<(&'a (), *mut Input, *const Output)>
 }
 
-unsafe impl<'a, Input, Output, Stack> Send for Generator<'a, Input, Output, Stack>
-  where Input: Send + 'a, Output: Send + 'a, Stack: stack::Stack + Send {}
-
 impl<'a, Input, Output, Stack> Generator<'a, Input, Output, Stack>
     where Input: 'a, Output: 'a, Stack: stack::Stack {
   /// Creates a new generator.
@@ -98,7 +95,7 @@ impl<'a, Input, Output, Stack> Generator<'a, Input, Output, Stack>
   /// See also the [contract](../trait.GuardedStack.html) that needs to be fulfilled by `stack`.
   pub fn new<F>(stack: Stack, f: F) -> Generator<'a, Input, Output, Stack>
       where Stack: stack::GuardedStack,
-            F: FnOnce(&Yielder<Input, Output>, Input) + Send + 'a {
+            F: FnOnce(&Yielder<Input, Output>, Input) + 'a {
     unsafe { Generator::unsafe_new(stack, f) }
   }
 
@@ -110,7 +107,7 @@ impl<'a, Input, Output, Stack> Generator<'a, Input, Output, Stack>
   ///
   /// See also the [contract](../trait.Stack.html) that needs to be fulfilled by `stack`.
   pub unsafe fn unsafe_new<F>(stack: Stack, f: F) -> Generator<'a, Input, Output, Stack>
-      where F: FnOnce(&Yielder<Input, Output>, Input) + Send + 'a {
+      where F: FnOnce(&Yielder<Input, Output>, Input) + 'a {
     unsafe extern "C" fn generator_wrapper<Input, Output, Stack, F>(env: usize, stack_ptr: StackPointer) -> !
         where Stack: stack::Stack, F: FnOnce(&Yielder<Input, Output>, Input) {
       // Retrieve our environment from the callee and return control to it.
diff --git a/tests/generator.rs b/tests/generator.rs
index 0d45c0e..fda5489 100644
--- a/tests/generator.rs
+++ b/tests/generator.rs
@@ -87,8 +87,6 @@ fn with_owned_stack() {
 fn forget_yielded() {
   struct Dropper(*mut bool);
 
-  unsafe impl Send for Dropper {}
-
   impl Drop for Dropper {
     fn drop(&mut self) {
       unsafe {