From c87fb208c5162e2067138dd8fcea2e5ab8c1d6de Mon Sep 17 00:00:00 2001 From: griesemer Date: Tue, 10 Oct 2017 15:53:51 -0700 Subject: [PATCH] cmd/compile/internal/syntax: remove some outdated comments (cleanup) Change-Id: If242bb99d501420827b764c908580f2363e01ac4 Reviewed-on: https://go-review.googlesource.com/69730 Reviewed-by: Matthew Dempsky --- src/cmd/compile/internal/syntax/parser.go | 23 ----------------------- 1 file changed, 23 deletions(-) diff --git a/src/cmd/compile/internal/syntax/parser.go b/src/cmd/compile/internal/syntax/parser.go index f910052697..845e3f0eb7 100644 --- a/src/cmd/compile/internal/syntax/parser.go +++ b/src/cmd/compile/internal/syntax/parser.go @@ -484,20 +484,6 @@ func (p *parser) funcDeclOrNil() *FuncDecl { 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.Type = p.funcType() if p.tok == _Lbrace { @@ -506,14 +492,8 @@ func (p *parser) funcDeclOrNil() *FuncDecl { checkBranches(f.Body, p.errh) } } - 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 } @@ -1709,9 +1689,6 @@ func (p *parser) forStmt() Stmt { 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) { p.want(keyword) -- 2.48.1