Re: [boost] Boost.Log: Any Comments? (Was: [log] Permission to merge to trunk)

On Thursday 04 April 2013 14:20:50 Rob Stewart wrote:

On Apr 1, 2013, at 1:23 PM, Andrey Semashev wrote:

...

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.