Better CLI UX (#8)

Author: cristaloleg

README.md

@@ -20,7 +20,7 @@ The original implementation was started by [@quasilyte](https://github.com/quasi
 $ go get -u github.com/go-perf/go-perftuner
 
 # Check installation (prints help):
-$ go-perftuner
+$ go-perftuner help
 
 # Run almostInlined sub-command on strings and bytes package:
 $ go-perftuner almostInlined strings bytes

almostInlined.go

@@ -3,7 +3,6 @@ package main
 import (
 	"flag"
 	"fmt"
-	"log"
 	"os/exec"
 	"regexp"
 )
@@ -61,7 +60,7 @@ func (r *almostInlinedRunner) Run(pkg string) error {
 	}
 
 	for _, r := range results {
-		log.Printf("%s: %s: budget exceeded by %d\n", r.Loc, r.Fn, r.Diff)
+		fmt.Printf("%s: %s: budget exceeded by %d\n", r.Loc, r.Fn, r.Diff)
 	}
 	return nil
 }

boundChecks.go

@@ -2,7 +2,6 @@ package main
 
 import (
 	"fmt"
-	"log"
 	"os/exec"
 	"regexp"
 )
@@ -42,7 +41,7 @@ func (r *boundCheckRunner) Run(pkg string) error {
 	}
 
 	for _, r := range results {
-		log.Printf("%s: slice/array has bound checks\n", r.Loc)
+		fmt.Printf("%s: slice/array has bound checks\n", r.Loc)
 	}
 	return nil
 }

escapeanalysis.go

@@ -2,7 +2,6 @@ package main
 
 import (
 	"fmt"
-	"log"
 	"os/exec"
 	"regexp"
 )
@@ -47,7 +46,7 @@ func (r *escapeAnalysisRunner) Run(pkg string) error {
 	}
 
 	for _, r := range results {
-		log.Printf("%s: %s\n", r.Loc, r.Variable)
+		fmt.Printf("%s: %s\n", r.Loc, r.Variable)
 	}
 	return nil
 }

go.mod

@@ -2,4 +2,4 @@ module github.com/go-perf/go-perftuner
 
 go 1.10
 
-require github.com/cristalhq/acmd v0.3.0
+require github.com/cristalhq/acmd v0.5.0

go.sum

@@ -1,2 +1,2 @@
-github.com/cristalhq/acmd v0.3.0 h1:P9/h5Mn5OlRCEu1+b7bFWMBPenC6LORwc76vpf/GK0E=
-github.com/cristalhq/acmd v0.3.0/go.mod h1:LG5oa43pE/BbxtfMoImHCQN++0Su7dzipdgBjMCBVDQ=
+github.com/cristalhq/acmd v0.5.0 h1:0bzePcUUuzazlaAJoZmDTnL9396RwqrdA16m5+LpBhY=
+github.com/cristalhq/acmd v0.5.0/go.mod h1:LG5oa43pE/BbxtfMoImHCQN++0Su7dzipdgBjMCBVDQ=

main.go

@@ -4,7 +4,6 @@ import (
 	"context"
 	"flag"
 	"fmt"
-	"log"
 	"os"
 	"strings"
 
@@ -19,11 +18,6 @@ var (
 )
 
 func main() {
-	if len(os.Args) < 2 {
-		panic("not enough arguments, expected sub-command name")
-	}
-
-	log.SetFlags(0)
 	flag.StringVar(&flagMod, "mod", "", `-mod compiler flag(readonly|vendor)`)
 	flag.BoolVar(&asJSON, "json", false, `return result as JSON`)
 	flag.Parse()
@@ -32,7 +26,8 @@ func main() {
 		Version: version,
 	})
 	if err := r.Run(); err != nil {
-		log.Fatal(fmt.Errorf("dbumper: %w", err))
+		fmt.Printf("go-perf: %v\n", err)
+		os.Exit(1)
 	}
 }
 
@@ -72,7 +67,7 @@ func run(cmd subCommandRunner) error {
 	cmd.Init()
 	for _, pkg := range flag.Args()[1:] {
 		if err := cmd.Run(pkg); err != nil {
-			log.Printf("%s: %v", pkg, err)
+			fmt.Printf("%s: %v", pkg, err)
 		}
 	}
 	return nil