Fixed the managed to unmanaged change

[nischitkumar]

Fixes #6953

Proposed Changes

Implemented the one word fix (managed->unmanaged). This was necessary because base_component::get_unmanaged_id returns an hpx::id_type tagged with management_type::managed instead of management_type::unmanaged. When the returned ID is serialized to a remote locality, this triggers an assertion failure in credit_handling.cpp in debug builds.

[StellarBot]

Can one of the admins verify this patch?

[hkaiser]

Could you please add a test that reproduces the issue and demonstrates that the issue is fixed by your PR?

[hkaiser]

@nischitkumar Would you have the time to finish this PR?

[hkaiser]

@nischitkumar Would you have the time to finish this PR?

Ping? Do you have any plans to wrap this up? Do you need help?

[nischitkumar]

Hey @hkaiser, apologies for the delay in getting back, I'd been tied up with my university exams. I shall resume working on the PR and shall find relevant test cases. Shall definitely ping you if I run into any issues.

[hkaiser]

@nischitkumar Do you still plan to work on this PR? I believe your proposed change is valid, so all we need to prove that it is would be a test that exercises the modified code path, thus showing that a problem that was there before is now gone.

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

_{TIP This summary will be updated as you push new changes. Give us feedback}