From: Ralf Thielow <ralf.thielow@googlemail.com>
Date: Mon, 19 Jul 2010 16:26:12 +0000 (+0200)
Subject: update-server-info: Shorten read_pack_info_file()
X-Git-Tag: v1.7.2~7^2
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=c173dad58787a7f11a526dbcdaa5a2fe9ff1c87f;p=git.git

update-server-info: Shorten read_pack_info_file()

The correct responses to a D and a T line in .git/objects/info/packs
are the same, so combine their case arms.  In both cases we already
‘goto’ out of the switch so while at it, remove a redundant ‘break’
to avoid yet another line of code.

Signed-off-by: Ralf Thielow <ralf.thielow@googlemail.com>
Reviewed-by: Jonathan Nieder <jrnieder <at> gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---

diff --git a/server-info.c b/server-info.c
index 4098ca2b5..9ec744e9f 100644
--- a/server-info.c
+++ b/server-info.c
@@ -113,11 +113,8 @@ static int read_pack_info_file(const char *infofile)
 				goto out_stale;
 			break;
 		case 'D': /* we used to emit D but that was misguided. */
-			goto out_stale;
-			break;
 		case 'T': /* we used to emit T but nobody uses it. */
 			goto out_stale;
-			break;
 		default:
 			error("unrecognized: %s", line);
 			break;