{"thread":{"id":"19726","subject":"[PATCH v2] symlinks.c: small style cleanup","startedAt":"2009-06-07T14:43:56Z","lastAt":"2009-06-07T14:43:56Z","messageCount":1,"participants":["Kjetil Barvik"],"isPatch":true,"patchVersion":2,"patchTotal":null},"messages":[{"id":"115708","messageId":"1244385836-8540-1-git-send-email-barvik@broadpark.no","threadId":"19726","inReplyTo":null,"subject":"[PATCH v2] symlinks.c: small style cleanup","fromName":"Kjetil Barvik","fromEmail":"barvik@broadpark.no","sentAt":"2009-06-07T14:43:56Z","receivedAt":"2009-06-07T14:43:56Z","isPatch":true,"sender":{"key":"barvik@broadpark.no","avatar":null},"body":"Remove some unnecessary \"return;\"-statements at the end of \"void\nfoo()\"-functions.\n\nSigned-off-by: Kjetil Barvik <barvik@broadpark.no>\n---\n\nJohannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n\n> Hi,\n>\n> On Sun, 7 Jun 2009, Kjetil Barvik wrote:\n>\n>> Add {}-braces around an else-part, where the if-part already has\n>> {}-braces.\n>\n> This is the wrong way round.  We prefer _not_ to add unnecessary\n> braces, but do tolerate them from time to time.\n\n  OK.  Have taken away that part of the patch in this verson (v2).\n\n  -- kjetil\n\n\n symlinks.c |    3 ---\n 1 files changed, 0 insertions(+), 3 deletions(-)\n\ndiff --git a/symlinks.c b/symlinks.c\nindex 1d6b35b..64c8cd3 100644\n--- a/symlinks.c\n+++ b/symlinks.c\n@@ -263,7 +263,6 @@ static void do_remove_scheduled_dirs(int new_len)\n \t\t\t removal.path[removal.len] != '/');\n \t}\n \tremoval.len = new_len;\n-\treturn;\n }\n \n void schedule_dir_for_removal(const char *name, int len)\n@@ -296,11 +295,9 @@ void schedule_dir_for_removal(const char *name, int len)\n \t\t       last_slash - match_len);\n \t\tremoval.len = last_slash;\n \t}\n-\treturn;\n }\n \n void remove_scheduled_dirs(void)\n {\n \tdo_remove_scheduled_dirs(0);\n-\treturn;\n }\n-- \n1.6.3.2.277.gd10543\n"}]}