From a355012c2eac1399ef25aae5b65cb2273b19da81 Mon Sep 17 00:00:00 2001 From: Adriaan Moors Date: Thu, 9 Jul 2015 14:49:08 -0700 Subject: [PATCH] Do not use `ListBuffer#readOnly` It's dangerous, deprecated, and was removed in 2.12.0-M1. See https://github.com/scala/scala/pull/4140. `ListBuffer#toList` has equivalent performance, except it actually returns an immutable copy(-on-write). --- util/log/src/main/scala/sbt/BufferedLogger.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/log/src/main/scala/sbt/BufferedLogger.scala b/util/log/src/main/scala/sbt/BufferedLogger.scala index a40d3f1be..08a60577d 100644 --- a/util/log/src/main/scala/sbt/BufferedLogger.scala +++ b/util/log/src/main/scala/sbt/BufferedLogger.scala @@ -39,7 +39,7 @@ class BufferedLogger(delegate: AbstractLogger) extends BasicLogger { * Flushes the buffer to the delegate logger. This method calls logAll on the delegate * so that the messages are written consecutively. The buffer is cleared in the process. */ - def play(): Unit = synchronized { delegate.logAll(buffer.readOnly); buffer.clear() } + def play(): Unit = synchronized { delegate.logAll(buffer.toList); buffer.clear() } /** Clears buffered events and disables buffering. */ def clear(): Unit = synchronized { buffer.clear(); recording = false } /** Plays buffered events and disables buffering. */