rancher/os
1
0
Fork 1
mirror of https://github.com/rancher/os.git synced 2025-09-15 22:49:08 +00:00
Code Issues Releases Wiki Activity

Merge pull request #1628 from juliengk/yes_issue_1597

Browse Source 
Use of bufio instead of fmt.Scan for yes function
This commit is contained in:
Sven Dowideit
2017-02-18 03:21:32 +10:00
committed by GitHub
parent db5da1ebfe 14257bdde8
commit 77cd10e840
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
cmd/control/util.go
View File

@@ -1,7 +1,9 @@
package control package control
import ( import (
"bufio"
"fmt" "fmt"
"os"
"strings" "strings"
"github.com/rancher/os/log" "github.com/rancher/os/log"
@@ -9,8 +11,8 @@ import (
func yes(question string) bool { func yes(question string) bool {
fmt.Printf("%s [y/N]: ", question) fmt.Printf("%s [y/N]: ", question)
var line string in := bufio.NewReader(os.Stdin)
_, err := fmt.Scan(&line) line, err := in.ReadString('\n')
if err != nil { if err != nil {
log.Fatal(err) log.Fatal(err)
} }