Merge pull request #146 from accelerated/promise-constructor

Added compile test for the Proimse constructor

Author: Alex Damian

quantum/util/impl/quantum_generic_future_impl.h

@@ -63,9 +63,15 @@ GenericFuture<T>::GenericFuture()
 {}
 
 template <typename T>
-GenericFuture<T>::GenericFuture(const Promise<T>& p) :
-    _context(local::context() ? p.getICoroFuture() : p.getIThreadFuture())
-{}
+GenericFuture<T>::GenericFuture(const Promise<T>& p)
+{
+    if (local::context()) {
+        _context = p.getICoroFuture();
+    }
+    else {
+        _context = p.getIThreadFuture();
+    }
+}
 
 template <typename T>
 GenericFuture<T>::GenericFuture(ThreadContextPtr<T> f) :

tests/quantum_generic_future_tests.cpp

@@ -87,6 +87,9 @@ TEST(GenericFuture, TestCopyable)
     //create one future
     v.push_back(dispatcher.post([](VoidContextPtr)->int{ return 33; }));
 
+    //use Promise constructor
+    v.push_back(Promise<int>{});
+    
     //copy it
     v.push_back(v.front());