Modify ReentrancyGuard to reduce contract size

[abhi3700]

Refer issue #3514

I separated the code present in function modifier with 2 (private) functions outside & just call those function(s) from inside the modifier.

[Amxx]

Hello @abhi3700 and thank you for this PR.

I think we would want better naming. Maybe something like _nonReentrantBefore, _nonReentrantAfter ?

[abhi3700]

Yeah great!

[frangio]

@abhi3700 Can you implement the changes suggested by @Amxx?

[frangio]

Did you measure the additional runtime cost?

[abhi3700]

@abhi3700 Can you implement the changes suggested by @Amxx?

Done! ✅

[abhi3700]

Did you measure the additional runtime cost?

In my repo,
the compile result with standard "ReentrancyGuard" is 9.72s

And the compile result with standard "ReentrancyGuard" is 9.29s

The compile time differs because of the machine running concurrent applications at this time.

So, it can be seen that the contract size is highly reduced by 0.308 KB

[frangio]

Thanks!