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

[PATCH] ensure open mode of svn_io_open_unique_file2() is clearly defined

From: Charles Acknin <charlesacknin_at_gmail.com>
Date: 2007-07-27 23:47:56 CEST

I bumped into svn_io_open_unique_file2()'s docstring being not clear
on what mode it uses to open the file.

Below are the flags passed to apr_file_open() in the implementation of
the function:
(APR_READ | APR_WRITE | APR_CREATE | APR_EXCL | APR_BUFFERED)

[[[
* subversion/include/svn_io.h
  (svn_io_open_unique_file2): ensure the function's docstring is clear
about how it opens the file -- read/write and not just read.

Patch by: cacknin
Suggested by: glasser
]]]

[[[
Index: subversion/include/svn_io.h
===================================================================
--- subversion/include/svn_io.h (revision 25772)
+++ subversion/include/svn_io.h (working copy)
@@ -112,8 +112,8 @@
                                         apr_pool_t *pool);

-/** Open a new file (for writing) with a unique name based on utf-8
- * encoded @a path, in the same directory as @a path. The file handle is
+/** Open a new file (for reading and writing) with a unique name based on
+ * utf-8 encoded @a path, in the same directory as @a path. The file handle is
  * returned in @a *f, and the name, which ends with @a suffix, is returned
  * in @a *unique_name_p, also utf8-encoded. Either @a f or @a unique_name_p
  * may be @c NULL.
]]]

Cheers,
Charles

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jul 27 23:46:36 2007

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.