[PATCH v2] symlinks.c: small style cleanup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Remove some unnecessary "return;"-statements at the end of "void
foo()"-functions.

Signed-off-by: Kjetil Barvik <barvik@xxxxxxxxxxxx>
---

Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes:

> Hi,
>
> On Sun, 7 Jun 2009, Kjetil Barvik wrote:
>
>> Add {}-braces around an else-part, where the if-part already has
>> {}-braces.
>
> This is the wrong way round.  We prefer _not_ to add unnecessary
> braces, but do tolerate them from time to time.

  OK.  Have taken away that part of the patch in this verson (v2).

  -- kjetil


 symlinks.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/symlinks.c b/symlinks.c
index 1d6b35b..64c8cd3 100644
--- a/symlinks.c
+++ b/symlinks.c
@@ -263,7 +263,6 @@ static void do_remove_scheduled_dirs(int new_len)
 			 removal.path[removal.len] != '/');
 	}
 	removal.len = new_len;
-	return;
 }
 
 void schedule_dir_for_removal(const char *name, int len)
@@ -296,11 +295,9 @@ void schedule_dir_for_removal(const char *name, int len)
 		       last_slash - match_len);
 		removal.len = last_slash;
 	}
-	return;
 }
 
 void remove_scheduled_dirs(void)
 {
 	do_remove_scheduled_dirs(0);
-	return;
 }
-- 
1.6.3.2.277.gd10543

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]