Re: [PATCH 0/5] misc cleanups to path functions

From: Junio C Hamano <gitster@pobox.com>
Date: Sep 4, 2012, 19:24 UTC
Message-ID: <7v627ta8wz.fsf@alter.siamese.dyndns.org>
In-Reply-To: <50463981.2020701@ramsay1.demon.co.uk>

Ramsay Jones <ramsay@ramsay1.demon.co.uk> writes:

> These patches are from another unfinished branch. However, this
> time I can't remember why it's unfinished! ;-)

That scares me. Perhaps you found some hard-to-debug problem but didn't have time to follow it through, or something?

;-)

Thanks.

Show 19 quoted lines

>
> ATB,
> Ramsay Jones
>
> Ramsay Jones (5):
>   path.c: Remove the 'git_' prefix from a file scope function
>   path.c: Don't discard the return value of vsnpath()
>   path.c: Use vsnpath() in the implementation of git_path()
>   Call git_pathdup() rather than xstrdup(git_path("..."))
>   Call mkpathdup() rather than xstrdup(mkpath(...))
>
>  bisect.c          |  2 +-
>  builtin/add.c     |  3 ++-
>  builtin/branch.c  |  2 +-
>  builtin/clone.c   |  4 ++--
>  builtin/prune.c   |  2 +-
>  merge-recursive.c | 13 +++++++------
>  path.c            | 28 ++++++++++------------------
>  7 files changed, 24 insertions(+), 30 deletions(-)