Rename popByte and popAddress

4 files changed, 12 insertions, 12 deletions

diff --git a/packages/contracts/src/contracts/current/protocol/Exchange/MixinSettlement.sol b/packages/contracts/src/contracts/current/protocol/Exchange/MixinSettlement.sol
index 69b70112f..29a9c87bd 100644
--- a/packages/contracts/src/contracts/current/protocol/Exchange/MixinSettlement.sol
+++ b/packages/contracts/src/contracts/current/protocol/Exchange/MixinSettlement.sol
@@ -62,8 +62,8 @@ contract MixinSettlement is
     )
         internal
     {
-        uint8 makerAssetProxyId = uint8(popByte(order.makerAssetData));
-        uint8 takerAssetProxyId = uint8(popByte(order.takerAssetData));
+        uint8 makerAssetProxyId = uint8(popLastByte(order.makerAssetData));
+        uint8 takerAssetProxyId = uint8(popLastByte(order.takerAssetData));
         bytes memory zrxAssetData = ZRX_ASSET_DATA;
         dispatchTransferFrom(
             order.makerAssetData,
@@ -108,8 +108,8 @@ contract MixinSettlement is
     )
         internal
     {
-        uint8 leftMakerAssetProxyId = uint8(popByte(leftOrder.makerAssetData));
-        uint8 rightMakerAssetProxyId = uint8(popByte(rightOrder.makerAssetData));
+        uint8 leftMakerAssetProxyId = uint8(popLastByte(leftOrder.makerAssetData));
+        uint8 rightMakerAssetProxyId = uint8(popLastByte(rightOrder.makerAssetData));
         bytes memory zrxAssetData = ZRX_ASSET_DATA;
         // Order makers and taker
         dispatchTransferFrom(
diff --git a/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol b/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol
index 48a0c5552..1a556dfe2 100644
--- a/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol
+++ b/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol
@@ -93,7 +93,7 @@ contract MixinSignatureValidator is
         );
 
         // Pop last byte off of signature byte array.
-        SignatureType signatureType = SignatureType(uint8(popByte(signature)));
+        SignatureType signatureType = SignatureType(uint8(popLastByte(signature)));
 
         // Variables are not scoped in Solidity.
         uint8 v;
@@ -183,7 +183,7 @@ contract MixinSignatureValidator is
         // | 0x14 + x | 1      | Signature type is always "\x06" |
         } else if (signatureType == SignatureType.Validator) {
             // Pop last 20 bytes off of signature byte array.
-            address validator = popAddress(signature);
+            address validator = popLast20Bytes(signature);
             // Ensure signer has approved validator.
             if (!allowedValidators[signer][validator]) {
                 return false;
diff --git a/packages/contracts/src/contracts/current/test/TestLibBytes/TestLibBytes.sol b/packages/contracts/src/contracts/current/test/TestLibBytes/TestLibBytes.sol
index abce0cb22..6f1898acd 100644
--- a/packages/contracts/src/contracts/current/test/TestLibBytes/TestLibBytes.sol
+++ b/packages/contracts/src/contracts/current/test/TestLibBytes/TestLibBytes.sol
@@ -28,24 +28,24 @@ contract TestLibBytes is
     /// @dev Pops the last byte off of a byte array by modifying its length.
     /// @param b Byte array that will be modified.
     /// @return The byte that was popped off.
-    function publicPopByte(bytes memory b)
+    function publicPopLastByte(bytes memory b)
         public
         pure
         returns (bytes memory, bytes1 result)
     {
-        result = popByte(b);
+        result = popLastByte(b);
         return (b, result);
     }
 
     /// @dev Pops the last 20 bytes off of a byte array by modifying its length.
     /// @param b Byte array that will be modified.
     /// @return The 20 byte address that was popped off.
-    function publicPopAddress(bytes memory b)
+    function publicPopLast20Bytes(bytes memory b)
         public
         pure
         returns (bytes memory, address result)
     {
-        result = popAddress(b);
+        result = popLast20Bytes(b);
         return (b, result);
     }
 
diff --git a/packages/contracts/src/contracts/current/utils/LibBytes/LibBytes.sol b/packages/contracts/src/contracts/current/utils/LibBytes/LibBytes.sol
index 339270a57..10d7ce41a 100644
--- a/packages/contracts/src/contracts/current/utils/LibBytes/LibBytes.sol
+++ b/packages/contracts/src/contracts/current/utils/LibBytes/LibBytes.sol
@@ -35,7 +35,7 @@ contract LibBytes is
     /// @dev Pops the last byte off of a byte array by modifying its length.
     /// @param b Byte array that will be modified.
     /// @return The byte that was popped off.
-    function popByte(bytes memory b)
+    function popLastByte(bytes memory b)
         internal
         pure
         returns (bytes1 result)
@@ -59,7 +59,7 @@ contract LibBytes is
     /// @dev Pops the last 20 bytes off of a byte array by modifying its length.
     /// @param b Byte array that will be modified.
     /// @return The 20 byte address that was popped off.
-    function popAddress(bytes memory b)
+    function popLast20Bytes(bytes memory b)
         internal
         pure
         returns (address result)