Hi Bert,

rhuij...@apache.org wrote:
Author: rhuijben
Date: Tue Sep  7 07:32:34 2010
New Revision: 993258

URL: http://svn.apache.org/viewvc?rev=993258&view=rev
Log:
* subversion/mod_dav_svn/reports/log.c
  (dav_svn__log_report): Partially revert r993246 to fix compilation.
    The malformed_element_error() function is not defined nor a public
    api.

I was about to commit the attached patch in order to fix this, but I saw your commit before that. Since malformed_element_error() is used in many files I think we make this subversion private.

Thank You.
--
Senthil Kumaran S
http://www.stylesen.org/
[[[
Follow up r993183.

* subversion/mod_dav_svn/reports/log.c
  (malformed_element_error): Add this static function to return
   malformed errors. Ideally this should go into the private headers
   since its used in many files, which I ll do on a separate commit.
]]]

Index: subversion/mod_dav_svn/reports/log.c
===================================================================
--- subversion/mod_dav_svn/reports/log.c        (revision 993224)
+++ subversion/mod_dav_svn/reports/log.c        (working copy)
@@ -82,7 +82,21 @@
   return SVN_NO_ERROR;
 }
 
+/* Return a specific error associated with the contents of TAGNAME
+   being malformed.  Use pool for allocations.  */
+static dav_error *
+malformed_element_error(const char *tagname, apr_pool_t *pool)
+{
+  return dav_svn__new_error_tag(pool, HTTP_BAD_REQUEST, 0,
+                                apr_pstrcat(pool,
+                                            "The request's '", tagname,
+                                            "' element is malformed; there "
+                                            "is a problem with the client.",
+                                            NULL),
+                                SVN_DAV_ERROR_NAMESPACE, SVN_DAV_ERROR_TAG);
+}
 
+
 /* This implements `svn_log_entry_receiver_t'.
    BATON is a `struct log_receiver_baton *'.  */
 static svn_error_t *

Reply via email to