Changed handler classes: std-handler now inherits from base class and uses default functions
Description
Description
Details
Details
- Auditors
floca - Provenance
kalali Authored on Apr 11 2018, 5:21 PM kalali Pushed on Apr 11 2018, 5:22 PM - Parents
- rMITKeac8b99e0769: Added comments and refactored according to review in T24216
- Branches
- Unknown
- Tags
Event Timeline
/Modules/Core/include/mitkDisplayActionEventDefaultFunctions.h | ||
---|---|---|
24 | comments are wrong; will be changed with the next commit |
Comment Actions
think its good.
/Modules/Core/include/mitkDisplayActionEventDefaultFunctions.h | ||
---|---|---|
22 | If you use a sub namespace I would name it differently. So that it also fits other variantes, that are not default but implemented by others. e.g. DisplayActionEventFunctions or so. | |
29–47 | Minor issue/nice to have: Additionally I think the default should some thing else in the future and more locally than the old implementations. Maybe? e.g. |