Subject: Re: [boost] Boost.Log: Any Comments? (Was: [log] Permission to merge to trunk)
From: Andrey Semashev (andrey.semashev_at_[hidden])
Date: 2013-04-05 06:14:48
On Thursday 04 April 2013 14:20:50 Rob Stewart wrote:
> On Apr 1, 2013, at 1:23 PM, Andrey Semashev <andrey.semashev_at_[hidden]>
> > Also, I want to point out that simply turning warnings off globally in
> > Boost.Log kind of defeats the purpose for turning these warnings on in
> > the first place.
> Using push and pop means you're not disabling the warnings globally.
Yes, "globally" means for the whole Boost.Log code in this case.
> >> Since there are shops that require use of higher warnings levels, or even
> >> warnings as errors, (despite what you might think about their decisions)
> >> you may make your reportedly excellent and very widely applicable log
> >> library unusable for them. This would be a pity?>
> > Sure it would. However, this would be the result of counterproductive
> > management decisions and not of particular flaws of Boost.Log, IMHO. One
> > should fix the root of the problem.
> That totally misses the point. Management decisions are often very difficult
> to alter. Those with such requirements would be unable to use your library,
> limiting its appeal, if you give up too easily due to that attitude.
I realize that. Yet I want to encourage people to make reasonable decisions
rather than blindly follow inadequate restrictions.
Anyways, let's end it here by me saying that I'll try to silence bogus
warnings as they get reported.