Pull Request №688 RosettaCommons/rosetta/main ← lyskov-ai/rosetta/refactor/core-signal-events-rule-of-zero
Merge: b8e64ecdc0338b77c231760f7f9bdcd4d500a240←532a3428f85793cb075127b7e01dea59ebbedbf8
Apply Rule of Zero to core signal event hierarchies
----------------
Merge commit message:
Apply Rule of Zero to core signal event hierarchies
The signal event classes in core/conformation/signals/ and core/pose/signals/
each had hand-written copy constructor, copy assignment, and empty destructor
that did nothing the compiler-generated versions wouldn't do. Remove the
boilerplate and let the compiler generate the copy operations; keep the
destructor declared (defaulted) where the class is a polymorphic base.
Touched: GeneralEvent, ConnectionEvent, IdentityEvent, XYZEvent (conformation);
GeneralEvent, DestructionEvent, ConformationEvent, EnergyEvent (pose).
LengthEvent is intentionally left alone -- its copy constructor and copy
assignment invoke check_consistency() in debug builds, so they have semantic
content beyond member-wise copy.