Boost logo

Boost :

Subject: Re: [boost] [1.47.0] Release branch now closed
From: Rene Rivera (grafikrobot_at_[hidden])
Date: 2011-06-18 09:56:25


On 6/18/2011 4:50 AM, Daniel James wrote:
> Can I merge this change to iostreams?
>
> https://svn.boost.org/trac/boost/changeset/72543
>
> It cleans up an exception safety issue. In the case of 'close'
> throwing an exception, 'reset' needs to be called in order to delete
> the allocated resources.
>
> The test results are looking okay. There are a few failures but these
> seem to be caused either by dependencies or because bzip is missing or
> hasn't linked correctly, rather than a problem with this change.

Looks safe enough.. And reset looks idempotent, so should be OK to
execute even if the close fails. So, yes go ahead.

-- 
-- Grafik - Don't Assume Anything
-- Redshift Software, Inc. - http://redshift-software.com
-- rrivera/acm.org (msn) - grafik/redshift-software.com
-- 102708583/icq - grafikrobot/aim,yahoo,skype,efnet,gmail

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