Closed xermicus closed 7 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 88.57%. Comparing base (
5cab937
) to head (a9b7e02
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Codecov says -0.01%
coverage on the project however the diff hit 100%. I'm gonna merge
Closes #1623
The basic idea of the algorithm checking whether a function name is eligible for mangling or not is to mangle the name if that function is publicly callable but the functions name appears multiple time in the contract. But this doesn't account for virtual functions also appearing more than one time in the same contract if they are overridden. With this PR, we bail early if the function we are checking overrides, marking only the single one non-overriding implementation as eligible for mangling. Consequently, functions which override but do not overload are no longer unnecessarily mangled.