mirror of
https://github.com/golang/go
synced 2024-09-15 22:20:06 +00:00
cmd/compile: don't exit early because of hidden error messages
Non-syntax errors are always counted to determine if to exit early, but then deduplication eliminates them. This can lead to situations which report "too many errors" and only one error is shown. De-duplicate non-syntax errors early, at least the ones that appear consecutively, and only count the ones actually being shown. This doesn't work perfectly as they may not appear in sequence, but it's cheap and good enough. Fixes #14136. Change-Id: I7b11ebb2e1e082f0d604b88e544fe5ba967af1d7 Reviewed-on: https://go-review.googlesource.com/23259 Reviewed-by: Matthew Dempsky <mdempsky@google.com>
This commit is contained in:
parent
dc4427f372
commit
448246adff
|
@ -87,8 +87,39 @@ func linestr(line int32) string {
|
|||
return Ctxt.Line(int(line))
|
||||
}
|
||||
|
||||
// lasterror keeps track of the most recently issued error.
|
||||
// It is used to avoid multiple error messages on the same
|
||||
// line.
|
||||
var lasterror struct {
|
||||
syntax int32 // line of last syntax error
|
||||
other int32 // line of last non-syntax error
|
||||
msg string // error message of last non-syntax error
|
||||
}
|
||||
|
||||
func yyerrorl(line int32, format string, args ...interface{}) {
|
||||
adderr(line, format, args...)
|
||||
msg := fmt.Sprintf(format, args...)
|
||||
|
||||
if strings.HasPrefix(msg, "syntax error") {
|
||||
nsyntaxerrors++
|
||||
// only one syntax error per line, no matter what error
|
||||
if lasterror.syntax == line {
|
||||
return
|
||||
}
|
||||
lasterror.syntax = line
|
||||
} else {
|
||||
// only one of multiple equal non-syntax errors per line
|
||||
// (Flusherrors shows only one of them, so we filter them
|
||||
// here as best as we can (they may not appear in order)
|
||||
// so that we don't count them here and exit early, and
|
||||
// then have nothing to show for.)
|
||||
if lasterror.other == line && lasterror.msg == msg {
|
||||
return
|
||||
}
|
||||
lasterror.other = line
|
||||
lasterror.msg = msg
|
||||
}
|
||||
|
||||
adderr(line, "%s", msg)
|
||||
|
||||
hcrash()
|
||||
nerrors++
|
||||
|
@ -99,32 +130,8 @@ func yyerrorl(line int32, format string, args ...interface{}) {
|
|||
}
|
||||
}
|
||||
|
||||
var yyerror_lastsyntax int32
|
||||
|
||||
func Yyerror(format string, args ...interface{}) {
|
||||
msg := fmt.Sprintf(format, args...)
|
||||
if strings.HasPrefix(msg, "syntax error") {
|
||||
nsyntaxerrors++
|
||||
|
||||
// only one syntax error per line
|
||||
if yyerror_lastsyntax == lineno {
|
||||
return
|
||||
}
|
||||
yyerror_lastsyntax = lineno
|
||||
|
||||
yyerrorl(lineno, "%s", msg)
|
||||
return
|
||||
}
|
||||
|
||||
adderr(lineno, "%s", msg)
|
||||
|
||||
hcrash()
|
||||
nerrors++
|
||||
if nsavederrors+nerrors >= 10 && Debug['e'] == 0 {
|
||||
Flusherrors()
|
||||
fmt.Printf("%v: too many errors\n", linestr(lineno))
|
||||
errorexit()
|
||||
}
|
||||
yyerrorl(lineno, format, args...)
|
||||
}
|
||||
|
||||
func Warn(fmt_ string, args ...interface{}) {
|
||||
|
|
19
test/fixedbugs/issue14136.go
Normal file
19
test/fixedbugs/issue14136.go
Normal file
|
@ -0,0 +1,19 @@
|
|||
// errorcheck
|
||||
|
||||
// Copyright 2016 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.
|
||||
|
||||
// Test that > 10 non-syntax errors on the same line
|
||||
// don't lead to early exit. Specifically, here test
|
||||
// that we see the initialization error for variable
|
||||
// s.
|
||||
|
||||
package main
|
||||
|
||||
type T struct{}
|
||||
|
||||
func main() {
|
||||
t := T{X: 1, X: 1, X: 1, X: 1, X: 1, X: 1, X: 1, X: 1, X: 1, X: 1} // ERROR "unknown T field"
|
||||
var s string = 1 // ERROR "cannot use 1"
|
||||
}
|
Loading…
Reference in a new issue