aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChen Quan <terasum@163.com>2018-07-31 15:48:27 +0800
committerPéter Szilágyi <peterke@gmail.com>2018-07-31 15:48:27 +0800
commitfb9f7261ec51e38eedb454594fc19f00de1a6834 (patch)
tree9d026596c599e832fd7deb8eb548248b4de837c9
parentd927cbb638419866de47929a723b5f2b8922c88e (diff)
downloadgo-tangerine-fb9f7261ec51e38eedb454594fc19f00de1a6834.tar
go-tangerine-fb9f7261ec51e38eedb454594fc19f00de1a6834.tar.gz
go-tangerine-fb9f7261ec51e38eedb454594fc19f00de1a6834.tar.bz2
go-tangerine-fb9f7261ec51e38eedb454594fc19f00de1a6834.tar.lz
go-tangerine-fb9f7261ec51e38eedb454594fc19f00de1a6834.tar.xz
go-tangerine-fb9f7261ec51e38eedb454594fc19f00de1a6834.tar.zst
go-tangerine-fb9f7261ec51e38eedb454594fc19f00de1a6834.zip
cmd/evm: change error msg output to stderr (#17118)
* cmd/evm: change error msg output to stderr * cmd/evm: fix some linter error
-rw-r--r--cmd/evm/compiler.go2
-rw-r--r--cmd/evm/disasm.go2
-rw-r--r--cmd/evm/internal/compiler/compiler.go5
-rw-r--r--cmd/evm/runner.go28
-rw-r--r--cmd/evm/staterunner.go8
5 files changed, 23 insertions, 22 deletions
diff --git a/cmd/evm/compiler.go b/cmd/evm/compiler.go
index c019a2fe7..f8aef2074 100644
--- a/cmd/evm/compiler.go
+++ b/cmd/evm/compiler.go
@@ -50,6 +50,6 @@ func compileCmd(ctx *cli.Context) error {
if err != nil {
return err
}
- fmt.Println(bin)
+ fmt.Fprintln(ctx.App.Writer, bin)
return nil
}
diff --git a/cmd/evm/disasm.go b/cmd/evm/disasm.go
index 4a442cf78..0f3491818 100644
--- a/cmd/evm/disasm.go
+++ b/cmd/evm/disasm.go
@@ -45,6 +45,6 @@ func disasmCmd(ctx *cli.Context) error {
}
code := strings.TrimSpace(string(in[:]))
- fmt.Printf("%v\n", code)
+ fmt.Fprintf(ctx.App.Writer, "%v\n", code)
return asm.PrintDisassembled(code)
}
diff --git a/cmd/evm/internal/compiler/compiler.go b/cmd/evm/internal/compiler/compiler.go
index 753ca6226..a2abc8f7d 100644
--- a/cmd/evm/internal/compiler/compiler.go
+++ b/cmd/evm/internal/compiler/compiler.go
@@ -30,10 +30,11 @@ func Compile(fn string, src []byte, debug bool) (string, error) {
bin, compileErrors := compiler.Compile()
if len(compileErrors) > 0 {
// report errors
+ errs := ""
for _, err := range compileErrors {
- fmt.Printf("%s:%v\n", fn, err)
+ errs += fmt.Sprintf("%s:%v\n", fn, err)
}
- return "", errors.New("compiling failed")
+ return "", errors.New(errs + "compiling failed\n")
}
return bin, nil
}
diff --git a/cmd/evm/runner.go b/cmd/evm/runner.go
index 7138a9ddd..fed41f53a 100644
--- a/cmd/evm/runner.go
+++ b/cmd/evm/runner.go
@@ -128,13 +128,13 @@ func runCmd(ctx *cli.Context) error {
if ctx.GlobalString(CodeFileFlag.Name) == "-" {
//Try reading from stdin
if hexcode, err = ioutil.ReadAll(os.Stdin); err != nil {
- fmt.Printf("Could not load code from stdin: %v\n", err)
+ fmt.Fprintf(ctx.App.ErrWriter, "Could not load code from stdin: %v\n", err)
os.Exit(1)
}
} else {
// Codefile with hex assembly
if hexcode, err = ioutil.ReadFile(ctx.GlobalString(CodeFileFlag.Name)); err != nil {
- fmt.Printf("Could not load code from file: %v\n", err)
+ fmt.Fprintf(ctx.App.ErrWriter, "Could not load code from file: %v\n", err)
os.Exit(1)
}
}
@@ -172,11 +172,11 @@ func runCmd(ctx *cli.Context) error {
if cpuProfilePath := ctx.GlobalString(CPUProfileFlag.Name); cpuProfilePath != "" {
f, err := os.Create(cpuProfilePath)
if err != nil {
- fmt.Println("could not create CPU profile: ", err)
+ fmt.Fprintf(ctx.App.ErrWriter, "could not create CPU profile: %v\n", err)
os.Exit(1)
}
if err := pprof.StartCPUProfile(f); err != nil {
- fmt.Println("could not start CPU profile: ", err)
+ fmt.Fprintf(ctx.App.ErrWriter, "could not start CPU profile: %v\n", err)
os.Exit(1)
}
defer pprof.StopCPUProfile()
@@ -200,17 +200,17 @@ func runCmd(ctx *cli.Context) error {
if ctx.GlobalBool(DumpFlag.Name) {
statedb.IntermediateRoot(true)
- fmt.Println(string(statedb.Dump()))
+ fmt.Fprintln(ctx.App.Writer, string(statedb.Dump()))
}
if memProfilePath := ctx.GlobalString(MemProfileFlag.Name); memProfilePath != "" {
f, err := os.Create(memProfilePath)
if err != nil {
- fmt.Println("could not create memory profile: ", err)
+ fmt.Fprintf(ctx.App.ErrWriter, "could not create memory profile: %v\n", err)
os.Exit(1)
}
if err := pprof.WriteHeapProfile(f); err != nil {
- fmt.Println("could not write memory profile: ", err)
+ fmt.Fprintf(ctx.App.ErrWriter, "could not create memory profile: %v\n", err)
os.Exit(1)
}
f.Close()
@@ -218,17 +218,17 @@ func runCmd(ctx *cli.Context) error {
if ctx.GlobalBool(DebugFlag.Name) {
if debugLogger != nil {
- fmt.Fprintln(os.Stderr, "#### TRACE ####")
- vm.WriteTrace(os.Stderr, debugLogger.StructLogs())
+ fmt.Fprintln(ctx.App.ErrWriter, "#### TRACE ####")
+ vm.WriteTrace(ctx.App.ErrWriter, debugLogger.StructLogs())
}
- fmt.Fprintln(os.Stderr, "#### LOGS ####")
- vm.WriteLogs(os.Stderr, statedb.Logs())
+ fmt.Fprintln(ctx.App.ErrWriter, "#### LOGS ####")
+ vm.WriteLogs(ctx.App.ErrWriter, statedb.Logs())
}
if ctx.GlobalBool(StatDumpFlag.Name) {
var mem goruntime.MemStats
goruntime.ReadMemStats(&mem)
- fmt.Fprintf(os.Stderr, `evm execution time: %v
+ fmt.Fprintf(ctx.App.ErrWriter, `evm execution time: %v
heap objects: %d
allocations: %d
total allocations: %d
@@ -238,9 +238,9 @@ Gas used: %d
`, execTime, mem.HeapObjects, mem.Alloc, mem.TotalAlloc, mem.NumGC, initialGas-leftOverGas)
}
if tracer == nil {
- fmt.Printf("0x%x\n", ret)
+ fmt.Fprintf(ctx.App.Writer, "0x%x\n", ret)
if err != nil {
- fmt.Printf(" error: %v\n", err)
+ fmt.Fprintf(ctx.App.ErrWriter, " error: %v\n", err)
}
}
diff --git a/cmd/evm/staterunner.go b/cmd/evm/staterunner.go
index 6d5ff069f..bee1b34dc 100644
--- a/cmd/evm/staterunner.go
+++ b/cmd/evm/staterunner.go
@@ -107,7 +107,7 @@ func stateTestCmd(ctx *cli.Context) error {
}
// print state root for evmlab tracing (already committed above, so no need to delete objects again
if ctx.GlobalBool(MachineFlag.Name) && state != nil {
- fmt.Fprintf(os.Stderr, "{\"stateRoot\": \"%x\"}\n", state.IntermediateRoot(false))
+ fmt.Fprintf(ctx.App.ErrWriter, "{\"stateRoot\": \"%x\"}\n", state.IntermediateRoot(false))
}
results = append(results, *result)
@@ -115,13 +115,13 @@ func stateTestCmd(ctx *cli.Context) error {
// Print any structured logs collected
if ctx.GlobalBool(DebugFlag.Name) {
if debugger != nil {
- fmt.Fprintln(os.Stderr, "#### TRACE ####")
- vm.WriteTrace(os.Stderr, debugger.StructLogs())
+ fmt.Fprintln(ctx.App.ErrWriter, "#### TRACE ####")
+ vm.WriteTrace(ctx.App.ErrWriter, debugger.StructLogs())
}
}
}
}
out, _ := json.MarshalIndent(results, "", " ")
- fmt.Println(string(out))
+ fmt.Fprintln(ctx.App.Writer, string(out))
return nil
}