Details
-
Type:
Story
-
Status: To Do
-
Resolution: Unresolved
-
Fix Version/s: None
-
Component/s: afw
-
Labels:None
-
Story Points:2
-
Team:Data Release Production
-
Urgent?:No
Description
afw::math::Function is conceptually pure virtual but has no pure-virtual methods, making it possible to instantiate with possibly confusing results.
My recommended fix is to move the clone() method defined in its immediate subclasses down to it. This would require switching to a nonvirtual public + protected virtual pattern to maintain covariant return types, which would require a bit more work but have other benefits.
Attachments
Issue Links
- mentioned in
-
Page Loading...
Reviewed for DM-CCB 2020-06-24. Still valid.