Merge pull request #556 from ethereum/fixGasLimit

fix gas Limit
pull/1/head
yann300 8 years ago committed by GitHub
commit 6a7159f34a
  1. 10
      src/app/txRunner.js

@ -75,16 +75,12 @@ TxRunner.prototype.execute = function () {
} else { } else {
// NOTE: estimateGas very likely will return a large limit if execution of the code failed // NOTE: estimateGas very likely will return a large limit if execution of the code failed
// we want to be able to run the code in order to debug and find the cause for the failure // we want to be able to run the code in order to debug and find the cause for the failure
var blockGasLimit = Math.floor(block.gasLimit - block.gasLimit / 1024) // we can't use the blockGasLimit cause the next blocks could have a lower limit : https://github.com/ethereum/remix/issues/506
// tx.gas = blockGasLimit < gasEstimation ? blockGasLimit : gasEstimation var blockGasLimit = Math.floor(block.gasLimit - (5 * block.gasLimit) / 1024)
tx.gas = blockGasLimit // that's temporary, uncomment when the following is fixed https://github.com/ethereum/go-ethereum/issues/3653 tx.gas = blockGasLimit
/* same as above
if (tx.gas > gasLimit) { if (tx.gas > gasLimit) {
return callback('Gas required exceeds limit: ' + tx.gas) return callback('Gas required exceeds limit: ' + tx.gas)
} }
*/
var sendTransaction = self.personalMode ? self.web3.personal.sendTransaction : self.web3.eth.sendTransaction var sendTransaction = self.personalMode ? self.web3.personal.sendTransaction : self.web3.eth.sendTransaction
sendTransaction(tx, function (err, resp) { sendTransaction(tx, function (err, resp) {
if (err) { if (err) {

Loading…
Cancel
Save