git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
* [PATCH] builtin-merge-recursive.c: make merge_recursive() static
@ 2008-07-20 10:21 Nanako Shiraishi
  2008-07-20 12:03 ` Johannes Schindelin
  0 siblings, 1 reply; 3+ messages in thread
From: Nanako Shiraishi @ 2008-07-20 10:21 UTC (permalink / raw)
  To: git; +Cc: Junio C Hamano

This function is not used by any other file.

Signed-off-by: Nanako Shiraishi <nanako3@lavabit.com>
---
 builtin-merge-recursive.c |    2 +-
 merge-recursive.h         |    7 -------
 2 files changed, 1 insertions(+), 8 deletions(-)

diff --git a/builtin-merge-recursive.c b/builtin-merge-recursive.c
index 1ab1945..553aa16 100644
--- a/builtin-merge-recursive.c
+++ b/builtin-merge-recursive.c
@@ -1247,7 +1247,7 @@ static struct commit_list *reverse_commit_list(struct commit_list *list)
  * Merge the commits h1 and h2, return the resulting virtual
  * commit object and a flag indicating the cleanness of the merge.
  */
-int merge_recursive(struct commit *h1,
+static int merge_recursive(struct commit *h1,
 		    struct commit *h2,
 		    const char *branch1,
 		    const char *branch2,
diff --git a/merge-recursive.h b/merge-recursive.h
index f37630a..8924371 100644
--- a/merge-recursive.h
+++ b/merge-recursive.h
@@ -1,13 +1,6 @@
 #ifndef MERGE_RECURSIVE_H
 #define MERGE_RECURSIVE_H
 
-int merge_recursive(struct commit *h1,
-		    struct commit *h2,
-		    const char *branch1,
-		    const char *branch2,
-		    struct commit_list *ancestors,
-		    struct commit **result);
-
 int merge_trees(struct tree *head,
 		struct tree *merge,
 		struct tree *common,
-- 
1.5.6.3

-- 
Nanako Shiraishi
http://ivory.ap.teacup.com/nanako3/

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] builtin-merge-recursive.c: make merge_recursive() static
  2008-07-20 10:21 [PATCH] builtin-merge-recursive.c: make merge_recursive() static Nanako Shiraishi
@ 2008-07-20 12:03 ` Johannes Schindelin
  2008-07-22  8:47   ` Junio C Hamano
  0 siblings, 1 reply; 3+ messages in thread
From: Johannes Schindelin @ 2008-07-20 12:03 UTC (permalink / raw)
  To: Nanako Shiraishi; +Cc: git, Junio C Hamano

Hi,

On Sun, 20 Jul 2008, Nanako Shiraishi wrote:

> This function is not used by any other file.

And I thought it would be, in a future version of builtin-merge...

Of course, we could apply this patch now, and revert it later, increasing 
your commit count in the process :-)

Ciao,
Dscho

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] builtin-merge-recursive.c: make merge_recursive() static
  2008-07-20 12:03 ` Johannes Schindelin
@ 2008-07-22  8:47   ` Junio C Hamano
  0 siblings, 0 replies; 3+ messages in thread
From: Junio C Hamano @ 2008-07-22  8:47 UTC (permalink / raw)
  To: Johannes Schindelin; +Cc: Nanako Shiraishi, git

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

> Of course, we could apply this patch now, and revert it later, increasing 
> your commit count in the process :-)

Heh, don't tempt people, especially when we would have an interesting
set of tools for statistics just around the corner ;-)

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-07-22  8:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-07-20 10:21 [PATCH] builtin-merge-recursive.c: make merge_recursive() static Nanako Shiraishi
2008-07-20 12:03 ` Johannes Schindelin
2008-07-22  8:47   ` Junio C Hamano

Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).