From db222c6a53036e651519eb36048692b3e5caf28e Mon Sep 17 00:00:00 2001
From: cristovaoth <cristovao.honorato@gmail.com>
Date: Sat, 30 Sep 2023 23:05:44 +0200
Subject: [PATCH] Rename helper function

---
 contracts/signature/SignatureChecker.sol | 23 ++++++++---------------
 1 file changed, 8 insertions(+), 15 deletions(-)

diff --git a/contracts/signature/SignatureChecker.sol b/contracts/signature/SignatureChecker.sol
index bff85941..755eca3d 100644
--- a/contracts/signature/SignatureChecker.sol
+++ b/contracts/signature/SignatureChecker.sol
@@ -32,18 +32,19 @@ abstract contract SignatureChecker {
                 bytes32 r,
                 bytes32 s,
                 uint8 v,
-                bool isContractSignature,
-                uint256 start,
-                uint256 end
+                bool isEIP1271,
+                uint256 start
             ) = _splitSignature(data);
 
             bytes32 hash = _moduleTxHash(data[:start], nonce);
 
-            if (isContractSignature) {
+            if (isEIP1271) {
                 // When handling contract signatures the address
                 // of the contract is encoded into r
                 signer = address(uint160(uint256(r)));
-                if (!isValidContractSignature(signer, hash, data[start:end])) {
+                uint256 end = data.length - 65;
+
+                if (!_isValidContractSignature(signer, hash, data[start:end])) {
                     signer = address(0);
                 }
             } else {
@@ -61,14 +62,7 @@ abstract contract SignatureChecker {
     )
         private
         pure
-        returns (
-            bytes32 r,
-            bytes32 s,
-            uint8 v,
-            bool isEIP1271,
-            uint256 start,
-            uint256 end
-        )
+        returns (bytes32 r, bytes32 s, uint8 v, bool isEIP1271, uint256 start)
     {
         uint256 length = data.length;
         r = bytes32(data[length - 65:]);
@@ -76,7 +70,6 @@ abstract contract SignatureChecker {
         v = uint8(bytes1(data[length - 1:]));
         isEIP1271 = v == 0 && (uint256(s) < (length - 65));
         start = isEIP1271 ? uint256(s) : length - 65;
-        end = isEIP1271 ? length - 65 : length;
     }
 
     /**
@@ -101,7 +94,7 @@ abstract contract SignatureChecker {
         return keccak256(moduleTxData);
     }
 
-    function isValidContractSignature(
+    function _isValidContractSignature(
         address signer,
         bytes32 hash,
         bytes calldata signature