[go] go/parser: simplify expectSemi

0 views
Skip to first unread message

Emmanuel Odeke (Gerrit)

unread,
1:22 AM (22 hours ago) 1:22 AM
to Mateusz Poliwczak, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Michael Knyszek, Go LUCI, Alan Donovan, golang-co...@googlegroups.com

Emmanuel Odeke submitted the change

Unreviewed changes

2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Change information

Commit message:
go/parser: simplify expectSemi

Move the outer if statement, directly into the switch.
Change-Id: I6a6a6964ff15dbcda4f4c9ae1c15423adbfb0ae5
Reviewed-by: Michael Knyszek <mkny...@google.com>
Reviewed-by: Alan Donovan <adon...@google.com>
Files:
  • M src/go/parser/parser.go
Change size: S
Delta: 1 file changed, 21 insertions(+), 22 deletions(-)
Branch: refs/heads/master
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Michael Knyszek, +2 by Alan Donovan
  • requirement satisfiedTryBots-Pass: LUCI-TryBot-Result+1 by Go LUCI
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: I6a6a6964ff15dbcda4f4c9ae1c15423adbfb0ae5
Gerrit-Change-Number: 703835
Gerrit-PatchSet: 4
Gerrit-Owner: Mateusz Poliwczak <mpoliw...@gmail.com>
Gerrit-Reviewer: Alan Donovan <adon...@google.com>
Gerrit-Reviewer: Emmanuel Odeke <emma...@orijtech.com>
Gerrit-Reviewer: Mateusz Poliwczak <mpoliw...@gmail.com>
Gerrit-Reviewer: Michael Knyszek <mkny...@google.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages