This might cause a cascade of fixes - if so, it belongs to another cleanup MP...
121 + virtual ~SessionTarget() {}
Shouldn't that be virtual "~SessionTarget() = default" (there is a semantic difference - the latter is noexcept).
« Back to merge proposal
This might cause a cascade of fixes - if so, it belongs to another cleanup MP...
121 + virtual ~SessionTarget() {}
Shouldn't that be virtual "~SessionTarget() = default" (there is a semantic difference - the latter is noexcept).