Philip Martin <philip.mar...@wandisco.com> writes:

> jszakmeis...@apache.org writes:
>
>> Author: jszakmeister
>> Date: Tue Jan  4 11:56:18 2011
>> New Revision: 1054989
>>
>> URL: http://svn.apache.org/viewvc?rev=1054989&view=rev
>> Log:
>> Fix syntax inconsistency and update comments.
>>
>> * libsvn_client/locking_commands.c
>>   (organize_lock_targets): Fix syntax inconsistency by removing
>>     unnecessary braces from 'if' block. Update comments to reflect new
>>     function names used.
>>
>> Patch by: Noorul Islam K M <noorul{_AT_}collab.net>
>>
>> Modified:
>>     subversion/trunk/subversion/libsvn_client/locking_commands.c
>>
>> Modified: subversion/trunk/subversion/libsvn_client/locking_commands.c
>> URL: 
>> http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/locking_commands.c?rev=1054989&r1=1054988&r2=1054989&view=diff
>> ==============================================================================
>> --- subversion/trunk/subversion/libsvn_client/locking_commands.c (original)
>> +++ subversion/trunk/subversion/libsvn_client/locking_commands.c Tue Jan  4 
>> 11:56:18 2011
>> @@ -191,18 +191,15 @@ organize_lock_targets(const char **commo
>>  
>>    /* Get the common parent and all paths */
>>    if (url_mode)
>> -    {
>>        SVN_ERR(svn_uri_condense_targets(common_parent_url, &rel_targets,
>>                                         targets, TRUE, pool, pool));
>> -    }
>>    else
>> -    {
>>        SVN_ERR(svn_dirent_condense_targets(common_parent_url, &rel_targets,
>>                                            targets, TRUE, pool, pool));
>> -    }
>
> The remaining lines have the wrong indentation.

My mistake. I am not sure whether another patch is necessary for
this. I am attaching one which fixes the indentation problem.

Log
[[[

Fix wrong indentation caused by r1054989.

* subversion/libsvn_client/locking_commands.c
  (organize_lock_targets): Fix wrong indentation.

]]]

Thanks and Regards
Noorul

Index: subversion/libsvn_client/locking_commands.c
===================================================================
--- subversion/libsvn_client/locking_commands.c (revision 1055082)
+++ subversion/libsvn_client/locking_commands.c (working copy)
@@ -191,11 +191,11 @@
 
   /* Get the common parent and all paths */
   if (url_mode)
-      SVN_ERR(svn_uri_condense_targets(common_parent_url, &rel_targets,
-                                       targets, TRUE, pool, pool));
+    SVN_ERR(svn_uri_condense_targets(common_parent_url, &rel_targets,
+                                     targets, TRUE, pool, pool));
   else
-      SVN_ERR(svn_dirent_condense_targets(common_parent_url, &rel_targets,
-                                          targets, TRUE, pool, pool));
+    SVN_ERR(svn_dirent_condense_targets(common_parent_url, &rel_targets,
+                                        targets, TRUE, pool, pool));
 
   /* svn_uri_condense_targets and svn_dirent_condense_targets leaves
      URLs/paths empty if TARGETS only had 1 member, so we special case

Reply via email to