-
Notifications
You must be signed in to change notification settings - Fork 412
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
Allow publishing borrowed messages with intra-process enabled #2108
Allow publishing borrowed messages with intra-process enabled #2108
Conversation
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
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.
@MiguelCompany the fix looks good to me, but i think we need to address rpr job failures cz of this change.
@MiguelCompany after Rpr failures are addressed, i will start the CI. |
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
@fujitatomoya I updated the tests accordingly to the changes. |
@fujitatomoya Do you think this can be merged? I also think it can be backported into Humble |
@Mergifyio backport humble |
✅ Backports have been created
|
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com> (cherry picked from commit 3062dec)
This PR avoids a runtime_error exception when trying to publish a
LoanedMessage
on a publisher that has intra-process enabled.The changes make
can_loan_messages()
return false in that case, which in turn makesborrow_loaned_message()
allocate a standard message.