Skip to content

Commit

Permalink
Fix partition copy in QoS (#1947)
Browse files Browse the repository at this point in the history
Signed-off-by: Jay Sridharan <[email protected]>
Co-authored-by: Jay Sridharan <[email protected]>
(cherry picked from commit 7bcbfa3)
  • Loading branch information
jay-sridharan authored and mergify[bot] committed Jun 21, 2023
1 parent f09ff93 commit 42fcd0f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/cpp/fastdds/publisher/qos/WriterQos.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ void WriterQos::setQos(
m_presentation = qos.m_presentation;
m_presentation.hasChanged = true;
}
if (first_time || qos.m_partition.names().size() > 0)
if (first_time || qos.m_partition.names() != m_partition.names())
{
m_partition = qos.m_partition;
m_partition.hasChanged = true;
Expand Down
2 changes: 2 additions & 0 deletions src/cpp/rtps/builtin/discovery/participant/PDP.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1046,13 +1046,15 @@ bool PDP::remove_remote_participant(
// Return reader proxy objects to pool
for (auto pit : *pdata->m_readers)
{
pit.second->clear();
reader_proxies_pool_.push_back(pit.second);
}
pdata->m_readers->clear();

// Return writer proxy objects to pool
for (auto pit : *pdata->m_writers)
{
pit.second->clear();
writer_proxies_pool_.push_back(pit.second);
}
pdata->m_writers->clear();
Expand Down

0 comments on commit 42fcd0f

Please sign in to comment.