-
Notifications
You must be signed in to change notification settings - Fork 765
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
[15118] Adding DataReader::get_unread_count (bool mark_as_read) #2825
Conversation
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
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.
The changes look good to me, although some tests would be appreciated
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
@richiprosima Please test this |
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>
Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>
@richiprosima please tests this |
@richiprosima please test this |
Test failures are definitely unrelated as this PR adds a new API that is not called from anywhere in the library but the specific test, which passes in all platforms. |
@Mergifyio backport 2.6.x |
* Refs #15118. Adding overload on RTPSReader. Signed-off-by: Miguel Company <MiguelCompany@eprosima.com> * Refs #15118. Adding method on DataReaderHistory. Signed-off-by: Miguel Company <MiguelCompany@eprosima.com> * Refs #15118. Changed method on DataReaderImpl. Signed-off-by: Miguel Company <MiguelCompany@eprosima.com> * Refs #15118. Adding overload on DataReader. Signed-off-by: Miguel Company <MiguelCompany@eprosima.com> * Refs #15118. Fixed returned value on RTPSReader. Signed-off-by: Miguel Company <MiguelCompany@eprosima.com> * Refs #15118. Added unit test. Signed-off-by: Miguel Company <MiguelCompany@eprosima.com> * Refs #14940. Fix error Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev> * Refs #14940. Improve test Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev> Co-authored-by: Ricardo González Moreno <ricardo@richiware.dev> (cherry picked from commit 9586068)
✅ Backports have been created
|
Description
This PR adds an overload of
DataReader::get_unread_count
that receives a boolean indicating whether the unread changes should be marked as read.This allows the application to implement a pattern where the number of newly available samples is obtained inside
on_data_available()
.Contributor Checklist
versions.md
file (if applicable).Reviewer Checklist