You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by iv...@apache.org on 2015/09/07 09:18:25 UTC
svn commit: r1701565 - /subversion/trunk/subversion/libsvn_subr/stream.c
Author: ivan
Date: Mon Sep 7 07:18:24 2015
New Revision: 1701565
URL: http://svn.apache.org/r1701565
Log:
* subversion/libsvn_subr/stream.c
(svn_stream__create_for_install): Update comment with rationale why we
use buffered mode for temporary file.
Modified:
subversion/trunk/subversion/libsvn_subr/stream.c
Modified: subversion/trunk/subversion/libsvn_subr/stream.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_subr/stream.c?rev=1701565&r1=1701564&r2=1701565&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_subr/stream.c (original)
+++ subversion/trunk/subversion/libsvn_subr/stream.c Mon Sep 7 07:18:24 2015
@@ -2217,8 +2217,8 @@ svn_stream__create_for_install(svn_strea
/* Wrap as a standard APR file to allow sharing implementation.
But do note that some file functions (such as retrieving the name)
- don't work on this wrapper. */
- /* ### Buffered, or not? */
+ don't work on this wrapper.
+ Use buffered mode to match svn_io_open_unique_file3() behavior. */
status = apr_os_file_put(&file, &hInstall,
APR_WRITE | APR_BINARY | APR_BUFFERED,
result_pool);