[PATCH] diff-lib.c: don't strdup twice
- From
René Scharfe <rene.scharfe@lsrfire.ath.cx>
- Date
- Jul 7, 2007, 18:19 UTC
- Message-ID
- <468FD91C.7040506@lsrfire.ath.cx>
The static function read_directory in diff-lib.c is only ever called with struct path_list lists with .strdup_paths turned on, i.e. path_list_insert will strdup the paths for us (again). Let's take advantage of that and stop doing it twice.
Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx> ---
diff-lib.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/diff-lib.c b/diff-lib.c index 7fb19c7..92c0e39 100644 --- a/diff-lib.c +++ b/diff-lib.c @@ -24,7 +24,7 @@ static int read_directory(const char *path, struct path_list *list) while ((e = readdir(dir))) if (strcmp(".", e->d_name) && strcmp("..", e->d_name)) - path_list_insert(xstrdup(e->d_name), list); + path_list_insert(e->d_name, list); closedir(dir); return 0;