accounts/abi: estimateGas before send transaction

This commit is contained in:
nkbai 2017-12-28 09:00:20 +08:00
parent cd918b7731
commit 9417a681bc
2 changed files with 17 additions and 16 deletions

View file

@ -189,7 +189,6 @@ func (c *BoundContract) transact(opts *TransactOpts, contract *common.Address, i
}
}
gasLimit := opts.GasLimit
if gasLimit == nil {
// Gas estimation cannot succeed without code for method invocations
if contract != nil {
if code, err := c.transactor.PendingCodeAt(ensureContext(opts.Context), c.address); err != nil {
@ -204,6 +203,8 @@ func (c *BoundContract) transact(opts *TransactOpts, contract *common.Address, i
if err != nil {
return nil, fmt.Errorf("failed to estimate gas needed: %v", err)
}
if opts.GasLimit != nil && opts.GasLimit.Cmp(gasLimit) < 0 {
return nil, fmt.Errorf("Gas required is %s, it exceeds limit: %s. An important gas estimation might also be the sign of a problem in the contract code", gasLimit, opts.GasLimit)
}
// Create the transaction, sign it and schedule it for execution
var rawTx *types.Transaction

View file

@ -45,8 +45,8 @@ func NewClient(c *rpc.Client) *Client {
}
// Version returns the Whisper sub-protocol version.
func (sc *Client) Version(ctx context.Context) (string, error) {
var result string
func (sc *Client) Version(ctx context.Context) (uint, error) {
var result uint
err := sc.c.CallContext(ctx, &result, "shh_version")
return result, err
}