FRosner#123 remove useless volatile and synchronized (thx @Gerrrr)

firemonk9 · Jan 27, 2017 · fcc11d8 · fcc11d8
1 parent d19c86a
commit fcc11d8
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/main/scala/de/frosner/ddq/reporters/EmailReporter.scala b/src/main/scala/de/frosner/ddq/reporters/EmailReporter.scala
@@ -25,7 +25,6 @@ import scala.concurrent.Await
   *                          or else you will lose the results.
   *
  **/
-@volatile
 case class EmailReporter(smtpServer: String,
                          to: Set[String],
                          cc: Set[String] = Set.empty,
@@ -50,7 +49,7 @@ case class EmailReporter(smtpServer: String,
     *
     * @param checkResult The [[CheckResult]] to be reported
    **/
-  override def report(checkResult: CheckResult, header: String, prologue: String): Unit = synchronized {
+  override def report(checkResult: CheckResult, header: String, prologue: String): Unit = {
     val numSuccessfulConstraints = computeSuccessful(checkResult)
     val numFailedConstraints = computeFailed(checkResult)
     val numErroredConstraints = computeErrored(checkResult)