receive-pack: don't copy "dir" parameter
We used to do this so could pass a mutable string to
enter_repo. But since 1c64b48
(enter_repo: do not modify
input, 2011-10-04), this is not necessary.
The resulting code is simpler, and it fixes a minor leak.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:

committed by
Junio C Hamano

parent
e156455ea4
commit
d51428bf17
@ -1125,7 +1125,7 @@ int cmd_receive_pack(int argc, const char **argv, const char *prefix)
|
|||||||
int advertise_refs = 0;
|
int advertise_refs = 0;
|
||||||
int stateless_rpc = 0;
|
int stateless_rpc = 0;
|
||||||
int i;
|
int i;
|
||||||
char *dir = NULL;
|
const char *dir = NULL;
|
||||||
struct command *commands;
|
struct command *commands;
|
||||||
struct sha1_array shallow = SHA1_ARRAY_INIT;
|
struct sha1_array shallow = SHA1_ARRAY_INIT;
|
||||||
struct sha1_array ref = SHA1_ARRAY_INIT;
|
struct sha1_array ref = SHA1_ARRAY_INIT;
|
||||||
@ -1160,7 +1160,7 @@ int cmd_receive_pack(int argc, const char **argv, const char *prefix)
|
|||||||
}
|
}
|
||||||
if (dir)
|
if (dir)
|
||||||
usage(receive_pack_usage);
|
usage(receive_pack_usage);
|
||||||
dir = xstrdup(arg);
|
dir = arg;
|
||||||
}
|
}
|
||||||
if (!dir)
|
if (!dir)
|
||||||
usage(receive_pack_usage);
|
usage(receive_pack_usage);
|
||||||
|
Reference in New Issue
Block a user