From c63f0ab7ec9a2f40f1038bae07793ea80818f2c8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=A0est=C3=A1k=20V=C3=ADt?= Date: Mon, 18 Jan 2016 13:14:14 +0100 Subject: [PATCH] Suppressed an IDEA warning. I believe that having the same style of parentheses on both sides of compare is more readable. --- app/controllers/ProjectsWithReports.scala | 1 + 1 file changed, 1 insertion(+) diff --git a/app/controllers/ProjectsWithReports.scala b/app/controllers/ProjectsWithReports.scala index 20b3148..caf553c 100644 --- a/app/controllers/ProjectsWithReports.scala +++ b/app/controllers/ProjectsWithReports.scala @@ -9,6 +9,7 @@ final case class ReportInfo( import scala.math.Ordered.orderingToOrdered + //noinspection ScalaUnnecessaryParentheses override def compare(that: ReportInfo): Int = ((projectName, subprojectNameOption, fullId)) compare ((that.projectName, that.subprojectNameOption, that.fullId)) // It seems to be a good idea to have a custom equals and hashCode for performance reasons