Merge pull request #1054 from akka/wip-2928-log-postStop-failures-∂π

log failures in postStop, see #2928

akka-actor-tests/src/test/scala/akka/actor/ActorLifeCycleSpec.scala

@@ -119,6 +119,16 @@ class ActorLifeCycleSpec extends AkkaSpec with BeforeAndAfterEach with ImplicitS
       system.stop(supervisor)
     }
 
+    "log failues in postStop" in {
+      val a = system.actorOf(Props(new Actor {
+        def receive = Actor.emptyBehavior
+        override def postStop { throw new Exception("hurrah") }
+      }))
+      EventFilter[Exception]("hurrah", occurrences = 1) intercept {
+        a ! PoisonPill
+      }
+    }
+
     "clear the behavior stack upon restart" in {
       case class Become(recv: ActorContext ⇒ Receive)
       val a = system.actorOf(Props(new Actor {

akka-actor/src/main/scala/akka/actor/dungeon/FaultHandling.scala

@@ -199,7 +199,9 @@ private[akka] trait FaultHandling { this: ActorCell ⇒
      * specific order.
      */
     try if (a ne null) a.postStop()
-    finally try dispatcher.detach(this)
+    catch {
+      case NonFatal(e) ⇒ publish(Error(e, self.path.toString, clazz(a), e.getMessage))
+    } finally try dispatcher.detach(this)
     finally try parent.sendSystemMessage(ChildTerminated(self))
     finally try parent ! NullMessage // read ScalaDoc of NullMessage to see why
     finally try tellWatchersWeDied(a)