[PATCH 0/5] misc cleanups to path functions

Ramsay Jones· Sep 4, 2012, 17:25 UTC · lore

Hi Junio,

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

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(-)

-- 
1.7.12

Junio C Hamano· Sep 4, 2012, 19:24 UTC · re: Ramsay Jones · lore

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

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(-)

patch, 5 partsmisc cleanups to path functions