-
Notifications
You must be signed in to change notification settings - Fork 607
unix: switch (*CPUSet).Zero to clear builtin #262
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
Conversation
clear was added to Go 1.21 and is better than the manual loop approach. Signed-off-by: Aleksa Sarai <[email protected]>
|
This PR (HEAD: 71dc7f0) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/sys/+/698495. Important tips:
|
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/698495. |
|
Message from Aleksa Sarai: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/698495. |
|
Message from Keith Randall: Patch Set 1: Auto-Submit+1 Code-Review+2 Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/698495. |
|
Message from Go LUCI: Patch Set 1: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2025-08-23T02:04:06Z","revision":"49710f620cb88ee0436265cd88bb2550ec830bf1"} Please don’t reply on this GitHub thread. Visit golang.org/cl/698495. |
|
Message from Keith Randall: Patch Set 1: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/698495. |
|
Message from Keith Randall: Patch Set 1: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/698495. |
|
Message from Go LUCI: Patch Set 1: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/698495. |
|
Message from Go LUCI: Patch Set 1: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/698495. |
|
Message from Tobias Klauser: Patch Set 1: Code-Review+2 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/698495. |
clear was added to Go 1.21 and is better than the manual loop approach. Change-Id: I851203714446e21b6329e2bcf308f2571d339e36 GitHub-Last-Rev: 71dc7f0 GitHub-Pull-Request: #262 Reviewed-on: https://go-review.googlesource.com/c/sys/+/698495 Reviewed-by: Keith Randall <[email protected]> Reviewed-by: Cherry Mui <[email protected]> Reviewed-by: Keith Randall <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Tobias Klauser <[email protected]> Auto-Submit: Keith Randall <[email protected]>
|
This PR is being closed because golang.org/cl/698495 has been merged. |
clear was added to Go 1.21 and is better than the manual loop approach.