[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1335566 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Tue, 8 May 2012 15:10:11 -0400

On 05/08/2012 02:20 PM, Ivan Zhakov wrote:
>> +/* Implements svn_close_fn_t */
>> +static svn_error_t *
>> +close_handler_lazyopen(void *baton)
>> +{
>> + lazyopen_baton_t *b = baton;
>> +
>> + SVN_ERR(lazyopen_if_unopened(b));
>> + SVN_ERR(svn_stream_close(b->real_stream));
>
> I think we should just no-op in case if wrapped stream is not opened. I meant:
> [[[
> if (opened)
> {
> SVN_ERR(svn_stream_close(b->real_stream));
> }
> return SVN_NO_ERROR;
> ]]]

I'm 50/50 on this change of behavior, but you clearly have an opinion, so
I've changed (and documented) it. See r1335704.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development

Received on 2012-05-08 21:10:48 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.