Merge pull request #225 from sangria-graphql/code_clean_up

minor code cleanup

src/main/scala/sangria/marshalling/ArrayMapBuilder.scala

@@ -12,7 +12,7 @@ class ArrayMapBuilder[T](keys: Seq[String]) extends Iterable[(String, T)] {
   private val indexLookup = keys.iterator.zipWithIndex.toMap
   private val indexesSet = MutableSet[Int]()
 
-  def add(key: String, elem: T) = {
+  def add(key: String, elem: T): ArrayMapBuilder[T] = {
     val idx = indexLookup(key)
 
     elements(idx) = key -> elem
@@ -65,9 +65,9 @@ class ArrayMapBuilder[T](keys: Seq[String]) extends Iterable[(String, T)] {
 
   override def iterator: Iterator[(String, T)] =
     new Iterator[(String, T)] {
-      var index = -1
-      var nextIndex = -1
-      val elementsLength = elements.length
+      private var index = -1
+      private var nextIndex = -1
+      private val elementsLength = elements.length
 
       @tailrec def nextIndex(current: Int): Int = {
         val next = current + 1

src/main/scala/sangria/marshalling/FromInput.scala

@@ -1,7 +1,6 @@
 package sangria.marshalling
 
 import scala.annotation.implicitNotFound
-import scala.language.higherKinds
 
 @implicitNotFound(
   "Type ${Val} cannot be used as an input. Please consider defining an implicit instance of `FromInput` for it.")

src/main/scala/sangria/marshalling/MarshallingUtil.scala

@@ -35,7 +35,7 @@ object MarshallingUtil {
   }
 
   implicit class MarshaledConverter[In: InputUnmarshaller](in: In) {
-    def convertMarshaled[Out: ResultMarshallerForType] = convert(in)
+    def convertMarshaled[Out: ResultMarshallerForType]: Out = convert(in)
   }
 
   implicit class ResultMarshallerOps(val m: ResultMarshaller) extends AnyVal {

src/main/scala/sangria/marshalling/ToInput.scala

@@ -3,7 +3,6 @@ package sangria.marshalling
 import sangria.util.tag._
 
 import scala.annotation.implicitNotFound
-import scala.language.higherKinds
 
 @implicitNotFound(
   "Type ${Val} cannot be used as a default value. Please consider defining an implicit instance of `ToInput` for it.")