diff options
author | Felix Lange <fjl@twurst.com> | 2016-03-21 21:05:22 +0800 |
---|---|---|
committer | Felix Lange <fjl@twurst.com> | 2016-04-12 21:56:49 +0800 |
commit | dff9b4246f3ef9e6c254b57eef6d0433809f16b9 (patch) | |
tree | 71cab4093300a6e9bd1b5801f05e3507246eef7e /cmd/geth/chaincmd.go | |
parent | 83877a0f9d3d5716ee01393f10c2dfa19bb0310b (diff) | |
download | dexon-dff9b4246f3ef9e6c254b57eef6d0433809f16b9.tar.gz dexon-dff9b4246f3ef9e6c254b57eef6d0433809f16b9.tar.zst dexon-dff9b4246f3ef9e6c254b57eef6d0433809f16b9.zip |
cmd/geth, cmd/utils: improve input handling
These changes make prompting behave consistently on all platforms:
* The input buffer is now global.
Buffering was previously set up for each prompt, which can cause weird
behaviour, e.g. when running "geth account update <input.txt" where
input.txt contains three lines. In this case, the first password
prompt would fill up the buffer with all lines and then use only the
first one.
* Print the "unsupported terminal" warning only once.
Now that stdin prompting has global state, we can use it to track
the warning there.
* Work around small liner issues, particularly on Windows.
Prompting didn't work under most of the third-party terminal emulators
on Windows because liner assumes line editing is always available.
Diffstat (limited to 'cmd/geth/chaincmd.go')
-rw-r--r-- | cmd/geth/chaincmd.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cmd/geth/chaincmd.go b/cmd/geth/chaincmd.go index 868ee7db1..32eacc99e 100644 --- a/cmd/geth/chaincmd.go +++ b/cmd/geth/chaincmd.go @@ -116,7 +116,7 @@ func exportChain(ctx *cli.Context) { } func removeDB(ctx *cli.Context) { - confirm, err := utils.PromptConfirm("Remove local database?") + confirm, err := utils.Stdin.ConfirmPrompt("Remove local database?") if err != nil { utils.Fatalf("%v", err) } |