Minor refactorings

gatling · Sep 18, 2019 · 7f1232a · 7f1232a
1 parent e635d3e
commit 7f1232a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 13 deletions.
diff --git a/...ing-charts/src/main/scala/io/gatling/charts/stats/buffers/SessionDeltaPerSecBuffers.scala b/...ing-charts/src/main/scala/io/gatling/charts/stats/buffers/SessionDeltaPerSecBuffers.scala
@@ -27,11 +27,7 @@ private[stats] object SessionDeltas {
   val Empty = SessionDeltas(0, 0)
 }
 
-private[stats] final case class SessionDeltas(starts: Int, ends: Int) {
-
-  def addStart() = copy(starts = starts + 1)
-  def addEnd() = copy(ends = ends + 1)
-}
+private[stats] final case class SessionDeltas(starts: Int, ends: Int)
 
 private[stats] class SessionDeltaBuffer(minTimestamp: Long, maxTimestamp: Long, buckets: Array[Int], runDurationInSeconds: Int) {
 

diff --git a/gatling-core/src/main/scala/io/gatling/core/check/css/FormExtractor.scala b/gatling-core/src/main/scala/io/gatling/core/check/css/FormExtractor.scala
@@ -104,13 +104,9 @@ private[css] object FormExtractor {
         } yield {
           val childNode = currentNode.getChild(i)
           childNode.getNodeName match {
-            case "input" => extractInput(childNode).map(Seq(_)).getOrElse(Nil)
-            case "select" =>
-              extractSelect(childNode) match {
-                case Some(input) => Seq(input)
-                case None        => Nil
-              }
-            case "textarea" => extractTextArea(childNode).map(Seq(_)).getOrElse(Nil)
+            case "input"    => extractInput(childNode).toList
+            case "select"   => extractSelect(childNode).toList
+            case "textarea" => extractTextArea(childNode).toList
             case _          => processFormRec(childNode, inputs)
           }
         }

diff --git a/gatling-core/src/main/scala/io/gatling/core/check/regex/regex.scala b/gatling-core/src/main/scala/io/gatling/core/check/regex/regex.scala
@@ -26,7 +26,9 @@ package object regex {
 
     def foldLeft[T](zero: T)(f: (JMatcher, T) => T): T = {
       var temp = zero
-      while (matcher.find) temp = f(matcher, temp)
+      while (matcher.find) {
+        temp = f(matcher, temp)
+      }
       temp
     }