make Print format gnostic

Author: Massimiliano Pippi

cli/feedback/exported.go

@@ -52,8 +52,8 @@ func Printf(format string, v ...interface{}) {
 }
 
 // Print behaves like fmt.Print but writes on the out writer and adds a newline.
-func Print(v ...interface{}) {
-	fb.Print(v...)
+func Print(v interface{}) {
+	fb.Print(v)
 }
 
 // Errorf behaves like fmt.Printf but writes on the error writer and adds a
@@ -74,7 +74,9 @@ func PrintJSON(v interface{}) {
 	fb.PrintJSON(v)
 }
 
-// PrintResult is a convenient wrapper...
+// PrintResult is a convenient wrapper to provide feedback for complex data,
+// where the contents can't be just serialized to JSON but requires more
+// structure.
 func PrintResult(res Result) {
 	fb.PrintResult(res)
 }

cli/feedback/feedback.go

@@ -86,8 +86,12 @@ func (fb *Feedback) Printf(format string, v ...interface{}) {
 }
 
 // Print behaves like fmt.Print but writes on the out writer and adds a newline.
-func (fb *Feedback) Print(v ...interface{}) {
-	fmt.Fprintln(fb.out, v...)
+func (fb *Feedback) Print(v interface{}) {
+	if fb.format == JSON {
+		fb.PrintJSON(v)
+	} else {
+		fmt.Fprintln(fb.out, v)
+	}
 }
 
 // Errorf behaves like fmt.Printf but writes on the error writer and adds a
@@ -109,18 +113,16 @@ func (fb *Feedback) PrintJSON(v interface{}) {
 	if d, err := json.MarshalIndent(v, "", "  "); err != nil {
 		fb.Errorf("Error during JSON encoding of the output: %v", err)
 	} else {
-		fb.Print(string(d))
+		fmt.Fprint(fb.out, string(d))
 	}
 }
 
-// PrintResult is a convenient wrapper...
+// PrintResult is a convenient wrapper to provide feedback for complex data,
+// where the contents can't be just serialized to JSON but requires more
+// structure.
 func (fb *Feedback) PrintResult(res Result) {
 	if fb.format == JSON {
-		if d, err := json.MarshalIndent(res.Data(), "", "  "); err != nil {
-			fb.Errorf("Error during JSON encoding of the output: %v", err)
-		} else {
-			fb.Print(string(d))
-		}
+		fb.PrintJSON(res.Data())
 	} else {
 		fb.Print(fmt.Sprintf("%s", res))
 	}