Skip to content

Commit

Permalink
WIP change library functions to internal
Browse files Browse the repository at this point in the history
change some library function from public to internal to account for foundry-rs/foundry#4266 (comment)
  • Loading branch information
jonwalch authored Jul 14, 2023
1 parent ce6c654 commit 74f6628
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/SafeTestTools.sol
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ library SafeTestLib {
address gasToken,
address refundReceiver,
bytes memory signatures
) public returns (bool) {
) internal returns (bool) {
if (instance.owners.length == 0) {
revert("SAFETEST: Instance not initialized. Call _setupSafe() to initialize a test safe");
}
Expand Down Expand Up @@ -171,13 +171,13 @@ library SafeTestLib {
uint256 value,
bytes memory data,
Enum.Operation operation
) public returns (bool) {
) internal returns (bool) {
return execTransaction(instance, to, value, data, operation, 0, 0, 0, address(0), address(0), "");
}

/// @dev performs a noraml "call"
function execTransaction(SafeInstance memory instance, address to, uint256 value, bytes memory data)
public
internal
returns (bool)
{
return execTransaction(instance, to, value, data, Enum.Operation.Call, 0, 0, 0, address(0), address(0), "");
Expand Down Expand Up @@ -227,7 +227,7 @@ library SafeTestLib {
);
}

function EIP1271Sign(SafeInstance memory instance, bytes memory data) public {
function EIP1271Sign(SafeInstance memory instance, bytes memory data) internal {
address signMessageLib = address(new SignMessageLib());
execTransaction({
instance: instance,
Expand Down

0 comments on commit 74f6628

Please sign in to comment.