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

The total_count_change value should be the incremental changes in tot… #969

Merged
merged 1 commit into from Nov 6, 2018

Conversation

jwillemsen
Copy link
Member

…al_count since the last time the listener was called or the status was read. Tested and fixed this for on_offered_deadline_missed and on_requested_deadline_missed, see issue #619

* dds/DCPS/OfferedDeadlineWatchdog.cpp:
* dds/DCPS/RequestedDeadlineWatchdog.cpp:
* tests/DCPS/Deadline/DataReaderListenerImpl.cpp:
* tests/DCPS/Deadline/DataReaderListenerImpl.h:
* tests/DCPS/Deadline/DataWriterListenerImpl.cpp:
* tests/DCPS/Deadline/DataWriterListenerImpl.h:
* tests/DCPS/Deadline/publisher.cpp:
* tests/DCPS/Deadline/subscriber.cpp:

…al_count since the last time the listener was called or the status was read. Tested and fixed this for on_offered_deadline_missed and on_requested_deadline_missed, see issue OpenDDS#619

    * dds/DCPS/OfferedDeadlineWatchdog.cpp:
    * dds/DCPS/RequestedDeadlineWatchdog.cpp:
    * tests/DCPS/Deadline/DataReaderListenerImpl.cpp:
    * tests/DCPS/Deadline/DataReaderListenerImpl.h:
    * tests/DCPS/Deadline/DataWriterListenerImpl.cpp:
    * tests/DCPS/Deadline/DataWriterListenerImpl.h:
    * tests/DCPS/Deadline/publisher.cpp:
    * tests/DCPS/Deadline/subscriber.cpp:
@mitza-oci mitza-oci merged commit 4c73012 into OpenDDS:master Nov 6, 2018
mitza-oci added a commit that referenced this pull request Jan 17, 2019
The total_count_change value should be the incremental changes in tot…

(cherry picked from commit 4c73012)
@jwillemsen jwillemsen deleted the jwi-totalcountchangefixes-619 branch April 6, 2019 18:09
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.

None yet

2 participants