Code

update-server-info: Shorten read_pack_info_file()
authorRalf Thielow <ralf.thielow@googlemail.com>
Mon, 19 Jul 2010 16:26:12 +0000 (18:26 +0200)
committerJunio C Hamano <gitster@pobox.com>
Mon, 19 Jul 2010 18:13:52 +0000 (11:13 -0700)
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>
server-info.c

index 4098ca2b5c166c32cfe4aea9d1bff2e6593e9a60..9ec744e9f2da294a21ac02b46f0ea0b35889ae54 100644 (file)
@@ -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;