Unused private variable _pausedFlags in SelectivePausableUpgradable contract
SelectivePausableUpgradable defines a private _pausedFlags that is never used; developers may confuse it with SelectivePausableStorage._pausedFlags.
Description
The SelectivePausableUpgradable contract utilizes unstructured storage to avoid storage collision by storing the SelectivePausableStorage struct in a specific storage slot determined by keccak256(abi.encode(uint256(keccak256("aurora.SelectivePausable")) - 1)) & ~bytes32(uint256(0xff)). However, the contract also defines a private variable _pausedFlags that is not used anywhere in the code. This unused variable can cause confusion for developers in future development, as it might be mistaken for the SelectivePausableStorage._pausedFlags, leading to potential errors.
Impact
The presence of an unused private variable _pausedFlags can cause confusion during future development. Developers might mistakenly use this variable instead of SelectivePausableStorage._pausedFlags, leading to potential errors or unintended behavior in the contract's functionality.
Recommendation
Remove the unused private variable _pausedFlags from the SelectivePausableUpgradable contract to eliminate any confusion and ensure clarity in the code. This will prevent future developers from mistakenly using the wrong variable.
Resolution
Resolved.

