cmd/compile/internal/syntax: remove some outdated comments (cleanup)

Change-Id: If242bb99d501420827b764c908580f2363e01ac4
Reviewed-on: https://go-review.googlesource.com/69730
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
This commit is contained in:
griesemer 2017-10-10 15:53:51 -07:00 committed by Robert Griesemer
parent 3a165bba34
commit c87fb208c5

View file

@ -484,20 +484,6 @@ func (p *parser) funcDeclOrNil() *FuncDecl {
return nil return nil
} }
// TODO(gri) check for regular functions only
// if name.Sym.Name == "init" {
// name = renameinit()
// if params != nil || result != nil {
// p.error("func init must have no arguments and no return values")
// }
// }
// if localpkg.Name == "main" && name.Name == "main" {
// if params != nil || result != nil {
// p.error("func main must have no arguments and no return values")
// }
// }
f.Name = p.name() f.Name = p.name()
f.Type = p.funcType() f.Type = p.funcType()
if p.tok == _Lbrace { if p.tok == _Lbrace {
@ -506,14 +492,8 @@ func (p *parser) funcDeclOrNil() *FuncDecl {
checkBranches(f.Body, p.errh) checkBranches(f.Body, p.errh)
} }
} }
f.Pragma = p.pragma f.Pragma = p.pragma
// TODO(gri) deal with function properties
// if noescape && body != nil {
// p.error("can only use //go:noescape with external func implementations")
// }
return f return f
} }
@ -1709,9 +1689,6 @@ func (p *parser) forStmt() Stmt {
return s return s
} }
// TODO(gri) This function is now so heavily influenced by the keyword that
// it may not make sense anymore to combine all three cases. It
// may be simpler to just split it up for each statement kind.
func (p *parser) header(keyword token) (init SimpleStmt, cond Expr, post SimpleStmt) { func (p *parser) header(keyword token) (init SimpleStmt, cond Expr, post SimpleStmt) {
p.want(keyword) p.want(keyword)