Modified as suggested.

Thanks
With regards
Kamesh Jayachandran

-----Original Message-----
From: Blair Zajac [mailto:[email protected]]
Sent: Mon 3/1/2010 10:47 PM
To: [email protected]
Cc: [email protected]
Subject: Re: svn commit: r917512 - in /subversion/trunk/subversion/mod_dav_svn: 
mirror.c mod_dav_svn.c
 
[email protected] wrote:
> Author: kameshj
> Date: Mon Mar  1 13:15:58 2010
> New Revision: 917512
> 
> URL: http://svn.apache.org/viewvc?rev=917512&view=rev
> Log:
> Follow-up to r916286.
> 
> * subversion/mod_dav_svn/mod_dav_svn.c
> (create_dir_config):
> * subversion/mod_dav_svn/mirror.c
> (dav_svn__location_in_filter, dav_svn__location_body_filter):
> Use 'svn_uri_canonicalize' on url paths.

A minor comment on the log messages.  The lines beginning with () are indented 
two spaces and each source file has one vertical whitespace separating it from 
other files.  This makes it much easier to visually parse the log message to 
quickly see what's being changed:

* subversion/mod_dav_svn/mod_dav_svn.c
   (create_dir_config):
   Use 'svn_uri_canonicalize' on url paths.

* subversion/mod_dav_svn/mirror.c
   (dav_svn__location_in_filter, dav_svn__location_body_filter):
   Use 'svn_uri_canonicalize' on url paths.

Regards,
Blair


Reply via email to