code review 181050: Add a test for issue 337.

Add a test for issue 337.

gccgo currently miscompiles this test.

[iant, 2009-12-24 01:47:46]

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

I'd like you to review the following change.

[r2, 2009-12-24 01:52:14]

LGTM

On 24/12/2009, at 12:47 PM, iant@golang.org wrote:

> Reviewers: rsc,
>
> Message:
> Hello rsc (cc: golang-dev@googlegroups.com),
>
> I'd like you to review the following change.
>
>
> Description:
> Add a test for issue 337.
>
> gccgo currently miscompiles this test.
>
> Please review this at http://codereview.appspot.com/181050
>
> Affected files:
>  A test/fixedbugs/bug237.go
>
>
> Index: test/fixedbugs/bug237.go
> ===================================================================
> new file mode 100644
> --- /dev/null
> +++ b/test/fixedbugs/bug237.go
> @@ -0,0 +1,25 @@
> +// $G $D/$F.go && $L $F.$A && ./$A.out
> +
> +// 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.
> +
> +package main
> +
> +import "fmt"
> +
> +var indent uint = 10
> +func main() {
> +	const dots =  
> ". . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . " +
> +		". . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . . "
> +	const n = uint(len(dots))
> +	i := 2 * indent
> +	var s string
> +	for ; i > n; i -= n {
> +		s += fmt.Sprint(dots)
> +	}
> +	s += dots[0:i]
> +	if s != ". . . . . . . . . . " {
> +		panic(s)
> +	}
> +}
>
>

[iant, 2009-12-24 06:08:36]

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

Add a test for issue 337.

gccgo currently miscompiles this test.

R=rsc
CC=golang-dev
http://codereview.appspot.com/181050

[nictuku, 2009-12-28 01:30:36]

FYI

The filed was named bug237.go but should be bug337.go.

[r2, 2009-12-28 08:19:04]

On 28/12/2009, at 12:30 PM, yves.junqueira@gmail.com wrote:

> FYI
>
> The filed was named bug237.go but should be bug337.go.
>
> http://codereview.appspot.com/181050

No. The similarity is a coincidence.  The bug tests are numbered  
sequentially, and this is the 237th recorded.

-rob