Skip to content
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

Support moving of stream_to. #707

Merged
merged 3 commits into from
Jul 6, 2023
Merged

Support moving of stream_to. #707

merged 3 commits into from
Jul 6, 2023

Conversation

jtv
Copy link
Owner

@jtv jtv commented Jun 19, 2023

Fixes #706.

Finally biting the bullet and supporting moves of transaction_focus, which requires dealing with that backpointer from the transaction back to the focus object.

@@ -526,6 +526,22 @@
}


void test_stream_to_moves_into_optional()

Check notice

Code scanning / CodeQL

Unused static function

Static function test_stream_to_moves_into_optional is unreachable ([tst_test_stream_to_moves_into_optional](1) must be removed at the same time)
@jtv jtv changed the title Add move constructor to stream_to. Support moving of stream_to. Jun 19, 2023
jtv added 2 commits June 22, 2023 21:21
Should help with #706, though we may need move assignment as well.

Finally biting the bullet and adding move construction to
`transaction_focus`, which requires dealing with that backpointer from
the transaction back to the focus object.

I'll try move assignment next.  The obvious complication there is that
`m_trans` will have to become a pointer.
@jtv jtv merged commit 68c93b6 into master Jul 6, 2023
@jtv jtv deleted the move-stream_to branch July 6, 2023 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

deprecation of the stream_to constructor
1 participant