code review 4307045: go/printer/gofmt: remove special case for multi-line ra...

go/printer/gofmt: remove special case for multi-line raw strings

As a special case, multi-line raw strings (i.e., strings in `` quotes)
were not indented if they were the only token on a line. This heuristic
was meant to improve formatting for multi-line raw strings where sub-
sequent lines are not indented at the level of the surrounding code.
Multiple people have complained about this. Removing the heuristic
again because it makes the formatting more regular, easier to under-
stand, and simplifies the implementation.

- manual changes to ebnf/ebnf_test.go for readability
- gofmt -w src misc

Fixes issue 1643.

[gri, 2011-03-29 22:54:44]

Hello r, rsc (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://go.googlecode.com/hg/

[r, 2011-03-30 00:05:12]

leaving for rsc. his code seems to get hit hard.

[rsc, 2011-03-30 00:47:11]

LGTM

[gri, 2011-03-30 01:31:05]

*** Submitted as http://code.google.com/p/go/source/detail?r=342e3b11f21a ***

go/printer/gofmt: remove special case for multi-line raw strings

As a special case, multi-line raw strings (i.e., strings in `` quotes)
were not indented if they were the only token on a line. This heuristic
was meant to improve formatting for multi-line raw strings where sub-
sequent lines are not indented at the level of the surrounding code.
Multiple people have complained about this. Removing the heuristic
again because it makes the formatting more regular, easier to under-
stand, and simplifies the implementation.

- manual changes to ebnf/ebnf_test.go for readability
- gofmt -w src misc

Fixes issue 1643.

R=r, rsc
CC=golang-dev
http://codereview.appspot.com/4307045