[protocol review] Suggestion of code refactors (nitpick)

[QGarchery]

Here is a list of potential refactors:

• choose either msgSender() or msg.sender, currently both are used #238
• 2. use _convertToAssetsWithFeeAccrue in accruedFeeShares. It requires to rename _convertToAssetsWithFeeAccrue to a more general name
• 3. _supplyBalance could be inlined, it’s a one liner
• 4. _suppliable could be inlined, it is used only once
• 5. _staticWithdrawMorpho could be inlined, it is used only once

[Rubilmax]

Agreed with 1
I need to check for 2

Regarding 3,4,5: I disagree that because a function can be inlined, it should be defactorized. Factorizing them also provides documentation

[MerlinEgalite]

Aligned with @Rubilmax

[Rubilmax]

If we apply 4 & 5 we should also inline _supplyMorpho and _withdrawMorpho