-
Notifications
You must be signed in to change notification settings - Fork 17.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
go/ast: add func Unparen(Expr) Expr #60061
Comments
Change https://go.dev/cl/495315 mentions this issue: |
Based on the discussion above, this proposal seems like a likely accept. |
No change in consensus, so accepted. 🎉 |
Change https://go.dev/cl/548882 mentions this issue: |
Updates #60061 Change-Id: Ifa51f0bac0346ebc352c7524204aa94336a2b5ad Reviewed-on: https://go-review.googlesource.com/c/go/+/548882 LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Michael Pratt <[email protected]>
Updates golang#60061 Change-Id: Ifa51f0bac0346ebc352c7524204aa94336a2b5ad Reviewed-on: https://go-review.googlesource.com/c/go/+/548882 LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Michael Pratt <[email protected]>
Change https://go.dev/cl/612038 mentions this issue: |
The go1.22 go/ast package now provides it. Updates golang/go#60061 Change-Id: I24e201660e3a8752dd505eefc894c7acae4c99f3 Reviewed-on: https://go-review.googlesource.com/c/tools/+/612038 Auto-Submit: Alan Donovan <[email protected]> Reviewed-by: Robert Findley <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]>
Almost every application that uses go/ast needs a function to strip parens. There are 4 copies of
unparen
in the standard library. golang.org/x/tools has astutil.Unparen. I propose we add this function togo/ast
.See https://go.dev/cl/495315 for the implementation.
The text was updated successfully, but these errors were encountered: