Merge pull request #224 from ethereum/updateCallAndKeccak256

Update test code to support new way of using keccak256 and call.

Author: axic

test/DAO/DAO.sol

@@ -455,7 +455,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
         p.recipient = _recipient;
         p.amount = _amount;
         p.description = _description;
-        p.proposalHash = keccak256(_recipient, _amount, _transactionData);
+        p.proposalHash = keccak256(abi.encodePacked(_recipient, _amount, _transactionData));
         p.votingDeadline = now + _debatingPeriod;
         p.open = true;
         //p.proposalPassed = False; // that's default
@@ -484,7 +484,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
         bytes _transactionData
     ) noEther view returns (bool _codeChecksOut) {
         Proposal p = proposals[_proposalID];
-        return p.proposalHash == keccak256(_recipient, _amount, _transactionData);
+        return p.proposalHash == keccak256(abi.encodePacked(_recipient, _amount, _transactionData));
     }
 
 
@@ -543,7 +543,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
             || !p.open
             || p.proposalPassed // anyone trying to call us recursively?
             // Does the transaction code match the proposal?
-            || p.proposalHash != keccak256(p.recipient, p.amount, _transactionData)) {
+            || p.proposalHash != keccak256(abi.encodePacked(p.recipient, p.amount, _transactionData))) {
 
             revert();
         }
@@ -699,7 +699,7 @@ contract DAO is DAOInterface, Token, TokenCreation {
     function newContract(address _newContract){
         if (msg.sender != address(this) || !allowedRecipients[_newContract]) return;
         // move all ether
-        if (!_newContract.call.value(address(this).balance)()) {
+        if (!_newContract.call.value(address(this).balance)("")) {
             revert();
         }
 

test/DAO/ManagedAccount.sol

@@ -57,7 +57,7 @@ contract ManagedAccount is ManagedAccountInterface{
     function payOut(address _recipient, uint _amount) returns (bool) {
         if (msg.sender != owner || msg.value > 0 || (payOwnerOnly && _recipient != owner))
             revert();
-        if (_recipient.call.value(_amount)()) {
+        if (_recipient.call.value(_amount)("")) {
             PayOut(_recipient, _amount);
             return true;
         } else {

test/DAO/TokenCreation.sol

@@ -105,7 +105,7 @@ contract TokenCreation is TokenCreationInterface, Token {
             && (privateCreation == 0x0000000000000000000000000000000000000000 || privateCreation == msg.sender)) {
 
             uint token = (msg.value * 20) / divisor();
-            extraBalance.call.value(msg.value - token)();
+            extraBalance.call.value(msg.value - token)("");
             balances[_tokenHolder] += token;
             totalSupply += token;
             weiGiven[_tokenHolder] += msg.value;
@@ -126,7 +126,7 @@ contract TokenCreation is TokenCreationInterface, Token {
                 extraBalance.payOut(address(this), extraBalance.accumulatedInput());
 
             // Execute refund
-            if (msg.sender.call.value(weiGiven[msg.sender])()) {
+            if (msg.sender.call.value(weiGiven[msg.sender])("")) {
                 Refund(msg.sender, weiGiven[msg.sender]);
                 totalSupply -= balances[msg.sender];
                 balances[msg.sender] = 0;