Re: [boost] [stacktrace] review (changing vote to NO)

Subject: Re: [boost] [stacktrace] review (changing vote to NO)
From: Andrey Semashev (andrey.semashev_at_[hidden])
Date: 2016-12-25 09:57:38

On 12/25/16 17:37, Peter Dimov wrote:
> Antony Polukhin wrote:
>> > Additionally I realized that with_trace and traced aren't parts of
>> the > library! These should be built in there by default.
>>
>> During the Review I've received multiple requests to embed stacktraces
>> into exceptions. And everyone was requesting to do that in a different
>> way. I've got about 5 different incompatible ways to do that, and
>> everyone thinks that there's only one (or at most 3) right ways to do
>> that.
>
> There is one and only one way to do that, and it's to use
> Boost.Exception. That's what it's there for.
>
> Every library defining its own with_foo and with_bar exception wrappers
> is not the way to go. And neither is duplicating the standard exception
> hierarchy as in
>
> https://sourceforge.net/p/cppcms/code/HEAD/tree/framework/trunk/booster/booster/backtrace.h

+1