Author: danielsh
Date: Tue Jan 24 04:22:48 2012
New Revision: 1235131
URL: http://svn.apache.org/viewvc?rev=1235131&view=rev
Log:
Document some common knowledge.
* subversion/include/private/svn_repos_private.h
(svn_repos__validate_prop):
Document the implications of making this validator function stricter.
Modified:
subversion/trunk/subversion/include/private/svn_repos_private.h
Modified: subversion/trunk/subversion/include/private/svn_repos_private.h
URL:
http://svn.apache.org/viewvc/subversion/trunk/subversion/include/private/svn_repos_private.h?rev=1235131&r1=1235130&r2=1235131&view=diff
==============================================================================
--- subversion/trunk/subversion/include/private/svn_repos_private.h (original)
+++ subversion/trunk/subversion/include/private/svn_repos_private.h Tue Jan 24
04:22:48 2012
@@ -44,6 +44,14 @@ extern "C" {
*
* Use @a pool for temporary allocations.
*
+ * @note This function is used to implement server-side validation.
+ * Consequently, if you make this function stricter in what it accepts, you
+ * (a) break svnsync'ing of existing repositories that contain now-invalid
+ * properties, (b) do not preclude such invalid values from entering the
+ * repository via tools that use the svn_fs_* API directly (possibly
+ * including svnadmin and svnlook). This has happened before and there
+ * are known (documented, but unsupported) upgrade paths in some cases.
+ *
* @since New in 1.7.
*/
svn_error_t *
