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
Fixing #2931 #3059
Fixing #2931 #3059
Conversation
@kleinhenz could you verify if this fixes the issue for you? |
Yes fixed for me. Thanks! |
src/hpx_init.cpp
Outdated
// (if the program options contain --hpx:help or --hpx:version) | ||
if (rp.terminate_after_parse()) | ||
{ | ||
return result; |
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.
Why have you remove conditionally setting of the result value to zero?
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.
Because it's always zero at this point in time as far as I can see.
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.
You're right. The code as it is right now leaves result
unchanged after it was initialized. This is a leftover problem from our move to the resource_partitioner
, though. I'll try to come up with a patch against this PR to fix things.
Thanks Hartmut, looks good. |
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.
LGTM, thanks!
runtime termination in the presence of --hpx:help or --hpx:version needs to
happen after the runtime has been initialized.