Ping, this doesn't seem to have been fixed yet?
Bert Huijben wrote on Mon, Oct 11, 2010 at 10:58:38 +0200: > > -----Original Message----- > > From: artag...@apache.org [mailto:artag...@apache.org] > > Sent: maandag 4 oktober 2010 17:27 > > To: comm...@subversion.apache.org > > Subject: svn commit: r1004286 - in /subversion/trunk: ./ > > subversion/libsvn_subr/io.c > > > > Author: artagnon > > Date: Mon Oct 4 15:26:44 2010 > > New Revision: 1004286 > > > > URL: http://svn.apache.org/viewvc?rev=1004286&view=rev > > Log: > > Merge r985477 from subversion/branches/performance > > > > * subversion/libsvn_subr/io.c > > (get_default_file_perms): Store the permissions of the created > > temporary file in a static variable and re-use it in subsequent > > calls instead of checking persmissions everytime. This has > > performance benefits. > > > > Review by: artagnon > > Approved by: julianfoad > > Delayed review: > > Shouldn't this function use some 'atomic initialization' handling? > > Currently it uses a static apr_fileperms_t (integer?) which can be > manipulated by multiple threads at the same time. > > This part of subversion is a library and inside tools like Subclipse, > TortoiseSVN, AnkhSVN and others it is used multithreaded. > > Bert > >
