I've heard arguments both for and against Solidity modifiers.
On the one hand, they make the code very readable, and the intents behind the calls are very clear. On the other, they increase the overall contract size, as the function code is actually inserted inline in place of the
_ keyword (as opposed to just calling the function code). Furthermore, since the code is inlined, both the function and the modifier share the same restricted variable stack, which can lead to a
Stack Too Deep error if more than a combined total of 16 variables are used.
What do you guys think? Are you with or against the use of modifiers in your code?