Subject: Re: [boost] Fw: [stacktrace] [winapi] header out of place
From: Andrey Semashev (andrey.semashev_at_[hidden])
Date: 2017-06-22 21:30:19
On 06/22/17 22:12, Peter Dimov via Boost wrote:
> The header
> 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
> 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.
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk