Boost logo

Boost :

Subject: Re: [boost] Official warnings policy?
From: Beman Dawes (bdawes_at_[hidden])
Date: 2009-11-05 07:05:46


On Wed, Nov 4, 2009 at 4:06 PM, Patrick Horgan <phorgan1_at_[hidden]> wrote:

>...
>   An example of one of my least favorite avoidable warnings is when one part
>   of boost moves an include file and deprecates the old location, and release
>   after release, I have to look at warnings from other parts of boost.  I
>   can't even say what I think about that.  I don't think a release should go
>   out with issues like that.  Boost should at least be internally consistent,
>   and not have warnings about misusing other parts of boost.  You'd think that
>   would be the bare minimum to avoid looking like amateurs.  How hard would it
>   have been to fix for example, in 1.40 when I see hundreds of warnings from
>   the graph stuff because:
>   boost/property_map.hpp:15:4: warning: #warning "This header is deprecated.
>   Please use: boost/property_map/property_map.hpp"
>   I'm guessing the graph guys are used to ignoring warnings.  But the code was
>   released like this.  That's embarrassing.  There's probably 35 places they
>   would have had to fix this so that hundreds of errors wouldn't spew out
>   whenever someone used their code.  They aren't even willing to grab the
>   low-hanging fruit.
>   There have long been other places where boost code used deprecated headers
>   from the standard libraries.  That's unprofessional.
>   Sorry to rant.  This is a long standing issue for me and I thought I better
>   write this before I saw the inevitable responses from people trying to
>   justify their bad coding.  I really would have ranted then!

The result of this long discussion may well be a Boost policy on
warnings. But it the meantime, you might want to open tickets against
against libraries that are using long deprecated headers.

--Beman


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk