restrict function types to pure

ChainSafe · Oct 3, 2023 · cad52f1 · cad52f1
1 parent d45a93d
commit cad52f1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/contracts/src/SyncStepLib.sol b/contracts/src/SyncStepLib.sol
@@ -28,7 +28,7 @@ library SyncStepLib {
     * @param keysPoseidonCommitment The commitment to the keys used in the sync step
     * @return The public input commitment that can be sent to the verifier contract.
      */
-    function toInputCommitment(SyncStepInput memory args, bytes32 keysPoseidonCommitment) internal view returns (uint256) {
+    function toInputCommitment(SyncStepInput memory args, bytes32 keysPoseidonCommitment) internal pure returns (uint256) {
         bytes32 h = sha256(abi.encodePacked(
             toLittleEndian64(args.attestedSlot),
             toLittleEndian64(args.finalizedSlot),

diff --git a/contracts/test/SyncStepExternal.sol b/contracts/test/SyncStepExternal.sol
@@ -11,7 +11,7 @@ import { SyncStepLib } from "../src/SyncStepLib.sol";
 contract SyncStepExternal {
     using SyncStepLib for SyncStepLib.SyncStepInput;
 
-    function toInputCommitment(SyncStepLib.SyncStepInput calldata args, bytes32 keysPoseidonCommitment) public view returns (uint256) {
+    function toInputCommitment(SyncStepLib.SyncStepInput calldata args, bytes32 keysPoseidonCommitment) public pure returns (uint256) {
         return args.toInputCommitment(keysPoseidonCommitment);
     }
 }