diff options
author | Shawn Pearce <spearce@spearce.org> | 2006-09-02 00:16:31 -0400 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2006-09-02 03:24:37 -0700 |
commit | 9befac470b4cfad529032dbcffcb71242ec71f91 (patch) | |
tree | f3715172da747456dd63c6d80c3f1e0a7d6f2d56 /blame.c | |
parent | ad1ed5ee896ba5d7f89bc04c7441b1532efb9853 (diff) | |
download | git-9befac470b4cfad529032dbcffcb71242ec71f91.tar.gz |
Replace uses of strdup with xstrdup.
Like xmalloc and xrealloc xstrdup dies with a useful message if
the native strdup() implementation returns NULL rather than a
valid pointer.
I just tried to use xstrdup in new code and found it to be missing.
However I expected it to be present as xmalloc and xrealloc are
already commonly used throughout the code.
[jc: removed the part that deals with last_XXX, which I am
finding more and more dubious these days.]
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'blame.c')
-rw-r--r-- | blame.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -617,7 +617,7 @@ static void simplify_commit(struct rev_info *revs, struct commit *commit) if (new_name) { struct util_info* putil = get_util(p); if (!putil->pathname) - putil->pathname = strdup(new_name); + putil->pathname = xstrdup(new_name); } else { *pp = parent->next; continue; |