I agree with Sam that the "after" code is cleaner and that the majority of the diff is low risk (adding tests & build script).
I'd quibble with making destructors pure, and NVPI but it seems to run fine.
Cleaner, unit tested code... what's not to like?
« Back to merge proposal
I agree with Sam that the "after" code is cleaner and that the majority of the diff is low risk (adding tests & build script).
I'd quibble with making destructors pure, and NVPI but it seems to run fine.
Cleaner, unit tested code... what's not to like?