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.

Sign up for GitHub

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

Jump to bottom

[20933] Set DataSharing in Writer|ReaderProxyData (backport #4761) #4801

Merged
merged 2 commits into from
May 20, 2024
Merged

[20933] Set DataSharing in Writer|ReaderProxyData (backport #4761) #4801

merged 2 commits into from
May 20, 2024

Commits on May 17, 2024

  1. Set DataSharing in Writer|ReaderProxyData (#4761) 

    * Refs #20933: Set datasharing on when pid is found

Signed-off-by: elianalf <[email protected]>

* Refs #20933: Set datasharing off after the clear

Signed-off-by: elianalf <[email protected]>

* Refs #20933: Add tests

Signed-off-by: elianalf <[email protected]>

* Refs #20933: Fix other_vendor_parameter_list_with_custom_pids test

Signed-off-by: elianalf <[email protected]>

* Refs #20933: Change implementation to fix failing tests

Signed-off-by: elianalf <[email protected]>

* Refs #20933: Apply suggestions

Signed-off-by: elianalf <[email protected]>

---------

Signed-off-by: elianalf <[email protected]>
(cherry picked from commit 904c523)

# Conflicts:
#	test/unittest/rtps/builtin/BuiltinDataSerializationTests.cpp
    @elianalf @EduPonz
    elianalf authored and EduPonz committed May 17, 2024
    Configuration menu
    Copy the full SHA
    90dbba8 View commit details
    Browse the repository at this point in the history

  2. Refs #20933: Fix conflicts 

    Signed-off-by: eduponz <[email protected]>
    @EduPonz
    EduPonz committed May 17, 2024
    Configuration menu
    Copy the full SHA
    dd190fe View commit details
    Browse the repository at this point in the history