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

Re: CVS update: subversion/subversion/include svn_io.h

From: Greg Stein <gstein_at_lyra.org>
Date: 2000-12-29 11:12:18 CET

On Sun, Dec 24, 2000 at 11:26:16AM -0000, ghudson@tigris.org wrote:
> User: ghudson
> Date: 00/12/24 03:26:16
>
> Modified: subversion/include svn_io.h
> Log:
> (svn_stream_from_aprfile, svn_stream_from_stdio): Doc change. The
> convenience of allowing file/fp to be NULL is still pretty convenient
> even though it's easy to create an empty readable stream. So we'll
> leave it around.

I'd disagree. Passing NULL might be a bug. Calling the empty() function
explicitly will make it clearer that you intended to get an empty stream.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
Received on Sat Oct 21 14:36:18 2006

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.