Bug #2042

semantics of rsc::runtime::Properties::set() not conform to documentation

Added by R. Haschke over 9 years ago. Updated about 6 years ago.

Status:ResolvedStart date:10/07/2014
Priority:NormalDue date:
Assignee:J. Moringen% Done:

100%

Category:Util
Target version:Robotics Service Bus - rsb-0.18

Description

While the doc says:

Sets a new property in the map. If a property with this name exits, the
new one will not be inserted. The old property has to be removed first.

However, the source code always does:

erase(key)

Please decide on desired semantics and change code or doc correspondingly.

Associated revisions

Revision e9b262e8
Added by J. Moringen about 6 years ago

Fixed confused semantics of Properties::set in src/rsc/runtime/Properties.h

fixes #2042

  • src/rsc/runtime/Properties.h (Properties::set): always replace
    existing entries; return Properties object

History

#1 Updated by J. Moringen about 6 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

#2 Updated by J. Moringen about 6 years ago

  • Category set to Util
  • Assignee set to J. Moringen
  • Target version set to rsb-0.18

Also available in: Atom PDF