-
Notifications
You must be signed in to change notification settings - Fork 3k
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
erts: Use processes_iterator/0, processes_next/1
#9182
erts: Use processes_iterator/0, processes_next/1
#9182
Conversation
CT Test Results 3 files 141 suites 49m 16s ⏱️ Results for commit 5c30071. ♻️ This comment has been updated with latest results. To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass. See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally. Artifacts// Erlang/OTP Github Action Bot |
805f2c4
to
7b0c625
Compare
de3f1c9
to
beaa2e6
Compare
beaa2e6
to
b3e93af
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please squash the commits.
b3e93af
to
c0bd4d9
Compare
b6ff680
to
eb9cde6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me; only one nitpick.
erts/preloaded/src/init.erl
Outdated
Iter = erlang:processes_iterator(), | ||
case kill_pids(Heart, Logger, Iter, false) of | ||
true -> | ||
% Continue until all are really killed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
% Continue until all are really killed. | |
%% Continue until all are really killed. |
eb9cde6
to
2ec0d63
Compare
2ec0d63
to
5c30071
Compare
No description provided.