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