From 44ecd1603aa88d03c822412fd99b44f2f65810df Mon Sep 17 00:00:00 2001 From: three9s <156545083+three9s@users.noreply.github.com> Date: Thu, 9 Jan 2025 11:19:01 -0800 Subject: [PATCH] fix: explicitly mark visibility of state for actions constants (#381) * fix: explicitly mark visibility of state for actions constants * Update Actions.sol --------- Co-authored-by: Sara Reynolds <30504811+snreynolds@users.noreply.github.com> --- src/libraries/Actions.sol | 50 +++++++++++++++++++-------------------- 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/src/libraries/Actions.sol b/src/libraries/Actions.sol index f8703847..a33c4e87 100644 --- a/src/libraries/Actions.sol +++ b/src/libraries/Actions.sol @@ -7,43 +7,43 @@ pragma solidity ^0.8.0; library Actions { // pool actions // liquidity actions - uint256 constant INCREASE_LIQUIDITY = 0x00; - uint256 constant DECREASE_LIQUIDITY = 0x01; - uint256 constant MINT_POSITION = 0x02; - uint256 constant BURN_POSITION = 0x03; - uint256 constant INCREASE_LIQUIDITY_FROM_DELTAS = 0x04; - uint256 constant MINT_POSITION_FROM_DELTAS = 0x05; + uint256 internal constant INCREASE_LIQUIDITY = 0x00; + uint256 internal constant DECREASE_LIQUIDITY = 0x01; + uint256 internal constant MINT_POSITION = 0x02; + uint256 internal constant BURN_POSITION = 0x03; + uint256 internal constant INCREASE_LIQUIDITY_FROM_DELTAS = 0x04; + uint256 internal constant MINT_POSITION_FROM_DELTAS = 0x05; // swapping - uint256 constant SWAP_EXACT_IN_SINGLE = 0x06; - uint256 constant SWAP_EXACT_IN = 0x07; - uint256 constant SWAP_EXACT_OUT_SINGLE = 0x08; - uint256 constant SWAP_EXACT_OUT = 0x09; + uint256 internal constant SWAP_EXACT_IN_SINGLE = 0x06; + uint256 internal constant SWAP_EXACT_IN = 0x07; + uint256 internal constant SWAP_EXACT_OUT_SINGLE = 0x08; + uint256 internal constant SWAP_EXACT_OUT = 0x09; // donate // note this is not supported in the position manager or router - uint256 constant DONATE = 0x0a; + uint256 internal constant DONATE = 0x0a; // closing deltas on the pool manager // settling - uint256 constant SETTLE = 0x0b; - uint256 constant SETTLE_ALL = 0x0c; - uint256 constant SETTLE_PAIR = 0x0d; + uint256 internal constant SETTLE = 0x0b; + uint256 internal constant SETTLE_ALL = 0x0c; + uint256 internal constant SETTLE_PAIR = 0x0d; // taking - uint256 constant TAKE = 0x0e; - uint256 constant TAKE_ALL = 0x0f; - uint256 constant TAKE_PORTION = 0x10; - uint256 constant TAKE_PAIR = 0x11; + uint256 internal constant TAKE = 0x0e; + uint256 internal constant TAKE_ALL = 0x0f; + uint256 internal constant TAKE_PORTION = 0x10; + uint256 internal constant TAKE_PAIR = 0x11; - uint256 constant CLOSE_CURRENCY = 0x12; - uint256 constant CLEAR_OR_TAKE = 0x13; - uint256 constant SWEEP = 0x14; + uint256 internal constant CLOSE_CURRENCY = 0x12; + uint256 internal constant CLEAR_OR_TAKE = 0x13; + uint256 internal constant SWEEP = 0x14; - uint256 constant WRAP = 0x15; - uint256 constant UNWRAP = 0x16; + uint256 internal constant WRAP = 0x15; + uint256 internal constant UNWRAP = 0x16; // minting/burning 6909s to close deltas // note this is not supported in the position manager or router - uint256 constant MINT_6909 = 0x17; - uint256 constant BURN_6909 = 0x18; + uint256 internal constant MINT_6909 = 0x17; + uint256 internal constant BURN_6909 = 0x18; }