Bug #2043
semantic of ParticipantConfig::setTransports not conform to doc/name
| Status: | Resolved | Start date: | 10/07/2014 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | J. Wienke | % Done: | 100% | |
| Category: | C++ | |||
| Target version: | rsb-0.13 |
Description
current implementation of ParticipantConfig::setTransports actually only adds new transports,
but neither removes existing ones nor reconfigures existing ones.
From function name and doc, I would expect that setTransports does
this->transports.swap (arg)i.e. replaces the existing set of transports with the new one.
Associated revisions
ParticipantConfig::setTransports: replace everything
Instead of adding only non-existing transports, make the semantics
clearer and completely replace all transport configurations in
ParticipantConfig::setTransports.
fixes #2043
History
#1 Updated by J. Wienke almost 9 years ago
- Category set to C++
- Status changed from New to In Progress
- Assignee set to J. Wienke
- Target version set to rsb-0.13
#2 Updated by J. Wienke almost 9 years ago
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Applied in changeset rsb-cpp|27feeda9bd5893f4b2014502da85c1309aacbe9d.