svn commit: r999164 - in /subversion/branches/atomic-revprop/subversion/include: svn_fs.h svn_ra.h

[da...@apache.org]

Author: danielsh
Date: Mon Sep 20 23:53:34 2010
New Revision: 999164

URL: http://svn.apache.org/viewvc?rev=999164&view=rev
Log:
On the atomic-revprop branch:

Tweak docstrings.

* subversion/include/svn_fs.h
  (svn_fs_change_rev_prop2):
    Tweak text.

* subversion/include/svn_ra.h
  (svn_ra_change_rev_prop2):
    Make the same promise as svn_fs_change_rev_prop2().
    This in particular introduces the "return <this> error code" promise.

Modified:
    subversion/branches/atomic-revprop/subversion/include/svn_fs.h
    subversion/branches/atomic-revprop/subversion/include/svn_ra.h

Modified: subversion/branches/atomic-revprop/subversion/include/svn_fs.h
URL: http://svn.apache.org/viewvc/subversion/branches/atomic-revprop/subversion/include/svn_fs.h?rev=999164&r1=999163&r2=999164&view=diff
==============================================================================
--- subversion/branches/atomic-revprop/subversion/include/svn_fs.h (original)
+++ subversion/branches/atomic-revprop/subversion/include/svn_fs.h Mon Sep 20 23:53:34 2010
@@ -1894,10 +1894,11 @@ svn_fs_revision_proplist(apr_hash_t **ta
  * - @a fs is a filesystem, and @a rev is the revision in that filesystem
  *   whose property should change.
  * - @a name is the name of the property to change.
- * - if @a old_value_p is not @c NULL, then @a *old_value_p is the expected old
- *   value of the property, and changing the value will fail with error
- *   #SVN_ERR_FS_PROP_BASEVALUE_MISMATCH if the present value of the property
- *   is not @a *old_value_p.  (This is an atomic test-and-set).
+ * - if @a old_value_p is not @c NULL, then changing the property will fail with
+ *   error #SVN_ERR_FS_PROP_BASEVALUE_MISMATCH if the present value of the
+ *   property is not @a *old_value_p.  (This is an atomic test-and-set).
+ *   @a *old_value_p may be @c NULL, representing that the property must be not
+ *   already set.
  * - @a value is the new value of the property, or zero if the property should
  *   be removed altogether.
  *

Modified: subversion/branches/atomic-revprop/subversion/include/svn_ra.h
URL: http://svn.apache.org/viewvc/subversion/branches/atomic-revprop/subversion/include/svn_ra.h?rev=999164&r1=999163&r2=999164&view=diff
==============================================================================
--- subversion/branches/atomic-revprop/subversion/include/svn_ra.h (original)
+++ subversion/branches/atomic-revprop/subversion/include/svn_ra.h Mon Sep 20 23:53:34 2010
@@ -747,9 +747,11 @@ svn_ra_get_dated_revision(svn_ra_session
  * If @a value is @c NULL, delete the named revision property.
  *
  * If the server advertises the #SVN_RA_CAPABILITY_ATOMIC_REVPROPS capability
- * and @a old_value_p is not @c NULL, then the property will be changed
- * iff its current value is @a *old_value_p.  (Note that @a *old_value_p
- * may be @c NULL, representing that the property must be currently unset.)
+ * and @a old_value_p is not @c NULL, then changing the property will fail with
+ * error #SVN_ERR_FS_PROP_BASEVALUE_MISMATCH if the present value of the
+ * property is not @a *old_value_p.  (This is an atomic test-and-set).
+ * @a *old_value_p may be @c NULL, representing that the property must be not
+ * already set.
  *
  * If the capability is not advertised, then @a old_value_p MUST be @c NULL.
  *
@@ -757,6 +759,8 @@ svn_ra_get_dated_revision(svn_ra_session
  *
  * Use @a pool for memory allocation.
  *
+ * @see svn_fs_change_rev_prop2().
+ *
  * @since New in 1.7.
  */
 svn_error_t *