fix code style

Author: andresilva

contracts/Operations.sol

@@ -52,6 +52,9 @@ contract Operations {
 	function addChecksum(bytes32 _release, bytes32 _platform, bytes32 _checksum)
 		public;
 
+	function setClientOwner(address _newOwner)
+		public;
+
 	function isLatest(bytes32 _client, bytes32 _release)
 		public
 		view
@@ -96,7 +99,4 @@ contract Operations {
 		public
 		view
 		returns (bytes32);
-
-	function setClientOwner(address _newOwner)
-		public;
 }

contracts/OperationsProxy.sol

@@ -22,6 +22,13 @@ import "./Operations.sol";
 /// Specialise proxy wallet. Owner can send transactions unhindered. Delegates
 /// can send only particular transactions to a named Operations contract.
 contract OperationsProxy {
+	event OwnerChanged(address indexed was, address indexed who);
+	event DelegateChanged(address indexed was, address indexed who, uint8 indexed track);
+	event ConfirmerChanged(address indexed was, address indexed who, uint8 indexed track);
+	event NewRequestWaiting(uint8 indexed track, bytes32 hash);
+	event RequestConfirmed(uint8 indexed track, bytes32 hash, bool success);
+	event RequestRejected(uint8 indexed track, bytes32 hash);
+
 	address public owner;
 	mapping(uint8 => address) public delegate;
 	mapping(uint8 => address) public confirmer;
@@ -32,14 +39,22 @@ contract OperationsProxy {
 
 	Operations public operations;
 
-	event OwnerChanged(address indexed was, address indexed who);
-	event DelegateChanged(address indexed was, address indexed who, uint8 indexed track);
-	event ConfirmerChanged(address indexed was, address indexed who, uint8 indexed track);
-	event NewRequestWaiting(uint8 indexed track, bytes32 hash);
-	event RequestConfirmed(uint8 indexed track, bytes32 hash, bool success);
-	event RequestRejected(uint8 indexed track, bytes32 hash);
+	modifier onlyOwner {
+		require(msg.sender == owner);
+		_;
+	}
+
+	modifier onlyDelegateOf(uint8 track) {
+		require(delegate[track] == msg.sender);
+		_;
+	}
+
+	modifier onlyConfirmerOf(uint8 track) {
+		require(confirmer[track] == msg.sender);
+		_;
+	}
 
-	function OperationsProxy(
+	constructor(
 		address _owner,
 		address _stable,
 		address _beta,
@@ -189,19 +204,4 @@ contract OperationsProxy {
 			o_relayed = false;
 		}
 	}
-
-	modifier onlyOwner {
-		require(msg.sender == owner);
-		_;
-	}
-
-	modifier onlyDelegateOf(uint8 track) {
-		require(delegate[track] == msg.sender);
-		_;
-	}
-
-	modifier onlyConfirmerOf(uint8 track) {
-		require(confirmer[track] == msg.sender);
-		_;
-	}
 }

contracts/SimpleOperations.sol

@@ -46,7 +46,23 @@ contract SimpleOperations is Operations {
 	uint32 public latestFork = 0;
 	address public grandOwner = msg.sender;
 
-	function SimpleOperations()
+	modifier onlyOwner {
+		require(grandOwner == msg.sender);
+		_;
+	}
+
+	modifier onlyClientOwner {
+		bytes32 newClient = clientOwner[msg.sender];
+		require(newClient != 0);
+		_;
+	}
+
+	modifier notClientOwner(address owner) {
+		require(clientOwner[owner] == 0);
+		_;
+	}
+
+	constructor()
 		public
 	{
 		client["parity"] = Client(msg.sender);
@@ -286,22 +302,4 @@ contract SimpleOperations is Operations {
 		Release storage r = client[_client].release[_release];
 		return clientExists(_client) && r.track != 0 && r.semver != 0;
 	}
-
-	// Modifiers
-
-	modifier onlyOwner {
-		require(grandOwner == msg.sender);
-		_;
-	}
-
-	modifier onlyClientOwner {
-		bytes32 newClient = clientOwner[msg.sender];
-		require(newClient != 0);
-		_;
-	}
-
-	modifier notClientOwner(address owner) {
-		require(clientOwner[owner] == 0);
-		_;
-	}
 }