Don't write side-effecting nullary methods.

Style says never write methods like this:

  def foo: Unit

If it is Unit, then it is side-effecting, and should be

  def foo(): Unit

Since -Xlint warns about this, we must adhere to its dictate.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/library/scala/concurrent/impl/Promise.scala b/src/library/scala/concurrent/impl/Promise.scala
index a1a3305db0..ff268d850c 100644
--- a/src/library/scala/concurrent/impl/Promise.scala
+++ b/src/library/scala/concurrent/impl/Promise.scala
@@ -107,7 +107,7 @@ private[concurrent] object Promise {
       case _ => None
     }
 
-    override def isCompleted(): Boolean = getState match { // Cheaper than boxing result into Option due to "def value"
+    override def isCompleted: Boolean = getState match { // Cheaper than boxing result into Option due to "def value"
       case _: Try[_] => true
       case _ => false
     }
@@ -156,7 +156,7 @@ private[concurrent] object Promise {
 
     val value = Some(resolveTry(suppliedValue))
 
-    override def isCompleted(): Boolean = true
+    override def isCompleted: Boolean = true
 
     def tryComplete(value: Try[T]): Boolean = false