Do not prefix getters

contracts/bounties/BreakInvariantBounty.sol

@@ -50,7 +50,7 @@ contract BreakInvariantBounty is PullPayment, Ownable {
    * @dev Transfers the current balance to the owner and terminates the contract.
    */
   function destroy() public onlyOwner {
-    selfdestruct(getOwner());
+    selfdestruct(owner());
   }
 
   /**

contracts/drafts/TokenVesting.sol

@@ -142,7 +142,7 @@ contract TokenVesting is Ownable {
 
     revoked_[_token] = true;
 
-    _token.safeTransfer(getOwner(), refund);
+    _token.safeTransfer(owner(), refund);
 
     emit Revoked();
   }

contracts/lifecycle/Pausable.sol

@@ -18,7 +18,7 @@ contract Pausable is Ownable {
   /**
    * @return true if the contract is paused, false otherwise.
    */
-  function isPaused() public view returns(bool) {
+  function paused() public view returns(bool) {
     return paused_;
   }
 

contracts/ownership/CanReclaimToken.sol

@@ -20,7 +20,7 @@ contract CanReclaimToken is Ownable {
    */
   function reclaimToken(IERC20 _token) external onlyOwner {
     uint256 balance = _token.balanceOf(this);
-    _token.safeTransfer(getOwner(), balance);
+    _token.safeTransfer(owner(), balance);
   }
 
 }

contracts/ownership/Claimable.sol

@@ -23,7 +23,7 @@ contract Claimable is Ownable {
   /**
    * @return the address of the pending owner.
    */
-  function getPendingOwner() public view returns(address) {
+  function pendingOwner() public view returns(address) {
     return pendingOwner_;
   }
 

contracts/ownership/DelayedClaimable.sol

@@ -16,14 +16,14 @@ contract DelayedClaimable is Claimable {
   /**
    * @return the start of the claimable period.
    */
-  function getStart() public view returns(uint256) {
+  function start() public view returns(uint256) {
     return start_;
   }
 
   /**
    * @return the end of the claimable period.
    */
-  function getEnd() public view returns(uint256) {
+  function end() public view returns(uint256) {
     return end_;
   }
 

contracts/ownership/Heritable.sol

@@ -51,9 +51,9 @@ contract Heritable is Ownable {
   }
 
   function setHeir(address _newHeir) public onlyOwner {
-    require(_newHeir != getOwner());
+    require(_newHeir != owner());
     heartbeat();
-    emit HeirChanged(getOwner(), _newHeir);
+    emit HeirChanged(owner(), _newHeir);
     heir_ = _newHeir;
   }
 
@@ -87,7 +87,7 @@ contract Heritable is Ownable {
    */
   function proclaimDeath() public onlyHeir {
     require(_ownerLives());
-    emit OwnerProclaimedDead(getOwner(), heir_, timeOfDeath_);
+    emit OwnerProclaimedDead(owner(), heir_, timeOfDeath_);
     // solium-disable-next-line security/no-block-members
     timeOfDeath_ = block.timestamp;
   }
@@ -96,7 +96,7 @@ contract Heritable is Ownable {
    * @dev Owner can send a heartbeat if they were mistakenly pronounced dead.
    */
   function heartbeat() public onlyOwner {
-    emit OwnerHeartbeated(getOwner());
+    emit OwnerHeartbeated(owner());
     timeOfDeath_ = 0;
   }
 
@@ -107,7 +107,7 @@ contract Heritable is Ownable {
     require(!_ownerLives());
     // solium-disable-next-line security/no-block-members
     require(block.timestamp >= timeOfDeath_ + heartbeatTimeout_);
-    emit HeirOwnershipClaimed(getOwner(), heir_);
+    emit HeirOwnershipClaimed(owner(), heir_);
     _transferOwnership(heir_);
     timeOfDeath_ = 0;
   }

contracts/ownership/Ownable.sol

@@ -28,7 +28,7 @@ contract Ownable {
   /**
    * @return the address of the owner.
    */
-  function getOwner() public view returns(address) {
+  function owner() public view returns(address) {
     return owner_;
   }
 

contracts/ownership/Superuser.sol

@@ -27,7 +27,7 @@ contract Superuser is Ownable, RBAC {
   }
 
   modifier onlyOwnerOrSuperuser() {
-    require(msg.sender == getOwner() || isSuperuser(msg.sender));
+    require(msg.sender == owner() || isSuperuser(msg.sender));
     _;
   }
 

contracts/payment/RefundEscrow.sol

@@ -32,14 +32,14 @@ contract RefundEscrow is Ownable, ConditionalEscrow {
   /**
    * @return the current state of the escrow.
    */
-  function getState() public view returns(State) {
+  function state() public view returns(State) {
     return state_;
   }
 
   /**
    * @return the beneficiary of the escrow.
    */
-  function getBeneficiary() public view returns(address) {
+  function beneficiary() public view returns(address) {
     return beneficiary_;
   }
 

contracts/payment/SplitPayment.sol

@@ -38,35 +38,35 @@ contract SplitPayment {
   /**
    * @return the total shares of the contract.
    */
-  function getTotalShares() public view returns(uint256) {
+  function totalShares() public view returns(uint256) {
     return totalShares_;
   }
 
   /**
    * @return the total amount already released.
    */
-  function getTotalReleased() public view returns(uint256) {
+  function totalReleased() public view returns(uint256) {
     return totalReleased_;
   }
 
   /**
    * @return the shares of an account.
    */
-  function getShares(address _account) public view returns(uint256) {
+  function shares(address _account) public view returns(uint256) {
     return shares_[_account];
   }
 
   /**
    * @return the amount already released to an account.
    */
-  function getReleased(address _account) public view returns(uint256) {
+  function released(address _account) public view returns(uint256) {
     return released_[_account];
   }
 
   /**
    * @return the address of a payee.
    */
-  function getPayee(uint256 index) public view returns(address) {
+  function payee(uint256 index) public view returns(address) {
     return payees_[index];
   }
 

contracts/token/ERC20/ERC20Mintable.sol

@@ -22,7 +22,7 @@ contract ERC20Mintable is ERC20, Ownable {
   }
 
   modifier hasMintPermission() {
-    require(msg.sender == getOwner());
+    require(msg.sender == owner());
     _;
   }
 

test/access/SignatureBouncer.test.js

@@ -20,7 +20,7 @@ contract('Bouncer', function ([_, owner, anyone, bouncerAddress, authorizedUser]
 
   context('management', function () {
     it('has a default owner of self', async function () {
-      (await this.bouncer.getOwner()).should.equal(owner);
+      (await this.bouncer.owner()).should.equal(owner);
     });
 
     it('allows the owner to add a bouncer', async function () {

test/crowdsale/MintedCrowdsale.test.js

@@ -21,7 +21,7 @@ contract('MintedCrowdsale', function ([_, investor, wallet, purchaser]) {
     });
 
     it('should be token owner', async function () {
-      (await this.token.getOwner()).should.equal(this.crowdsale.address);
+      (await this.token.owner()).should.equal(this.crowdsale.address);
     });
 
     shouldBehaveLikeMintedCrowdsale([_, investor, wallet, purchaser], rate, value);

test/ownership/Claimable.test.js

@@ -16,12 +16,12 @@ contract('Claimable', function ([_, owner, newOwner, anyone]) {
   });
 
   it('should have an owner', async function () {
-    (await claimable.getOwner()).should.not.equal(0);
+    (await claimable.owner()).should.not.equal(0);
   });
 
   it('changes pendingOwner after transfer', async function () {
     await claimable.transferOwnership(newOwner, { from: owner });
-    (await claimable.getPendingOwner()).should.equal(newOwner);
+    (await claimable.pendingOwner()).should.equal(newOwner);
   });
 
   it('should prevent to claimOwnership from anyone', async function () {
@@ -40,7 +40,7 @@ contract('Claimable', function ([_, owner, newOwner, anyone]) {
     it('changes allow pending owner to claim ownership', async function () {
       await claimable.claimOwnership({ from: newOwner });
 
-      (await claimable.getOwner()).should.equal(newOwner);
+      (await claimable.owner()).should.equal(newOwner);
     });
   });
 });

test/ownership/DelayedClaimable.test.js

@@ -17,35 +17,35 @@ contract('DelayedClaimable', function ([_, owner, newOwner]) {
     await this.delayedClaimable.transferOwnership(newOwner, { from: owner });
     await this.delayedClaimable.setLimits(0, 1000, { from: owner });
 
-    (await this.delayedClaimable.getEnd()).should.be.bignumber.equal(1000);
+    (await this.delayedClaimable.end()).should.be.bignumber.equal(1000);
 
-    (await this.delayedClaimable.getStart()).should.be.bignumber.equal(0);
+    (await this.delayedClaimable.start()).should.be.bignumber.equal(0);
   });
 
   it('changes pendingOwner after transfer successful', async function () {
     await this.delayedClaimable.transferOwnership(newOwner, { from: owner });
     await this.delayedClaimable.setLimits(0, 1000, { from: owner });
 
-    (await this.delayedClaimable.getEnd()).should.be.bignumber.equal(1000);
+    (await this.delayedClaimable.end()).should.be.bignumber.equal(1000);
 
-    (await this.delayedClaimable.getStart()).should.be.bignumber.equal(0);
+    (await this.delayedClaimable.start()).should.be.bignumber.equal(0);
 
-    (await this.delayedClaimable.getPendingOwner()).should.equal(newOwner);
+    (await this.delayedClaimable.pendingOwner()).should.equal(newOwner);
     await this.delayedClaimable.claimOwnership({ from: newOwner });
-    (await this.delayedClaimable.getOwner()).should.equal(newOwner);
+    (await this.delayedClaimable.owner()).should.equal(newOwner);
   });
 
   it('changes pendingOwner after transfer fails', async function () {
     await this.delayedClaimable.transferOwnership(newOwner, { from: owner });
     await this.delayedClaimable.setLimits(100, 110, { from: owner });
 
-    (await this.delayedClaimable.getEnd()).should.be.bignumber.equal(110);
+    (await this.delayedClaimable.end()).should.be.bignumber.equal(110);
 
-    (await this.delayedClaimable.getStart()).should.be.bignumber.equal(100);
+    (await this.delayedClaimable.start()).should.be.bignumber.equal(100);
 
-    (await this.delayedClaimable.getPendingOwner()).should.equal(newOwner);
+    (await this.delayedClaimable.pendingOwner()).should.equal(newOwner);
     await assertRevert(this.delayedClaimable.claimOwnership({ from: newOwner }));
-    (await this.delayedClaimable.getOwner()).should.not.equal(newOwner);
+    (await this.delayedClaimable.owner()).should.not.equal(newOwner);
   });
 
   it('set end and start invalid values fail', async function () {

test/ownership/Ownable.behavior.js

@@ -9,12 +9,12 @@ require('chai')
 function shouldBehaveLikeOwnable (owner, [anyone]) {
   describe('as an ownable', function () {
     it('should have an owner', async function () {
-      (await this.ownable.getOwner()).should.equal(owner);
+      (await this.ownable.owner()).should.equal(owner);
     });
 
     it('changes owner after transfer', async function () {
       await this.ownable.transferOwnership(anyone, { from: owner });
-      (await this.ownable.getOwner()).should.equal(anyone);
+      (await this.ownable.owner()).should.equal(anyone);
     });
 
     it('should prevent non-owners from transfering', async function () {
@@ -27,7 +27,7 @@ function shouldBehaveLikeOwnable (owner, [anyone]) {
 
     it('loses owner after renouncement', async function () {
       await this.ownable.renounceOwnership({ from: owner });
-      (await this.ownable.getOwner()).should.equal(ZERO_ADDRESS);
+      (await this.ownable.owner()).should.equal(ZERO_ADDRESS);
     });
 
     it('should prevent non-owners from renouncement', async function () {

test/ownership/Superuser.test.js

@@ -40,7 +40,7 @@ contract('Superuser', function ([_, firstOwner, newSuperuser, newOwner, anyone])
         'OwnershipTransferred'
       );
 
-      (await this.superuser.getOwner()).should.equal(newOwner);
+      (await this.superuser.owner()).should.equal(newOwner);
     });
 
     it('should change owner after the owner transfers the ownership', async function () {
@@ -49,7 +49,7 @@ contract('Superuser', function ([_, firstOwner, newSuperuser, newOwner, anyone])
         'OwnershipTransferred'
       );
 
-      (await this.superuser.getOwner()).should.equal(newOwner);
+      (await this.superuser.owner()).should.equal(newOwner);
     });
   });
 

test/payment/RefundEscrow.test.js

@@ -29,8 +29,8 @@ contract('RefundEscrow', function ([_, owner, beneficiary, refundee1, refundee2]
 
     context('active state', function () {
       it('has beneficiary and state', async function () {
-        (await this.escrow.getBeneficiary()).should.be.equal(beneficiary);
-        (await this.escrow.getState()).should.be.bignumber.equal(0);
+        (await this.escrow.beneficiary()).should.be.equal(beneficiary);
+        (await this.escrow.state()).should.be.bignumber.equal(0);
       });
 
       it('accepts deposits', async function () {

test/payment/SplitPayment.test.js

@@ -47,13 +47,13 @@ contract('SplitPayment', function ([_, owner, payee1, payee2, payee3, nonpayee1,
     });
 
     it('should have total shares', async function () {
-      (await this.contract.getTotalShares()).should.be.bignumber.equal(20 + 10 + 70);
+      (await this.contract.totalShares()).should.be.bignumber.equal(20 + 10 + 70);
     });
 
     it('should have payees', async function () {
       this.payees.forEach(async (payee, index) => {
-        (await this.getPayee(index)).should.be.equal(payee);
-        (await this.contract.getReleased(payee)).should.be.bignumber.equal(0);
+        (await this.payee(index)).should.be.equal(payee);
+        (await this.contract.released(payee)).should.be.bignumber.equal(0);
       });
     });
 
@@ -64,11 +64,11 @@ contract('SplitPayment', function ([_, owner, payee1, payee2, payee3, nonpayee1,
     });
 
     it('should store shares if address is payee', async function () {
-      (await this.contract.getShares(payee1)).should.be.bignumber.not.equal(0);
+      (await this.contract.shares(payee1)).should.be.bignumber.not.equal(0);
     });
 
     it('should not store shares if address is not payee', async function () {
-      (await this.contract.getShares(nonpayee1)).should.be.bignumber.equal(0);
+      (await this.contract.shares(nonpayee1)).should.be.bignumber.equal(0);
     });
 
     it('should throw if no funds to claim', async function () {

test/token/ERC20/ERC20Pausable.test.js

@@ -13,7 +13,7 @@ contract('ERC20Pausable', function ([_, owner, recipient, anotherAccount]) {
       describe('when the token is unpaused', function () {
         it('pauses the token', async function () {
           await this.token.pause({ from });
-          (await this.token.isPaused()).should.equal(true);
+          (await this.token.paused()).should.equal(true);
         });
 
         it('emits a Pause event', async function () {
@@ -55,7 +55,7 @@ contract('ERC20Pausable', function ([_, owner, recipient, anotherAccount]) {
 
         it('unpauses the token', async function () {
           await this.token.unpause({ from });
-          (await this.token.isPaused()).should.equal(false);
+          (await this.token.paused()).should.equal(false);
         });
 
         it('emits an Unpause event', async function () {
@@ -87,18 +87,18 @@ contract('ERC20Pausable', function ([_, owner, recipient, anotherAccount]) {
 
     describe('paused', function () {
       it('is not paused by default', async function () {
-        (await this.token.isPaused({ from })).should.equal(false);
+        (await this.token.paused({ from })).should.equal(false);
       });
 
       it('is paused after being paused', async function () {
         await this.token.pause({ from });
-        (await this.token.isPaused({ from })).should.equal(true);
+        (await this.token.paused({ from })).should.equal(true);
       });
 
       it('is not paused after being paused and then unpaused', async function () {
         await this.token.pause({ from });
         await this.token.unpause({ from });
-        (await this.token.isPaused()).should.equal(false);
+        (await this.token.paused()).should.equal(false);
       });
     });