summaryrefslogtreecommitdiffstats
path: root/builtin-apply.c
diff options
context:
space:
mode:
authorJens Lehmann <Jens.Lehmann@web.de>2010-04-09 22:08:35 +0200
committerJunio C Hamano <gitster@pobox.com>2010-04-10 07:42:57 +0200
commit54fd955c211db6f4a2c91301f792ec9de86eed31 (patch)
tree551d42922fa69f92d877a91ef754624848dc12b7 /builtin-apply.c
parentMerge branch 'ef/maint-empty-commit-log' into maint (diff)
downloadgit-54fd955c211db6f4a2c91301f792ec9de86eed31.tar.xz
git-54fd955c211db6f4a2c91301f792ec9de86eed31.zip
Let check_preimage() use memset() to initialize "struct checkout"
Every code site except check_preimage() uses either memset() or declares a static instance of "struct checkout" to achieve proper initialization. Lets use memset() instead of explicit initialization of all members here too to be on the safe side in case this structure is expanded someday. Signed-off-by: Jens Lehmann <Jens.Lehmann@web.de> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to '')
-rw-r--r--builtin-apply.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/builtin-apply.c b/builtin-apply.c
index 7ca90472c1..771c972c55 100644
--- a/builtin-apply.c
+++ b/builtin-apply.c
@@ -2824,11 +2824,8 @@ static int check_preimage(struct patch *patch, struct cache_entry **ce, struct s
if (stat_ret < 0) {
struct checkout costate;
/* checkout */
+ memset(&costate, 0, sizeof(costate));
costate.base_dir = "";
- costate.base_dir_len = 0;
- costate.force = 0;
- costate.quiet = 0;
- costate.not_new = 0;
costate.refresh_cache = 1;
if (checkout_entry(*ce, &costate, NULL) ||
lstat(old_name, st))