Enhancement #1621

Deprecate and later remove Registry

Added by J. Wienke almost 7 years ago. Updated over 2 years ago.

Status:ResolvedStart date:09/10/2013
Priority:NormalDue date:
Assignee:J. Moringen% Done:

100%

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

Description

I would propose to now deprecate rsc::misc::Registry and later remove it completely. The rationale is that it basically performs the job of the factory classes available in the project apart from the pre-main registration, which was always hard to handle and the plugin system might be a better candidate now. Moreover we don't use it in production anymore. This would probably be a solution to #234.

Any opinions on this?


Related issues

Related to Robotics Systems Commons - Enhancement #234: Check dualism of Registry and Factory Rejected 04/08/2011
Blocked by Robotics Systems Commons - Tasks #1629: Let logging system use factory instead of registry Resolved 09/13/2013

Associated revisions

Revision cb988e25
Added by J. Moringen over 6 years ago

Deprecated Registry class in src/rsc/misc/Registry.h

refs #1621

  • src/rsc/misc/Registry.h (Registry): marked as deprecated via
    DEPRECATED_CLASS macro, DEPRECATED_MSG macro on member functions and
    doxygen annotation

Revision b8dc9f81
Added by J. Moringen over 2 years ago

Removed Registry in src/rsc/misc/Registry.h

fixes #1621

  • src/rsc/misc/Registry.h: removed; has been deprecated for a long
    time
  • test/rsc/misc/RegistreeLib.{h,cpp}: removed; no longer needed
  • test/rsc/misc/testRegistry.{h,cpp}: likewise
  • test/rsc/misc/RegistryTest.cpp: likewise
  • test/CMakeLists.txt (REGISTRY_TEST_LIB_NAME): removed; no longer
    needed
    (REGISTREE_TEST_LIB_NAME): removed; no longer
    (target_link_libraries RSC_TEST_NAME): removed registry libraries

History

#1 Updated by J. Wienke almost 7 years ago

ping

#2 Updated by J. Moringen almost 7 years ago

I never used this so I don't have any strong opinions, but I'm not against removing it.

#3 Updated by J. Wienke almost 7 years ago

  • Status changed from New to In Progress

#4 Updated by J. Wienke almost 7 years ago

Hm, not as easy as I thought. The logging system relies on registry.

#5 Updated by J. Moringen over 6 years ago

  • Target version changed from rsb-0.10 to rsb-0.11

#6 Updated by J. Moringen over 6 years ago

  • Target version changed from rsb-0.11 to rsb-0.12

#7 Updated by J. Moringen over 6 years ago

  • Target version changed from rsb-0.12 to rsb-1.0

#8 Updated by J. Moringen over 6 years ago

  • % Done changed from 0 to 50

Deprecated Registry. Removal should happen in 1.0 release.

#9 Updated by J. Wienke about 4 years ago

  • Assignee deleted (J. Wienke)

#10 Updated by J. Moringen over 2 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 50 to 100

#11 Updated by J. Moringen over 2 years ago

  • Assignee set to J. Moringen
  • Target version changed from rsb-1.0 to rsb-0.18

Also available in: Atom PDF