ETCM-167: Use +: instead of :: on RLPList.

Author: aakoshh

src/main/scala/io/iohk/ethereum/network/discovery/codecs/RLPCodecs.scala

@@ -45,7 +45,7 @@ object RLPCodecs {
   implicit val nodeRLPCodec: RLPCodec[Node] =
     RLPCodec.instance[Node](
       { case Node(id, address) =>
-        RLPEncoder.encode(address).asInstanceOf[RLPList] ++ RLPList(id)
+        RLPEncoder.encode(address).asInstanceOf[RLPList] :+ id
       },
       {
         case list @ RLPList(items @ _*) if items.length >= 4 =>
@@ -62,10 +62,10 @@ object RLPCodecs {
       { case EthereumNodeRecord.Content(seq, attrs) =>
         val kvs = attrs
           .foldRight(RLPList()) { case ((k, v), kvs) =>
-            k.toArray :: v.toArray :: kvs
+            k.toArray +: v.toArray +: kvs
           }
 
-        seq :: kvs
+        seq +: kvs
       },
       { case RLPList(seq, kvs @ _*) =>
         val attrs = kvs
@@ -89,7 +89,8 @@ object RLPCodecs {
   implicit val enrRLPCodec: RLPCodec[EthereumNodeRecord] =
     RLPCodec.instance(
       { case EthereumNodeRecord(signature, content) =>
-        signature.toByteArray :: RLPEncoder.encode(content).asInstanceOf[RLPList]
+        val contentList = RLPEncoder.encode(content).asInstanceOf[RLPList]
+        signature.toByteArray +: contentList
       },
       { case RLPList(signature, content @ _*) =>
         EthereumNodeRecord(

src/main/scala/io/iohk/ethereum/rlp/RLPImplicitDerivations.scala

@@ -58,7 +58,7 @@ object RLPImplicitDerivations {
   }
 
   private def decodeError[T](subject: String, error: String, maybeEncodeable: Option[RLPEncodeable] = None): T =
-    throw RLPException(s"error decoding $subject: $error", maybeEncodeable)
+    throw RLPException(s"Cannot decode $subject: $error", maybeEncodeable)
 
   private def tryDecode[T](subject: => String, encodeable: RLPEncodeable)(f: RLPEncodeable => T): T = {
     try {

src/main/scala/io/iohk/ethereum/rlp/package.scala

@@ -15,9 +15,12 @@ package object rlp {
   sealed trait RLPEncodeable
 
   case class RLPList(items: RLPEncodeable*) extends RLPEncodeable {
-    def ::(item: RLPEncodeable): RLPList =
+    def +:(item: RLPEncodeable): RLPList =
       RLPList((item +: items): _*)
 
+    def :+(item: RLPEncodeable): RLPList =
+      RLPList((items :+ item): _*)
+
     def ++(other: RLPList): RLPList =
       RLPList((items ++ other.items): _*)
   }
@@ -96,7 +99,7 @@ package object rlp {
 
         override def decode(rlp: RLPEncodeable): T =
           if (dec.isDefinedAt(rlp)) dec(rlp)
-          else throw RLPException(s"Cannot decode type ${ct.runtimeClass.getSimpleName} from RLP.", rlp)
+          else throw RLPException(s"Cannot decode type ${ct.runtimeClass.getSimpleName} from unexpected RLP.", rlp)
       }
 
     def apply[T](enc: RLPEncoder[T], dec: RLPDecoder[T]): RLPCodec[T] =