Skip to content

Commit

Permalink
fix: config (#4)
Browse files Browse the repository at this point in the history
* config: update foundry.toml syntax

Removes a build warning. Requires newer versions
of foundry, which everybody should be using. If not
updated, the syntax will eventually not be supported anymore.

* tests: fix compiler warnings
  • Loading branch information
tynes authored Jul 29, 2022
1 parent 4fcdfd3 commit 81cd99c
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 8 deletions.
4 changes: 1 addition & 3 deletions foundry.toml
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
[default]
[profile.default]
src = 'src'
out = 'out'
libs = ['lib']
remappings = ['ds-test/=lib/ds-test/src/']

# See more config options https://github.com/gakonst/foundry/tree/master/config
10 changes: 5 additions & 5 deletions src/test/ExcessivelySafeCall.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -200,23 +200,23 @@ contract CallTarget {
called = msg.value;
}

function retBytes(uint256 _bytes) public view {
function retBytes(uint256 _bytes) public pure {
assembly {
return(0, _bytes)
}
}

function revBytes(uint256 _bytes) public view {
function revBytes(uint256 _bytes) public pure {
assembly {
revert(0, _bytes)
}
}

function badRet() external returns (bytes memory) {
function badRet() external pure returns (bytes memory) {
retBytes(1_000_000);
}

function badRev() external{
function badRev() external pure {
revBytes(1_000_000);
}
}
}

0 comments on commit 81cd99c

Please sign in to comment.