code review 223058: go/printer, gofmt: align comments in multi-line express...

src/pkg/go/printer/printer.go

 // Copyright 2009 The Go Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style
 // license that can be found in the LICENSE file.
 
 // The printer package implements printing of AST nodes.
 package printer
 
 import (
         "bytes"
         "fmt"
@@ skipping 22 matching lines @@
         newline  = whiteSpace('\n')
         formfeed = whiteSpace('\f')
         indent   = whiteSpace('>')
         unindent = whiteSpace('<')
 )
 
 
 var (
         esc       = []byte{tabwriter.Escape}
         htab      = []byte{'\t'}
-        htabs     = [...]byte{'\t', '\t', '\t', '\t', '\t', '\t', '\t', '\t'}
-        newlines  = [...]byte{'\n', '\n', '\n', '\n', '\n', '\n', '\n', '\n'} // more than maxNewlines
-        formfeeds = [...]byte{'\f', '\f', '\f', '\f', '\f', '\f', '\f', '\f'} // more than maxNewlines
+        htabs     = []byte("\t\t\t\t\t\t\t\t")
+        newlines  = []byte("\n\n\n\n\n\n\n\n") // more than maxNewlines
+        formfeeds = []byte("\f\f\f\f\f\f\f\f") // more than maxNewlines
 
         esc_quot = []byte("&#34;") // shorter than "&quot;"
         esc_apos = []byte("&#39;") // shorter than "&apos;"
         esc_amp  = []byte("&amp;")
         esc_lt   = []byte("&lt;")
         esc_gt   = []byte("&gt;")
 )
 
 
 // Use noPos when a position is needed but not known.
@@ skipping 84 matching lines @@
                         p.pos.Offset += i + 1 - i0
                         p.pos.Line++
                         p.pos.Column = 1
 
                         if !p.escape {
                                 // write indentation
                                 // use "hard" htabs - indentation columns
                                 // must not be discarded by the tabwriter
                                 j := p.indent
                                 for ; j > len(htabs); j -= len(htabs) {
-                                        p.write0(&htabs)
+                                        p.write0(htabs)
                                 }
                                 p.write0(htabs[0:j])
 
                                 // update p.pos
                                 p.pos.Offset += p.indent
                                 p.pos.Column += p.indent
                         }
 
                         // next segment start
                         i0 = i + 1
@@ skipping 358 matching lines @@
                         if bytes.HasSuffix(prefix, suffix) {
                                 prefix = prefix[0 : len(prefix)-len(suffix)]
                         }
                 }
         }
 
         // Handle last line: If it only contains a closing */, align it
         // with the opening /*, otherwise align the text with the other
         // lines.
         last := lines[len(lines)-1]
-        closing := []byte{'*', '/'}
+        closing := []byte("*/")
         i := bytes.Index(last, closing)
         if isBlank(last[0:i]) {
                 // last line only contains closing */
                 var sep []byte
                 if lineOfStars {
                         // insert an aligning blank
                         sep = []byte{' '}
                 }
                 lines[len(lines)-1] = bytes.Join([][]byte{prefix, closing}, sep)
         } else {
@@ skipping 468 matching lines @@
 }
 
 
 // Fprint "pretty-prints" an AST node to output.
 // It calls Config.Fprint with default settings.
 //
 func Fprint(output io.Writer, node interface{}) os.Error {
         _, err := (&Config{Tabwidth: 8}).Fprint(output, node) // don't care about number of bytes written
         return err
 }