On 06/22/17 22:12, Peter Dimov via Boost wrote:
The header
stacktrace: boost/detail/winapi/capture_stack.hpp
should probably either be moved to Winapi, as the name suggests, or renamed to reside in boost/stacktrace/detail.
By the looks of it, it's something rather specific to Boost.Stacktrace, so it should probably be moved inside boost/stacktrace. While doing so, the file contents should also be updated to avoid using Boost.WinAPI namespace and macro prefixes. The include guard should be fixed as well.
And, as a general rule, libraries should strive to keep their headers under boost/<libname>, with the only exception being boost/<libname>.hpp. We have many legacy libraries not following this requirement, but any new ones and any new headers added to existing ones should.
The reason for this requirement is that otherwise it's impossible to perform dependency analysis unless all of Boost is checked out so that the tool can tell in what module boost/random-header-name.hpp resides.
More importantly, putting files in other libraries' directories is asking for conflicts.