From 7b554cf461584a3c80dca93d02987708319be03f Mon Sep 17 00:00:00 2001
From: Sowmayjain <champ.sowmay@gmail.com>
Date: Sun, 7 Apr 2019 20:11:57 +0530
Subject: [PATCH] Small code refactoring.

---
 contracts/UserWallet.sol | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/contracts/UserWallet.sol b/contracts/UserWallet.sol
index 32d2cde..d35b766 100644
--- a/contracts/UserWallet.sol
+++ b/contracts/UserWallet.sol
@@ -24,8 +24,7 @@ contract AddressRecord {
      */
     modifier logicAuth(address logicAddr) {
         require(logicAddr != address(0), "logic-proxy-address-required");
-        bool islogic = RegistryInterface(registry).logic(logicAddr);
-        require(islogic, "logic-not-authorised");
+        require(RegistryInterface(registry).logic(logicAddr), "logic-not-authorised");
         _;
     }
 
@@ -110,7 +109,7 @@ contract UserNote {
  */
 contract UserWallet is UserAuth, UserNote {
 
-    event LogExecute(address sender, address target, uint srcNum, uint sessionNum);
+    event LogExecute(address target, uint srcNum, uint sessionNum);
 
     /**
      * @dev sets the "address registry", owner's last activity, owner's active period and initial owner
@@ -126,14 +125,14 @@ contract UserWallet is UserAuth, UserNote {
      * @dev Execute authorised calls via delegate call
      * @param _target logic proxy address
      * @param _data delegate call data
-     * @param _srcNum to find the source
-     * @param _sessionNum to find the session
+     * @param _src to find the source
+     * @param _session to find the session
      */
     function execute(
         address _target,
         bytes memory _data,
-        uint _srcNum,
-        uint _sessionNum
+        uint _src,
+        uint _session
     ) 
         public
         payable
@@ -143,10 +142,9 @@ contract UserWallet is UserAuth, UserNote {
         returns (bytes memory response)
     {
         emit LogExecute(
-            msg.sender,
             _target,
-            _srcNum,
-            _sessionNum
+            _src,
+            _session
         );
         
         // call contract in current context