On December 27, 2014 12:47:40 PM EST, Peter Dimov
Andrzej Krzemienski wrote:
2014-12-26 12:05 GMT+01:00 Antony Polukhin
: Are there any reasons not to change the existing implementation of Boost.Swap to Ion's version?
You may find it a silly argument, and it goes back to these dependency graph discussions. If you look at the dependency report, like this one http://www.pdimov.com/tmp/report-d5ca5be/module-levels.html It says Boost.Core is level 2 (so, fairly primitive) while Boost.Move is Level 6.
It's in fact not a silly argument at all. Boost.Core is explicitly not allowed to acquire new dependencies - this was the reason for creating it. We can in principle extract Boost.Swap into its own module though.
I'm sure Antony meant to change the implementation to match the ideas in Ion's code, not use the test implementation he showed. ___ Rob (Sent from my portable computation engine)