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);