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?
> 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