chore: use calldata instead of memory

Signed-off-by: Tomás Migone <[email protected]>

Author: tmigone

packages/horizon/contracts/payments/collectors/GraphTallyCollector.sol

@@ -60,14 +60,14 @@ contract GraphTallyCollector is EIP712, GraphDirectory, Authorizable, IGraphTall
      * @notice REVERT: This function may revert if ECDSA.recover fails, check ECDSA library for details.
      */
     /// @inheritdoc IPaymentsCollector
-    function collect(IGraphPayments.PaymentTypes paymentType, bytes memory data) external override returns (uint256) {
+    function collect(IGraphPayments.PaymentTypes paymentType, bytes calldata data) external override returns (uint256) {
         return _collect(paymentType, data, 0);
     }
 
     /// @inheritdoc IGraphTallyCollector
     function collect(
         IGraphPayments.PaymentTypes paymentType,
-        bytes memory data,
+        bytes calldata data,
         uint256 tokensToCollect
     ) external override returns (uint256) {
         return _collect(paymentType, data, tokensToCollect);
@@ -93,7 +93,7 @@ contract GraphTallyCollector is EIP712, GraphDirectory, Authorizable, IGraphTall
      */
     function _collect(
         IGraphPayments.PaymentTypes _paymentType,
-        bytes memory _data,
+        bytes calldata _data,
         uint256 _tokensToCollect
     ) private returns (uint256) {
         (SignedRAV memory signedRAV, uint256 dataServiceCut) = abi.decode(_data, (SignedRAV, uint256));

packages/subgraph-service/contracts/DisputeManager.sol

@@ -285,7 +285,7 @@ contract DisputeManager is
     }
 
     /// @inheritdoc IDisputeManager
-    function encodeReceipt(Attestation.Receipt memory receipt) external view override returns (bytes32) {
+    function encodeReceipt(Attestation.Receipt calldata receipt) external view override returns (bytes32) {
         return _encodeReceipt(receipt);
     }
 
@@ -310,8 +310,8 @@ contract DisputeManager is
 
     /// @inheritdoc IDisputeManager
     function areConflictingAttestations(
-        Attestation.State memory attestation1,
-        Attestation.State memory attestation2
+        Attestation.State calldata attestation1,
+        Attestation.State calldata attestation2
     ) external pure override returns (bool) {
         return Attestation.areConflicting(attestation1, attestation2);
     }