=all #3448 Newer Scalariform running on more files

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

@@ -131,7 +131,7 @@ class ActorLifeCycleSpec extends AkkaSpec("akka.actor.serialize-messages=off") w
       case class Become(recv: ActorContext ⇒ Receive)
       val a = system.actorOf(Props(new Actor {
         def receive = {
-          case Become(beh) ⇒ context.become(beh(context), discardOld = false); sender ! "ok"
+          case Become(beh) ⇒ { context.become(beh(context), discardOld = false); sender ! "ok" }
           case x           ⇒ sender ! 42
         }
       }))

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

@@ -201,8 +201,13 @@ object FSMTimingSpec {
       case Initial -> TestSingleTimerResubmit ⇒ setTimer("blah", Tick, 500.millis.dilated, false)
     }
     when(TestSingleTimerResubmit) {
-      case Event(Tick, _) ⇒ tester ! Tick; setTimer("blah", Tock, 500.millis.dilated, false); stay()
+      case Event(Tick, _) ⇒
-      case Event(Tock, _) ⇒ tester ! Tock; goto(Initial)
+        tester ! Tick
+        setTimer("blah", Tock, 500.millis.dilated, false)
+        stay()
+      case Event(Tock, _) ⇒
+        tester ! Tock
+        goto(Initial)
     }
     when(TestCancelTimer) {
       case Event(Tick, _) ⇒

akka-actor-tests/src/test/scala/akka/actor/dispatch/DispatcherActorSpec.scala

@@ -86,7 +86,7 @@ class DispatcherActorSpec extends AkkaSpec(DispatcherActorSpec.config) with Defa
       val slowOne = system.actorOf(
         Props(new Actor {
           def receive = {
-            case "hogexecutor" ⇒ sender ! "OK"; start.await
+            case "hogexecutor" ⇒ { sender ! "OK"; start.await }
             case "ping"        ⇒ if (works.get) latch.countDown()
           }
         }).withDispatcher(throughputDispatcher))
@@ -120,8 +120,8 @@ class DispatcherActorSpec extends AkkaSpec(DispatcherActorSpec.config) with Defa
       val slowOne = system.actorOf(
         Props(new Actor {
           def receive = {
-            case "hogexecutor" ⇒ ready.countDown(); start.await
+            case "hogexecutor" ⇒ { ready.countDown(); start.await }
-            case "ping"        ⇒ works.set(false); context.stop(self)
+            case "ping"        ⇒ { works.set(false); context.stop(self) }
           }
         }).withDispatcher(throughputDispatcher))
 

akka-actor-tests/src/test/scala/akka/event/EventStreamSpec.scala

@@ -44,7 +44,7 @@ object EventStreamSpec {
         bus.subscribe(context.self, classOf[SetTarget])
         bus.subscribe(context.self, classOf[UnhandledMessage])
         sender ! Logging.LoggerInitialized
-      case SetTarget(ref)      ⇒ dst = ref; dst ! "OK"
+      case SetTarget(ref)      ⇒ { dst = ref; dst ! "OK" }
       case e: Logging.LogEvent ⇒ dst ! e
       case u: UnhandledMessage ⇒ dst ! u
     }

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

@@ -170,8 +170,8 @@ private[akka] trait FaultHandling { this: ActorCell ⇒
       suspendNonRecursive()
       // suspend children
       val skip: Set[ActorRef] = currentMessage match {
-        case Envelope(Failed(_, _, _), child) ⇒ setFailed(child); Set(child)
+        case Envelope(Failed(_, _, _), child) ⇒ { setFailed(child); Set(child) }
-        case _                                ⇒ setFailed(self); Set.empty
+        case _                                ⇒ { setFailed(self); Set.empty }
       }
       suspendChildren(exceptFor = skip ++ childrenNotToSuspend)
       t match {

akka-actor/src/main/scala/akka/io/SslTlsSupport.scala

@@ -94,7 +94,7 @@ class SslTlsSupport(engine: SSLEngine) extends PipelineStage[HasLogging, Command
           // to shutdown the connection when getting CLOSED in encrypt
           closeEngine()
 
-        case x: Tcp.WriteCommand =>
+        case x: Tcp.WriteCommand ⇒
           throw new IllegalArgumentException(
             "SslTlsSupport doesn't support Tcp.WriteCommands of type " + x.getClass.getSimpleName)
 

akka-actor/src/main/scala/akka/io/Tcp.scala

@@ -370,8 +370,8 @@ object Tcp extends ExtensionId[TcpExt] with ExtensionIdProvider {
         def next(): SimpleWriteCommand =
           current match {
             case null                  ⇒ Iterator.empty.next()
-            case CompoundWrite(h, t)   ⇒ current = t; h
+            case CompoundWrite(h, t)   ⇒ { current = t; h }
-            case x: SimpleWriteCommand ⇒ current = null; x
+            case x: SimpleWriteCommand ⇒ { current = null; x }
           }
       }
   }

akka-actor/src/main/scala/akka/io/TcpManager.scala

@@ -50,11 +50,11 @@ private[io] class TcpManager(tcp: TcpExt)
   def receive = workerForCommandHandler {
     case c: Connect ⇒
       val commander = sender // cache because we create a function that will run asyncly
-      registry ⇒ Props(classOf[TcpOutgoingConnection], tcp, registry, commander, c)
+      (registry ⇒ Props(classOf[TcpOutgoingConnection], tcp, registry, commander, c))
 
     case b: Bind ⇒
       val commander = sender // cache because we create a function that will run asyncly
-      registry ⇒ Props(classOf[TcpListener], selectorPool, tcp, registry, commander, b)
+      (registry ⇒ Props(classOf[TcpListener], selectorPool, tcp, registry, commander, b))
   }
 
 }

akka-actor/src/main/scala/akka/io/UdpManager.scala

@@ -48,11 +48,11 @@ private[io] class UdpManager(udp: UdpExt) extends SelectionHandler.SelectorBased
   def receive = workerForCommandHandler {
     case b: Bind ⇒
       val commander = sender // cache because we create a function that will run asyncly
-      registry ⇒ Props(classOf[UdpListener], udp, registry, commander, b)
+      (registry ⇒ Props(classOf[UdpListener], udp, registry, commander, b))
 
     case SimpleSender(options) ⇒
       val commander = sender // cache because we create a function that will run asyncly
-      registry ⇒ Props(classOf[UdpSender], udp, registry, commander, options)
+      (registry ⇒ Props(classOf[UdpSender], udp, registry, commander, options))
   }
 
 }

akka-dataflow/src/test/scala/akka/dataflow/DataflowSpec.scala

@@ -27,8 +27,11 @@ object DataflowSpec {
 
   class TestDelayActor(await: TestLatch) extends Actor {
     def receive = {
-      case "Hello"   ⇒ Await.ready(await, TestLatch.DefaultTimeout); sender ! "World"
+      case "Hello" ⇒
-      case "NoReply" ⇒ Await.ready(await, TestLatch.DefaultTimeout)
+        Await.ready(await, TestLatch.DefaultTimeout)
+        sender ! "World"
+      case "NoReply" ⇒
+        Await.ready(await, TestLatch.DefaultTimeout)
       case "Failure" ⇒
         Await.ready(await, TestLatch.DefaultTimeout)
         sender ! Status.Failure(new RuntimeException("Expected exception; to test fault-tolerance"))

akka-persistence/src/main/scala/akka/persistence/Channel.scala

@@ -90,7 +90,7 @@ class Channel private (_channelId: Option[String]) extends Actor with Stash {
   }
 
   private val buffering: Actor.Receive = {
-    case DeliveredResolved | DeliveredUnresolved ⇒ context.unbecome(); unstashAll() // TODO: optimize
+    case DeliveredResolved | DeliveredUnresolved ⇒ { context.unbecome(); unstashAll() } // TODO: optimize
     case _: Deliver                              ⇒ stash()
   }
 
@@ -232,9 +232,9 @@ private trait ResolvedDelivery extends Actor {
 
   def receive = {
     case DeliverResolved             ⇒ context.actorSelection(path) ! Identify(1)
-    case ActorIdentity(1, Some(ref)) ⇒ onResolveSuccess(ref); shutdown(DeliveredResolved)
+    case ActorIdentity(1, Some(ref)) ⇒ { onResolveSuccess(ref); shutdown(DeliveredResolved) }
-    case ActorIdentity(1, None)      ⇒ onResolveFailure(); shutdown(DeliveredUnresolved)
+    case ActorIdentity(1, None)      ⇒ { onResolveFailure(); shutdown(DeliveredUnresolved) }
-    case ReceiveTimeout              ⇒ onResolveFailure(); shutdown(DeliveredUnresolved)
+    case ReceiveTimeout              ⇒ { onResolveFailure(); shutdown(DeliveredUnresolved) }
   }
 
   def shutdown(message: Any) {

akka-persistence/src/main/scala/akka/persistence/journal/inmem/InmemJournal.scala

@@ -50,10 +50,18 @@ private[persistence] class InmemStore extends Actor {
   var messages = Map.empty[String, Vector[PersistentImpl]]
 
   def receive = {
-    case Write(p)               ⇒ add(p); success()
+    case Write(p) ⇒
-    case WriteBatch(pb)         ⇒ pb.foreach(add); success()
+      add(p)
-    case Delete(p)              ⇒ update(p.processorId, p.sequenceNr)(_.copy(deleted = true)); success()
+      success()
-    case Confirm(pid, snr, cid) ⇒ update(pid, snr)(p ⇒ p.copy(confirms = cid +: p.confirms)); success()
+    case WriteBatch(pb) ⇒
+      pb.foreach(add)
+      success()
+    case Delete(p) ⇒
+      update(p.processorId, p.sequenceNr)(_.copy(deleted = true))
+      success()
+    case Confirm(pid, snr, cid) ⇒
+      update(pid, snr)(p ⇒ p.copy(confirms = cid +: p.confirms))
+      success()
     case Replay(pid, fromSnr, toSnr, callback) ⇒ {
       for {
         ms ← messages.get(pid)

akka-persistence/src/test/scala/akka/persistence/ProcessorStashSpec.scala

@@ -14,9 +14,9 @@ object ProcessorStashSpec {
     var state: List[String] = Nil
 
     val behaviorA: Actor.Receive = {
-      case Persistent("a", snr)  ⇒ update("a", snr); context.become(behaviorB)
+      case Persistent("a", snr)  ⇒ { update("a", snr); context.become(behaviorB) }
       case Persistent("b", snr)  ⇒ update("b", snr)
-      case Persistent("c", snr)  ⇒ update("c", snr); unstashAll()
+      case Persistent("c", snr)  ⇒ { update("c", snr); unstashAll() }
       case "x"                   ⇒ update("x")
       case "boom"                ⇒ throw new TestException("boom")
       case Persistent("boom", _) ⇒ throw new TestException("boom")
@@ -24,7 +24,7 @@ object ProcessorStashSpec {
     }
 
     val behaviorB: Actor.Receive = {
-      case Persistent("b", _) ⇒ stash(); context.become(behaviorA)
+      case Persistent("b", _) ⇒ { stash(); context.become(behaviorA) }
       case "x"                ⇒ stash()
     }
 

akka-remote-tests/src/multi-jvm/scala/akka/remote/RemoteNodeShutdownAndComesBackSpec.scala

@@ -35,7 +35,7 @@ object RemoteNodeShutdownAndComesBackSpec extends MultiNodeConfig {
 
   class Subject extends Actor {
     def receive = {
-      case "shutdown" => context.system.shutdown()
+      case "shutdown" ⇒ context.system.shutdown()
       case msg        ⇒ sender ! msg
     }
   }
@@ -137,7 +137,6 @@ abstract class RemoteNodeShutdownAndComesBackSpec
                     """).withFallback(system.settings.config))
         freshSystem.actorOf(Props[Subject], "subject")
 
-
         freshSystem.awaitTermination(30.seconds)
       }
 

akka-samples/akka-sample-cluster/src/multi-jvm/scala/sample/cluster/stats/StatsSampleSingleMasterSpec.scala

@@ -79,7 +79,7 @@ abstract class StatsSampleSingleMasterSpec extends MultiNodeSpec(StatsSampleSing
 
       Cluster(system) join firstAddress
 
-       receiveN(3).collect { case MemberUp(m) => m.address }.toSet must be (
+      receiveN(3).collect { case MemberUp(m) ⇒ m.address }.toSet must be(
         Set(firstAddress, secondAddress, thirdAddress))
 
       Cluster(system).unsubscribe(testActor)

akka-samples/akka-sample-cluster/src/multi-jvm/scala/sample/cluster/stats/StatsSampleSpec.scala

@@ -96,7 +96,7 @@ abstract class StatsSampleSpec extends MultiNodeSpec(StatsSampleSpecConfig)
       system.actorOf(Props[StatsWorker], "statsWorker")
       system.actorOf(Props[StatsService], "statsService")
 
-      receiveN(3).collect { case MemberUp(m) => m.address }.toSet must be (
+      receiveN(3).collect { case MemberUp(m) ⇒ m.address }.toSet must be(
         Set(firstAddress, secondAddress, thirdAddress))
 
       Cluster(system).unsubscribe(testActor)

akka-samples/akka-sample-cluster/src/multi-jvm/scala/sample/cluster/stats/japi/StatsSampleJapiSpec.scala

@@ -81,7 +81,7 @@ abstract class StatsSampleJapiSpec extends MultiNodeSpec(StatsSampleJapiSpecConf
       system.actorOf(Props[StatsWorker], "statsWorker")
       system.actorOf(Props[StatsService], "statsService")
 
-      receiveN(3).collect { case MemberUp(m) => m.address }.toSet must be (
+      receiveN(3).collect { case MemberUp(m) ⇒ m.address }.toSet must be(
         Set(firstAddress, secondAddress, thirdAddress))
 
       Cluster(system).unsubscribe(testActor)

akka-samples/akka-sample-cluster/src/multi-jvm/scala/sample/cluster/stats/japi/StatsSampleSingleMasterJapiSpec.scala

@@ -78,7 +78,7 @@ abstract class StatsSampleSingleMasterJapiSpec extends MultiNodeSpec(StatsSample
 
       Cluster(system) join firstAddress
 
-      receiveN(3).collect { case MemberUp(m) => m.address }.toSet must be (
+      receiveN(3).collect { case MemberUp(m) ⇒ m.address }.toSet must be(
         Set(firstAddress, secondAddress, thirdAddress))
 
       Cluster(system).unsubscribe(testActor)

akka-samples/akka-sample-multi-node/src/multi-jvm/scala/sample/multinode/MultiNodeSample.scala

@@ -46,7 +46,7 @@ class MultiNodeSample extends MultiNodeSpec(MultiNodeSampleConfig)
       runOn(node2) {
         system.actorOf(Props(new Actor {
           def receive = {
-            case "ping" => sender ! "pong"
+            case "ping" ⇒ sender ! "pong"
           }
         }), "ponger")
         enterBarrier("deployed")

akka-samples/akka-sample-persistence/src/main/scala/sample/persistence/SnapshotExample.scala

@@ -20,7 +20,7 @@ object SnapshotExample extends App {
       case Persistent(s, snr)                    ⇒ state = state.update(s"${s}-${snr}")
       case SaveSnapshotSuccess(metadata)         ⇒ // ...
       case SaveSnapshotFailure(metadata, reason) ⇒ // ...
-      case SnapshotOffer(_, s: ExampleState)     ⇒ println("offered state = " + s); state = s
+      case SnapshotOffer(_, s: ExampleState)     ⇒ { println("offered state = " + s); state = s }
       case "print"                               ⇒ println("current state = " + state)
       case "snap"                                ⇒ saveSnapshot(state)
     }

akka-zeromq/src/main/scala/akka/zeromq/ConcurrentSocketActor.scala

@@ -63,7 +63,7 @@ private[zeromq] class ConcurrentSocketActor(params: immutable.Seq[SocketOption])
   }
 
   private def handleConnectOption(msg: SocketConnectOption): Unit = msg match {
-    case Connect(endpoint) ⇒ socket.connect(endpoint); notifyListener(Connecting)
+    case Connect(endpoint) ⇒ { socket.connect(endpoint); notifyListener(Connecting) }
     case Bind(endpoint)    ⇒ socket.bind(endpoint)
   }