]> Cypherpunks repositories - gostls13.git/commitdiff
testing: fix output formatting
authorRobert Dinu <r@oktett.se>
Thu, 21 Feb 2013 22:17:43 +0000 (14:17 -0800)
committerRob Pike <r@golang.org>
Thu, 21 Feb 2013 22:17:43 +0000 (14:17 -0800)
Revision 5e7fd762f356 has changed the output formatting in a way that
is no longer in line with the format described by the revision
ff0ade0b937b which has introduced this functionality.
When decorating the first line, instead of indenting the whole line,
the current implementation adds indentation right after the "decorate"
part and  before the "log" message.
The fix addresses this issue.

R=golang-dev, iant, minux.ma, r, rsc, remyoudompheng
CC=golang-dev
https://golang.org/cl/7304094

src/pkg/testing/testing.go

index 357d6f5f2734fc4485351324cbbe76144102fe7f..b1dafd0c3d5ed7b2e231b8b768b442d3587e5671 100644 (file)
@@ -166,21 +166,17 @@ func decorate(s string) string {
                line = 1
        }
        buf := new(bytes.Buffer)
+       // Every line is indented at least one tab.
+       buf.WriteByte('\t')
        fmt.Fprintf(buf, "%s:%d: ", file, line)
-
        lines := strings.Split(s, "\n")
        if l := len(lines); l > 1 && lines[l-1] == "" {
                lines = lines[:l-1]
        }
        for i, line := range lines {
-               if i > 0 {
-                       buf.WriteByte('\n')
-               }
-               // Every line is indented at least one tab.
-               buf.WriteByte('\t')
                if i > 0 {
                        // Second and subsequent lines are indented an extra tab.
-                       buf.WriteByte('\t')
+                       buf.WriteString("\n\t\t")
                }
                buf.WriteString(line)
        }