Enhancement #1621
Deprecate and later remove Registry
Status: | Resolved | Start date: | 09/10/2013 | |
---|---|---|---|---|
Priority: | Normal | Due 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
Associated revisions
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
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 over 10 years ago
ping
#2 Updated by J. Moringen over 10 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 over 10 years ago
- Status changed from New to In Progress
#4 Updated by J. Wienke over 10 years ago
Hm, not as easy as I thought. The logging system relies on registry.
#5 Updated by J. Moringen over 10 years ago
- Target version changed from rsb-0.10 to rsb-0.11
#6 Updated by J. Moringen about 10 years ago
- Target version changed from rsb-0.11 to rsb-0.12
#7 Updated by J. Moringen about 10 years ago
- Target version changed from rsb-0.12 to rsb-1.0
#8 Updated by J. Moringen about 10 years ago
- % Done changed from 0 to 50
Deprecated Registry
. Removal should happen in 1.0 release.
#9 Updated by J. Wienke almost 8 years ago
- Assignee deleted (
J. Wienke)
#10 Updated by J. Moringen about 6 years ago
- Status changed from In Progress to Resolved
- % Done changed from 50 to 100
Applied in changeset rsc|b8dc9f81ef88fe01a2454031b7fbd5a6a41e0048.
#11 Updated by J. Moringen about 6 years ago
- Assignee set to J. Moringen
- Target version changed from rsb-1.0 to rsb-0.18