Reduce complexity in modifiers related to governance and strategist
The modifier onlyGovernance:\\n```\\nmodifier onlyGovernance() {\\n    \\_checkGovernance();\\n    \\_;\\n}\\n```\\n\\nCalls the internal function _checkGovernance:\\n```\\nfunction \\_checkGovernance() internal view {\\n    require(msg.sender == governance, "not governance");\\n}\\n```\\n\\nThere is no other case where the internal method `_checkGovernance` is called directly.\\nOne can reduce complexity by removing the internal function and moving its code directly in the modifier. This will increase code size but reduce gas used and code complexity.\\nThere are multiple similar instances:\\n```\\nfunction \\_checkStrategist() internal view {\\n    require(msg.sender == governance || strategists[msg.sender], "not strategist");\\n}\\n\\nmodifier onlyStrategist {\\n    \\_checkStrategist();\\n    \\_;\\n}\\n```\\n\\n```\\nfunction \\_checkGovernance() internal view {\\n    require(msg.sender == governance, "not governance");\\n}\\n\\nmodifier onlyGovernance() {\\n    \\_checkGovernance();\\n    \\_;\\n}\\n```\\n\\n```\\nfunction \\_checkGovernance() internal view {\\n    require(msg.sender == IGaugeController(controller).governance(), "not governance");\\n}\\n\\nmodifier onlyGovernance() {\\n    \\_checkGovernance();\\n    \\_;\\n}\\n```\\n
Consider removing the internal function and including its body in the modifier directly if the code size is not an issue.
```\\nmodifier onlyGovernance() {\\n    \\_checkGovernance();\\n    \\_;\\n}\\n```\\n