travis, appveyor: bump to Go 1.8.3, Android NDK 14b

core: fix various golint warnings and errors

1. fix typos
2. methods recevier of struct should be same
3. comments improve

(cherry picked from commit 1ba979539582a00b7fd1a7c8a37a6852e59eac0d)

Rewrite templates for (sub)commands help section

core/vm: allocate stack to 1024

Pre allocate the stack to 1024 optimising stack pushing, reducing calls
to runtime.makeslice and runtime.mallocgc

Enforce 256 bit keys on raw import, support raw mobile imports

consensus, core/*, params: metropolis preparation refactor

cmd/faucet: fix a few typos

Also add the package to the license tool ignore list.

cmd/geth: reintroduce wallet import subcommand

mobile: manage FilterQuery enabling contract events subscription

Byte padding function should return the given slice if the length is
smaller or equal rather than *only* smaller than.

This fix improves almost all EVM push operations.

Improve the duplication method of the stack to reuse big ints by passing
in an existing integer pool.

Improved push instructions by removing unnecessary big int allocations
and by making it int instead of big.Int

Signed-off-by: Lewis Marshall <lewis@lmars.net>

Fixes #14474.

Signed-off-by: Lewis Marshall <lewis@lmars.net>

More context in the bug This solves the problems of transactions being
submitted simultaneously, and getting the same nonce, due to the gap (due to
signing) between nonce-issuance and nonce-update. With this PR, a lock will
need to be acquired whenever a nonce is used, and released when the transaction
is submitted or errors out.

Add the object to the list of destructed objects during a selfdestruct /
suicide operation and also remove it from the list once the journal
reverts.