Pull Request №700 RosettaCommons/rosetta/main ← lyskov-ai/rosetta/refactor/core-private-unimpl-copy-modernization
Merge: dc594471462f126ea60fc1148900ed66e5b77e44←ff37fbbb56e33fdf9b332f6757faaef2bb50e2c6
Modernize pre-C++11 copy prevention to = delete across core/ and protocols/
----------------
Merge commit message:
Modernize pre-C++11 copy prevention to = delete across core/ and protocols/
Replace the pre-C++11 idiom (private undefined copy ctor / copy assignment to
prevent copies via link error) with explicit `= delete` for clearer diagnostics
and compile-time enforcement. 16 files; no behaviour change.
Two flavours:
1. Stand-alone non-copyable classes: replace the unimplemented private
declarations with `= delete`. Drops redundant comments ("deny use of the
copy constructor", "private and unimplemented", etc.) that are now
self-evident from `= delete`. Keeps explanatory comments where the *why*
is non-obvious (ScoreFunction / DockingScoreFunction / OtherContextScoreFunction:
"copying discards subclass information; use clone() instead").
2. Singleton (SymDofMoverSampler, inheriting utility::SingletonBase): drop
the redundant copy/assignment declarations entirely — the base already
`= delete`s its own copies, which transitively makes the derived
implicit copies deleted.
Files: ScoreFunction, MinScoreScoreFunction, DockingScoreFunction,
OtherContextScoreFunction, RotamerSet, RotamerSet_, RotamerSubset, RotamerSetBase,
DofPassport, ConstantLengthFragSetIterator_, FrameListIterator_,
MinimalFragSetIterator_, OrderedFragSetIterator_, FragmentPicker (QuotaDebug),
SymDofMoverSampler, FileSystemResourceLocator (FileStream).
Debug build passes clean.