Merge pull request #232 from ethereum/event-emit

Update DAO contracts (use emit for events)

Author: axic

test/DAO/DAO.sol

@@ -467,7 +467,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
 
         sumOfProposalDeposits += msg.value;
 
-        ProposalAdded(
+        emit ProposalAdded(
             _proposalID,
             _recipient,
             _amount,
@@ -517,7 +517,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
             blocked[msg.sender] = _proposalID;
         }
 
-        Voted(_proposalID, _supportsProposal, msg.sender);
+        emit Voted(_proposalID, _supportsProposal, msg.sender);
     }
 
 
@@ -610,7 +610,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
         closeProposal(_proposalID);
 
         // Initiate event
-        ProposalTallied(_proposalID, _success, quorum);
+        emit ProposalTallied(_proposalID, _success, quorum);
     }
 
 
@@ -835,7 +835,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
         if (msg.sender != curator)
             revert();
         allowedRecipients[_recipient] = _allowed;
-        AllowedRecipientChanged(_recipient, _allowed);
+        emit AllowedRecipientChanged(_recipient, _allowed);
         return true;
     }
 
@@ -880,7 +880,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
     }
 
     function createNewDAO(address _newCurator) internal returns (DAO _newDAO) {
-        NewCurator(_newCurator);
+        emit NewCurator(_newCurator);
         return daoCreator.createDAO(
             _newCurator,
             0,

test/DAO/ManagedAccount.sol

@@ -58,7 +58,7 @@ contract ManagedAccount is ManagedAccountInterface{
         if (msg.sender != owner || msg.value > 0 || (payOwnerOnly && _recipient != owner))
             revert();
         if (_recipient.call.value(_amount)("")) {
-            PayOut(_recipient, _amount);
+            emit PayOut(_recipient, _amount);
             return true;
         } else {
             return false;

test/DAO/Token.sol

@@ -104,7 +104,7 @@ contract Token is TokenInterface {
         if (balances[msg.sender] >= _amount && _amount > 0) {
             balances[msg.sender] -= _amount;
             balances[_to] += _amount;
-            Transfer(msg.sender, _to, _amount);
+            emit Transfer(msg.sender, _to, _amount);
             return true;
         } else {
            return false;
@@ -124,7 +124,7 @@ contract Token is TokenInterface {
             balances[_to] += _amount;
             balances[_from] -= _amount;
             allowed[_from][msg.sender] -= _amount;
-            Transfer(_from, _to, _amount);
+            emit Transfer(_from, _to, _amount);
             return true;
         } else {
             return false;
@@ -133,7 +133,7 @@ contract Token is TokenInterface {
 
     function approve(address _spender, uint256 _amount) public returns (bool success) {
         allowed[msg.sender][_spender] = _amount;
-        Approval(msg.sender, _spender, _amount);
+        emit Approval(msg.sender, _spender, _amount);
         return true;
     }
 

test/DAO/TokenCreation.sol

@@ -109,10 +109,10 @@ contract TokenCreation is TokenCreationInterface, Token {
             balances[_tokenHolder] += token;
             totalSupply += token;
             weiGiven[_tokenHolder] += msg.value;
-            CreatedToken(_tokenHolder, token);
+            emit CreatedToken(_tokenHolder, token);
             if (totalSupply >= minTokensToCreate && !isFueled) {
                 isFueled = true;
-                FuelingToDate(totalSupply);
+                emit FuelingToDate(totalSupply);
             }
             return true;
         }
@@ -127,7 +127,7 @@ contract TokenCreation is TokenCreationInterface, Token {
 
             // Execute refund
             if (msg.sender.call.value(weiGiven[msg.sender])("")) {
-                Refund(msg.sender, weiGiven[msg.sender]);
+                emit Refund(msg.sender, weiGiven[msg.sender]);
                 totalSupply -= balances[msg.sender];
                 balances[msg.sender] = 0;
                 weiGiven[msg.sender] = 0;