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

Re: svn commit: r1701564 - /subversion/trunk/subversion/libsvn_subr/stream.c

From: Branko Čibej <brane_at_wandisco.com>
Date: Mon, 7 Sep 2015 09:30:07 +0200

On 07.09.2015 09:15, ivan_at_apache.org wrote:
> Author: ivan
> Date: Mon Sep 7 07:15:25 2015
> New Revision: 1701564
>
> URL: http://svn.apache.org/r1701564
> Log:
> Reduce difference between Windows and non-Windows implementation of
> install_stream.
>
> * subversion/libsvn_subr/stream.c
> (install_close): Remove #ifdef WIN32
> (svn_stream__create_for_install): Always setup flush on close stream
> handler.
> (svn_stream__install_stream): Close temporary file before rename on all
> platforms.
> (svn_stream__install_get_info): Obtain file info from file handle on all
> platforms.
> (svn_stream__install_delete): Close temporary file before delete on all
> platforms.

Why do you think this is a good idea? I know you can't move/delete an
open file on Windows without jumping through hoops that APR doesn't
provide, but you certainly can do that on Unix and if memory serves,
that actually helps performance in some cases where remote file systems
are involved.

-- Brane
Received on 2015-09-07 09:30:24 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.