crypto: fixed domain was handled is big endian

harmony-dev · Aug 20, 2019 · 2bcf5c2 · 2bcf5c2
1 parent 2b4826c
commit 2bcf5c2
Showing 1 changed file with 8 additions and 10 deletions.
diff --git a/crypto/src/main/java/org/ethereum/beacon/crypto/MessageParameters.java b/crypto/src/main/java/org/ethereum/beacon/crypto/MessageParameters.java
@@ -1,9 +1,7 @@
 package org.ethereum.beacon.crypto;
 
 import tech.pegasys.artemis.ethereum.core.Hash32;
-import tech.pegasys.artemis.util.bytes.Bytes32;
 import tech.pegasys.artemis.util.bytes.Bytes8;
-import tech.pegasys.artemis.util.bytes.BytesValue;
 import tech.pegasys.artemis.util.uint.UInt64;
 
 /**
@@ -17,6 +15,14 @@
  */
 public interface MessageParameters {
 
+  static MessageParameters create(Hash32 hash, Bytes8 domain) {
+    return new Impl(hash, domain);
+  }
+
+  static MessageParameters create(Hash32 hash, UInt64 domain) {
+    return new Impl(hash, domain.toBytes8LittleEndian());
+  }
+
   /**
    * Returns a hash of the message.
    *
@@ -31,14 +37,6 @@ public interface MessageParameters {
    */
   Bytes8 getDomain();
 
-  static MessageParameters create(Hash32 hash, Bytes8 domain) {
-    return new Impl(hash, domain);
-  }
-
-  static MessageParameters create(Hash32 hash, UInt64 domain) {
-    return new Impl(hash, domain.toBytesBigEndian());
-  }
-
   /** A straightforward implementation of {@link MessageParameters}. */
   class Impl implements MessageParameters {
     private final Hash32 hash;