issues
search
safe-global
/
safe-smart-account
Safe allows secure management of blockchain assets.
https://safe.global
GNU Lesser General Public License v3.0
1.84k
stars
907
forks
source link
Reducing Safe Contract Code Size
#726
Closed
remedcu
closed
8 months ago
remedcu
commented
8 months ago
TODO:
[x] Using
uint256
for
v
. (Saved around 9 bytes)
[x] Using storage slot directly where possible. (Saved around 30+ bytes)
[x] Using
if
with
revert
instead of
require
where possible. (Saved around 20 bytes)
[x] Using assembly to
revert
in
Safe.sol
and inherited files. (Saved around ~1900 bytes~ 2700 bytes)
Closes #713
coveralls
commented
8 months ago
Pull Request Test Coverage Report for
Build 7503450091
0
of
0
changed or added relevant lines in
0
files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (
-0.2%
) to
94.397%
Totals
Change from base
Build 7503409586
:
-0.2%
Covered Lines:
396
Relevant Lines:
404
💛 -
Coveralls
TODO:
uint256
forv
. (Saved around 9 bytes)if
withrevert
instead ofrequire
where possible. (Saved around 20 bytes)revert
inSafe.sol
and inherited files. (Saved around ~1900 bytes~ 2700 bytes)Closes #713