Guanqun Lu wrote: > 'xmalloc' followed immediately by 'memset' is replaced > with 'xcalloc', and a simple grep in this project seems > to show that it's the only place. > > Signed-off-by: Guanqun Lu <guanqun.lu@xxxxxxxxx> > --- > sha1_file.c | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/sha1_file.c b/sha1_file.c > index 8868b80..93e5fc0 100644 > --- a/sha1_file.c > +++ b/sha1_file.c > @@ -858,8 +858,7 @@ unsigned char* use_pack(struct packed_git *p, > > static struct packed_git *alloc_packed_git(int extra) > { > - struct packed_git *p = xmalloc(sizeof(*p) + extra); > - memset(p, 0, sizeof(*p)); > + struct packed_git *p = xcalloc(1, sizeof(*p) + extra); > p->pack_fd = -1; > return p; > } I previously scanned through and did this. I left this one as is because the extra part is about as large as the sizeof(*p) part. 69 and 72 bytes respectively on 32-bit. The extra part is always filled in immediately by callers. It's only called once for each pack so it's not performance critical, so maybe your patch is worth it since it is simpler? -brandon -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html