Merge pull request #1019 from dialelo/future-pure-eval

Use Future#successful in pureEval when possible
typelevel · May 6, 2016 · 185d0d8 · 185d0d8
2 parents 9a96070 + 0e13ff9
commit 185d0d8
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/core/src/main/scala/cats/std/future.scala b/core/src/main/scala/cats/std/future.scala
@@ -13,7 +13,10 @@ trait FutureInstances extends FutureInstances1 {
     new FutureCoflatMap with MonadError[Future, Throwable]{
       def pure[A](x: A): Future[A] = Future.successful(x)
 
-      override def pureEval[A](x: Eval[A]): Future[A] = Future(x.value)
+      override def pureEval[A](x: Eval[A]): Future[A] = x match {
+        case Now(x) => Future.successful(x)
+        case _ => Future(x.value)
+      }
 
       def flatMap[A, B](fa: Future[A])(f: A => Future[B]): Future[B] = fa.flatMap(f)