mirror of
https://github.com/golang/go
synced 2024-11-05 18:46:11 -07:00
70544c91ff
This makes a bunch of changes to package syntax to tweak line numbers for AST nodes. For example, short variable declaration statements are now associated with the location of the ":=" token, and function calls are associated with the location of the final ")" token. These help satisfy many unit tests that assume the old parser's behavior. Because many of these changes are questionable, they're guarded behind a new "gcCompat" const to make them easy to identify and revisit in the future. A handful of remaining tests are too difficult to make behave identically. These have been updated to execute with -newparser=0 and comments explaining why they need to be fixed. all.bash now passes with both the old and new parsers. Change-Id: Iab834b71ca8698d39269f261eb5c92a0d55a3bf4 Reviewed-on: https://go-review.googlesource.com/27199 Run-TryBot: Matthew Dempsky <mdempsky@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Robert Griesemer <gri@golang.org>
18 lines
580 B
Go
18 lines
580 B
Go
// errorcheck -newparser=0
|
|
|
|
// Copyright 2010 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.
|
|
|
|
// TODO(mdempsky): Update error expectations for new parser.
|
|
// The new parser emits an extra "missing { after for clause" error.
|
|
// The old parser is supposed to emit this too, but it panics first
|
|
// due to a nil pointer dereference.
|
|
|
|
package main
|
|
|
|
func main() {
|
|
for x // GCCGO_ERROR "undefined"
|
|
{ // ERROR "missing .*{.* after for clause|missing operand"
|
|
z // GCCGO_ERROR "undefined"
|