Skip to content

Commit 370022b

Browse files
authored
Merge pull request #243 from ekpyron/replaceInvalidEmptyReturns
Replace empty return statements in functions that have return values.
2 parents ce5d3ca + a444e30 commit 370022b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/DAO/DAO.sol

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -531,7 +531,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
531531
// If we are over deadline and waiting period, assert proposal is closed
532532
if (p.open && now > p.votingDeadline + waitPeriod) {
533533
closeProposal(_proposalID);
534-
return;
534+
return false;
535535
}
536536

537537
// Check if the proposal can be executed
@@ -550,7 +550,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
550550
if (!isRecipientAllowed(p.recipient)) {
551551
closeProposal(_proposalID);
552552
p.creator.send(p.proposalDeposit);
553-
return;
553+
return false;
554554
}
555555

556556
bool proposalCheck = true;

0 commit comments

Comments
 (0)