Fixes #1419

[nidhimj22]

No description provided.

[hkaiser]

This array of state names needs adaptation as well (see the function get_runtime_state_name() below).

[nidhimj22]

Yes, prepending them with state_ would be better because these states are also the names of some functions (e.g shutdown, stopped). That is why some names have state_ prepended.

[hkaiser]

While this PR nicely merges the two state enum types, it fixes only the use of one of them (runtime::state). The use of the former hpx::state enum values has not been adapted. There are many places where the code today relies on bitmask operations for these values (for instance: https://github.com/STEllAR-GROUP/hpx/blob/master/src/runtime/parcelset/parcelhandler.cpp#L82). All these existing pieces need to be thoroughly analyzed to make sure we don't change existing behavior.

[hkaiser]

@nidhimj22: Are you still planning to work on this?

[nidhimj22]

@hkaiser Yes, I a am planning to fix this. I have been a little busy with school exams and pending projects,I will fix it asap.

[hkaiser]

@nidhimj22 It's been a while, do you still plan to work on this? We've had complaints from users that this needs to be fixed.

[hkaiser]

This is superseded by #1512