`Array2D`, `CompensatedSummation`, `Versor`: remove hand-written copy-constructor and assignment

InsightSoftwareConsortium / ITK

Insight Toolkit (ITK) -- Official Repository. ITK builds on a proven, spatially-oriented architecture for processing, segmentation, and registration of scientific images in two, three, or more dimensions.

https://itk.org

Apache License 2.0

1.37k stars 660 forks source link

`Array2D`, `CompensatedSummation`, `Versor`: remove hand-written copy-constructor and assignment #4740

Closed N-Dekker closed 1 week ago

N-Dekker commented 1 week ago

Replaced the hand-written copy-constructor and copy-assignment of Array2D and CompensatedSummation with = default.
Simply removed them from Versor, following the "Rule of Zero"

The hand-written copy-constructors were found with regular expression ^(\w+)<.+>::\1\(const Self

N-Dekker commented 1 week ago

For the record, applying the "Rule of Zero" to itk::Versor has the interesting "side effect" that it makes an instance like Versor<double> a trivial class. This may in some cases yield a performance improvement.