index fields in events

Author: mmv08

contracts/Safe.sol

@@ -59,8 +59,8 @@ contract Safe is
     event SafeSetup(address indexed initiator, address[] owners, uint256 threshold, address initializer, address fallbackHandler);
     event ApproveHash(bytes32 indexed approvedHash, address indexed owner);
     event SignMsg(bytes32 indexed msgHash);
-    event ExecutionFailure(bytes32 txHash, uint256 payment);
-    event ExecutionSuccess(bytes32 txHash, uint256 payment);
+    event ExecutionFailure(bytes32 indexed txHash, uint256 payment);
+    event ExecutionSuccess(bytes32 indexed txHash, uint256 payment);
 
     uint256 public nonce;
     bytes32 private _deprecatedDomainSeparator;

contracts/base/FallbackManager.sol

@@ -8,7 +8,7 @@ import "../common/SelfAuthorized.sol";
  * @author Richard Meissner - @rmeissner
  */
 abstract contract FallbackManager is SelfAuthorized {
-    event ChangedFallbackHandler(address handler);
+    event ChangedFallbackHandler(address indexed handler);
 
     // keccak256("fallback_manager.handler.address")
     bytes32 internal constant FALLBACK_HANDLER_STORAGE_SLOT = 0x6c9a6c4a39284e37ed1cf53d337577d14212a4870fb976a4366c693b939918d5;

contracts/base/GuardManager.sol

@@ -36,7 +36,7 @@ abstract contract BaseGuard is Guard {
  * @author Richard Meissner - @rmeissner
  */
 abstract contract GuardManager is SelfAuthorized {
-    event ChangedGuard(address guard);
+    event ChangedGuard(address indexed guard);
 
     // keccak256("guard_manager.guard.address")
     bytes32 internal constant GUARD_STORAGE_SLOT = 0x4a204f620c8c5ccdca3fd54d003badd85ba500436a431f0cbda4f558c93c34c8;

contracts/base/ModuleManager.sol

@@ -14,8 +14,8 @@ import "./Executor.sol";
  * @author Richard Meissner - @rmeissner
  */
 abstract contract ModuleManager is SelfAuthorized, Executor {
-    event EnabledModule(address module);
-    event DisabledModule(address module);
+    event EnabledModule(address indexed module);
+    event DisabledModule(address indexed module);
     event ExecutionFromModuleSuccess(address indexed module);
     event ExecutionFromModuleFailure(address indexed module);
 

contracts/base/OwnerManager.sol

@@ -10,8 +10,8 @@ import "../common/SelfAuthorized.sol";
  * @author Richard Meissner - @rmeissner
  */
 abstract contract OwnerManager is SelfAuthorized {
-    event AddedOwner(address owner);
-    event RemovedOwner(address owner);
+    event AddedOwner(address indexed owner);
+    event RemovedOwner(address indexed owner);
     event ChangedThreshold(uint256 threshold);
 
     address internal constant SENTINEL_OWNERS = address(0x1);

contracts/libraries/CreateCall.sol

@@ -9,7 +9,7 @@ pragma solidity >=0.7.0 <0.9.0;
  */
 contract CreateCall {
     /// @notice Emitted when a new contract is created
-    event ContractCreation(address newContract);
+    event ContractCreation(address indexed newContract);
 
     /**
      * @notice Deploys a new contract using the create2 opcode.