Lint

Author: stevieraykatz

src/L2/ReverseRegistrar.sol

@@ -71,14 +71,17 @@ contract ReverseRegistrar is Ownable {
     /// @notice Decorator for checking authorization status for a caller against a provided `addr`.
     ///
     /// @dev A caller is authorized to set the record for `addr` if they are one of:
-    ///     1. The `addr` is the sender 
+    ///     1. The `addr` is the sender
     ///     2. The sender is an approved `controller`
     ///     3. The sender is an approved operator for `addr` on the registry
     ///     4. The sender is `Ownable:ownerOf()` for `addr`
     ///
     /// @param addr The `addr` that is being modified.
     modifier authorized(address addr) {
-        if (addr != msg.sender && !controllers[msg.sender] && !registry.isApprovedForAll(addr, msg.sender) && !_ownsContract(addr)) {
+        if (
+            addr != msg.sender && !controllers[msg.sender] && !registry.isApprovedForAll(addr, msg.sender)
+                && !_ownsContract(addr)
+        ) {
             revert NotAuthorized(addr, msg.sender);
         }
         _;

test/ReverseRegistrar/SetControllerApproval.t.sol

@@ -10,11 +10,11 @@ contract SetControllerApproval is ReverseRegistrarBase {
         vm.assume(caller != owner && caller != address(0));
         vm.expectRevert(Ownable.Unauthorized.selector);
         reverse.setControllerApproval(caller, true);
-    } 
+    }
 
     function test_allowsTheOwner_toUpdateControllerApproval(address newController) public {
         vm.assume(newController != address(0));
-        
+
         vm.expectEmit(address(reverse));
         emit ReverseRegistrar.ControllerApprovalChanged(newController, true);
         vm.prank(owner);
@@ -27,4 +27,4 @@ contract SetControllerApproval is ReverseRegistrarBase {
         reverse.setControllerApproval(newController, false);
         assertFalse(reverse.controllers(newController));
     }
-} 
+}