function _sendFunds in Slingshot is internal and never used so it is basically a dead code (even unsafe as it sends funds twice). You should remove this code to eliminate the confusion and reduce gas costs.
function prependNumber in ConcatStrings is not used so it can also be removed to reduce deployment costs.
tommyz7
commented
2 years ago
Handle
pauliax
Vulnerability details
Impact
function _sendFunds in Slingshot is internal and never used so it is basically a dead code (even unsafe as it sends funds twice). You should remove this code to eliminate the confusion and reduce gas costs.
function prependNumber in ConcatStrings is not used so it can also be removed to reduce deployment costs.
Recommended Mitigation Steps
Get rid of dead code.