From: Robert Griesemer <gri@golang.org>
Date: Wed, 3 Dec 2008 01:01:31 +0000 (-0800)
Subject: - oversight: extra allow source-level line breaks inside statement lists
X-Git-Tag: weekly.2009-11-06~2588
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=acfd1fd4438060eebf929778cfef982f0b03e0f2;p=gostls13.git

- oversight: extra allow source-level line breaks inside statement lists
- fixed printing of empty structs/interfaces
- enable two more tests

R=r
OCL=20296
CL=20296
---

diff --git a/usr/gri/pretty/printer.go b/usr/gri/pretty/printer.go
index f1da8c3d39..29c65e2b1e 100644
--- a/usr/gri/pretty/printer.go
+++ b/usr/gri/pretty/printer.go
@@ -348,7 +348,7 @@ func (P *Printer) Fields(list *array.Array, end int) {
 	P.state = opening_scope;
 	P.String(0, "{");
 
-	if list != nil {
+	if list.Len() > 0 {
 		P.newlines = 1;
 		var prev int;
 		for i, n := 0, list.Len(); i < n; i++ {
@@ -547,6 +547,7 @@ func (P *Printer) StatementList(list *array.Array) {
 		for i, n := 0, list.Len(); i < n; i++ {
 			P.Stat(list.At(i).(*AST.Stat));
 			P.newlines = 1;
+			P.state = inside_list;
 		}
 	}
 }
diff --git a/usr/gri/pretty/selftest2.go b/usr/gri/pretty/selftest2.go
index 3062399329..eb7dcb2e2f 100644
--- a/usr/gri/pretty/selftest2.go
+++ b/usr/gri/pretty/selftest2.go
@@ -25,6 +25,9 @@ const /* enum */ (
 )
 
 
+type S struct {}
+
+
 type T struct {
 	x, y int;
 	s string;
diff --git a/usr/gri/pretty/test.sh b/usr/gri/pretty/test.sh
index 4840778d5f..3b8d6e6871 100755
--- a/usr/gri/pretty/test.sh
+++ b/usr/gri/pretty/test.sh
@@ -23,7 +23,7 @@ apply1() {
 	#echo $1 $2
 	case `basename $F` in
 	# these files don't pass the idempotency test yet
-	log.go | decimal.go | type.go | tabwriter_test.go | \
+	log.go | type.go | \
 	\
 	selftest1.go | func3.go | bug014.go | bug029.go | bug032.go | bug050.go | \
 	bug068.go | bug088.go | bug083.go | bug106.go | bug125.go ) ;;  # skip - files contain syntax errors