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
runtime: cleanup timer codes #54287
runtime: cleanup timer codes #54287
Conversation
The existing implementation has a poor structure and is a mess. I just moved exist codes to seprated files and structures to improve code and logic readability. No logic has been changed in this cleanup. Updates #53953
This PR (HEAD: c324f92) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/421615 to see it. Tip: You can toggle comments from me using the |
Message from Gopher Robot: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/421615. |
Message from Michael Knyszek: Patch Set 1: (5 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/421615. |
Message from Omid Hekayati: Patch Set 1: (5 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/421615. |
- rename timer.add() to timer.start() - rename timer.delete() to timer.stop() - change atomic fields to atomic.[T]
This PR (HEAD: 127edfd) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/421615 to see it. Tip: You can toggle comments from me using the |
Message from Omid Hekayati: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/421615. |
I think it is the time we must close this issue and related PR as It will never ever have chance to merge. I suggest main |
The existing implementation has a poor structure and is a mess.
I just moved exist codes to seprated files and structures
to improve code and logic readability.
No logic has been changed in this cleanup.
Updates #53953