Jurko Gospodnetić:
Also... the renamed class name as used in the original patch seemed clearer since Visual C++ still has a bug with exposing this identified in the global namespace... and 'critical_section' seems much more likely to cause problems than critical_section__whatever_we_write_here_as_an_explanation.
The bug appears to import all boost::detail identifiers into the global namespace, not just critical_section. It's not reasonable to change them all. A better way to deal with the problem would be to just remove the offending "using namespace boost::detail;" line from boost/serialization/detail/shared_count_132.hpp. It's even guarded for Borland. Note also the "using namespace boost;" in the same header, it imports all boost identifiers as well, and it's definitely not reasonable to change these to contain an "explanation" suffix.
No interest? Robert? Jurko?