Add hpx::iostreams::ostream overload to format_to #3174
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #3013.
Proposed Changes
hpx::util::format_to
taking ahpx::iostreams::ostream
to make sure that the correct operators (<<
) are called.Any background context you want to provide?
When a
hpx::iostreams::ostream
is used as astd::ostream
the correct virtual functions are not called, thus bypassing locks that are supposed to be taken in thehpx::iostreams::ostream
implementation (base class functions are not virtual?). A proper fix would make sure the correct functions are called, but at the moment I have no fix for that.